Tickets :: Comment on [#6967] user data removal patch

* Your Email Address
* Spam protection	Enter the letters below: . ..___.___..__ . . \|_/ [__ \| [__)\|\/\| \| \[___ \| \| \\| \|
Comment	>>> 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 > > > > Yes, but it no longer is passed through Auth_sql::removeUser() to the > admin interface calling removeUser(). Only if using the cyrsql > driver, but not if using the sql driver.
Attachment
Watch this ticket