Tickets :: [#9057] Horde_Perms / Horde_Perms_Ui broken after refactor to Horde_Db

6.0.0-beta1

7/7/25

Summary	Horde_Perms / Horde_Perms_Ui broken after refactor to Horde_Db
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	Horde Developers (at) , mrubinsk (at) horde (dot) org
Requester	mrubinsk (at) horde (dot) org
Created	05/24/2010 (5523 days ago)
Due
Updated	05/24/2010 (5523 days ago)
Assigned	05/24/2010 (5523 days ago)
Resolved	05/24/2010 (5523 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

05/24/2010 09:35:36 PM	Michael Rubinsky	Comment #8 State ⇒ Resolved	Reply to this comment
now it's fixed. Had reversed logic for deleting children. The rest of the issues were due to a buggy development build of xcache I was using.

05/24/2010 08:57:22 PM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git for this ticket: fix reveresed logic ~~Bug: 9057~~ http://git.horde.org/diff.php/framework/Perms/lib/Horde/Perms/Sql.php?rt=horde-git&r1=61cbfe4b3fd5b2b4939499e1648bcab186b8d290&r2=7c2b6ce5bf96c6c7d7ee5790ead47ad63817068a

05/24/2010 08:39:56 PM	Michael Rubinsky	Comment #6 Assigned to Horde Developers State ⇒ Assigned	Reply to this comment
I spoke too soon. There still seems to be some weirdness with the parent relationship. Adding new perms sometimes fails, and afterwords, the UI interface fails to load. Also, deleting perms with children doesn't seem to delete the children from the db.

05/24/2010 08:32:34 PM	Michael Rubinsky	Comment #5	Reply to this comment
Which makes this portion a duplicate of ~~Ticket #9049~~ (although, this fix is ok for the interim until we figure out the auto-increment migration stuff). Exactly. horde was unusable for me, due to fatals from Horde_Perms. This at least keeps me and others with the same issue running in the meantime.

05/24/2010 08:16:24 PM	Michael Rubinsky	Comment #4 Taken from Horde Developers State ⇒ Resolved	Reply to this comment
Seems to be working now.

05/24/2010 07:03:12 PM	Michael Slusarz	Comment #3	Reply to this comment
sql upgrade scripts are added, someone with more knowledge of pgsql and oracle might want to take a look though. http://lists.horde.org/archives/commits/2010-May/003977.html This upgrades the scripts - but my understanding is that we need to do the upgrade via migration scripts, not SQL files. Which makes this portion a duplicate of ~~Ticket #9049~~ (although, this fix is ok for the interim until we figure out the auto-increment migration stuff).

05/24/2010 06:55:14 PM	Michael Rubinsky	Comment #2 Assigned to Michael Rubinsky Taken from Michael Slusarz	Reply to this comment
sql upgrade scripts are added, someone with more knowledge of pgsql and oracle might want to take a look though. http://lists.horde.org/archives/commits/2010-May/003977.html

05/24/2010 05:34:24 PM	Michael Rubinsky	Comment #1 Priority ⇒ 1. Low State ⇒ Assigned Patch ⇒ No Milestone ⇒ Assigned to Horde Developers Assigned to Michael Slusarz Queue ⇒ Horde Framework Packages Summary ⇒ Horde_Perms / Horde_Perms_Ui broken after refactor to Horde_Db Type ⇒ Bug	Reply to this comment
- Need an upgrade script for the horde_perms tables to use autoincrement functionality - Adding child permissions is broken - ?