[Tickets #6967] Re: user data removal patch

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6967
------------------------------------------------------------------------------
  Ticket             | 6967
  Updated By         | Jan Schneider <jan at horde.org>
  Summary            | user data removal patch
  Queue              | Horde Framework Packages
  Version            | HEAD
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2008-06-26 07:38) wrote:

>> 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.