Thu, 20 Nov 2008 08:15:20 -0500 http://bugs.horde.org/ticket/6968 can not remove user data when deleting user User data removal for turba, nag, mnemo and kronolith fails if SQL share backend is used. Turba fails with: There was an error removing an address book for username Nag and others fail with: Permission Denied Mon, 23 Jun 2008 15:26:05 -0400 http://bugs.horde.org/ticket/6968#t46737 I don't think this is an issue with the share driver. At least in the case of Turba, it is most likely being caused by a source that does not support removing user data. I added some error logging, and (temporarily?) removed the error flag in this case... not sure about the other apps yet, still digging. Mon, 23 Jun 2008 16:34:06 -0400 http://bugs.horde.org/ticket/6968#t46739 I'm using pretty default sources, e.g. localsql, so this should not be the problem. Mon, 23 Jun 2008 16:42:59 -0400 http://bugs.horde.org/ticket/6968#t46743 > I'm using pretty default sources, e.g. localsql, so this should not > be the problem. right, but if you have any shares other than a sql share source...such as the favourites driver, or even any virtual address books you will get the error I'm referring to. Check the logs after applying the error logging changes, the Turba error should be gone. That being said, that's not the root of the problem. The problem is not specific to the sql share driver. The issue is that when you are deleting a user in Nag, for instance, the Nag::listTaskList() method always checks perms for the *currently logged in* user, not the user who you are deleting...so the task list is not returned. These methods need to be changed to allow filtering perms for other users as well. Mon, 23 Jun 2008 17:15:29 -0400 http://bugs.horde.org/ticket/6968#t46744 Setting this to Nag (and will create a duplicate in Mnemo as well), since this is not an issue with any of the share drivers. Mon, 23 Jun 2008 17:16:44 -0400 http://bugs.horde.org/ticket/6968#t46745 Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/nag/lib/api.php?r1=1.182&r2=1.183&ty=u Mon, 23 Jun 2008 22:22:30 -0400 http://bugs.horde.org/ticket/6968#t46752 fixed for 2.2.1 Mon, 23 Jun 2008 22:26:28 -0400 http://bugs.horde.org/ticket/6968#t46753 With latest api.php patches I don't get error message anymore, but actual data from backend is not cleared for nag and mnemo. [nag|mnemo]_shares and [nag|mnemo]_shares_users are still populated. In Turba on the other hand, data is removed from turba_shares but NOT from turba_shares_users. Sat, 28 Jun 2008 02:40:02 -0400 http://bugs.horde.org/ticket/6968#t46955 Please track Turba issues in a separate ticket. Sat, 28 Jun 2008 10:29:02 -0400 http://bugs.horde.org/ticket/6968#t46956 Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/mnemo/lib/api.php?r1=1.87&r2=1.88&ty=u http://cvs.horde.org/diff.php/nag/lib/Nag.php?r1=1.173&r2=1.174&ty=u http://cvs.horde.org/diff.php/nag/lib/api.php?r1=1.184&r2=1.185&ty=u Sat, 28 Jun 2008 10:54:38 -0400 http://bugs.horde.org/ticket/6968#t46957 Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Share/Share/sql.php?r1=1.35&r2=1.36&ty=u Sat, 28 Jun 2008 12:29:33 -0400 http://bugs.horde.org/ticket/6968#t46958 Should be fixed now in both HEAD and FW_3. There were a few things going on. First we were never even attempting to delete the actual share, just the data in Mnemo and Nag. Second, we were not removing any permissions the removed user may have had in *other* user's shares...and finally, there was a bug in the SQL Horde_Share driver that was causing only the share data itself, and not the user or group perm data to be deleted when removing a share...all have (hopefully) been fixed. Sat, 28 Jun 2008 18:27:15 -0400 http://bugs.horde.org/ticket/6968#t46959 > Should be fixed now in both HEAD and FW_3. There were a few things > going on. First we were never even attempting to delete the actual > share, just the data in Mnemo and Nag. Second, we were not removing Confirmed as fixed. > any permissions the removed user may have had in *other* user's > shares... Confirmed as fixed. > and finally, there was a bug in the SQL Horde_Share driver > that was causing only the share data itself, and not the user or > group perm data to be deleted when removing a share...all have > (hopefully) been fixed. Confirmed as fixed except for Kronolith. Per Chuck's request tracking this under bug 6998. Also filled another related issue under bug 6999. All tests were performed on FW_3 code. Mon, 30 Jun 2008 13:35:05 -0400 http://bugs.horde.org/ticket/6968#t47015