Tickets :: 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