6.0.0-beta6
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
4/10/26
H
istory
A
ttachments
C
omment
W
atch
Download
Comment on [#6967] user data removal patch
*
Your Email Address
*
Spam protection
Enter the letters below:
.___. __ . ..___..__ _/ / `|__| | [__) ./__.\__.| | | |
Comment
>> As far as I understand from Chuck's comment, you only need this > >> because you are using the cyrsql driver and don't want to stop > >> removing the user if parent::removeUser() is returning an error due > >> to the removeUserData() call in the sql driver. Is this correct? > > > > Yes. > > > >> But you patch unconditionally removes returning errors from > >> Auth_sql::removeUser() so this won't report any errors for people > >> using the sql driver directly. This is of course not acceptable. > > > > Ghm, correct me if I'm wrong but isn't application's removeUserData() > call reports these errors anyway? With my patch and *without* > http://bugs.horde.org/ticket/6968 patches I'm still getting errors > from nag, mnemo etc. BUT the user is removed succefully from the > backend. What is the difference between calling removeUser() from > Auth_cyrsql and Auth_sql? As I see in the source it executes exactly > the same API call in the needed applications.
Attachment
Watch this ticket
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers