Tickets :: [#6928] trean: Cannot delete folders with subfolders

6.0.0-alpha12

6/12/25

Summary	trean: Cannot delete folders with subfolders
Queue	Trean
Type	Bug
State	Resolved
Priority	1. Low
Owners	mrubinsk (at) horde (dot) org
Requester	felilx.leimbach (at) gmx (dot) net
Created	06/15/2008 (6206 days ago)
Due
Updated	06/20/2008 (6201 days ago)
Assigned	06/15/2008 (6206 days ago)
Resolved	06/15/2008 (6206 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

06/20/2008 02:54:32 PM	felix (dot) leimbach (at) gmx (dot) net	Comment #8	Reply to this comment
The bug fixed, so this ticket is correctly resolved. If you want a different behavior, file a request ticket. Alright: http://bugs.horde.org/ticket/6962 Thanks for the clarification!

06/20/2008 02:43:20 PM	Jan Schneider	Comment #7	Reply to this comment
Can't agree to setting this to "Resolved" as it is till impossible to delete folders with children. The bug fixed, so this ticket is correctly resolved. If you want a different behavior, file a request ticket.

06/20/2008 02:20:59 PM	felix (dot) leimbach (at) gmx (dot) net	Comment #6	Reply to this comment
Can't agree to setting this to "Resolved" as it is till impossible to delete folders with children. From a test-import I'm left with a folder which has approx. 500 nested children ... Yea, I know ways to get them deleted but I think the default delete function should do a recursive delete. It asks the user for confirmation already, so there's no danger IMHO.

06/15/2008 06:46:45 PM	felilx (dot) leimbach (at) gmx (dot) net	Comment #5	Reply to this comment
http://cvs.horde.org/diff.php/trean/edit.php?r1=1.54&r2=1.55&ty=u With that patch there's a error message displayed saying how many children exist. That's a progress, but still I'd love to have a recursive delete.

06/15/2008 06:41:02 PM	felilx (dot) leimbach (at) gmx (dot) net	Comment #4	Reply to this comment
I'm assuming that the 2006 is a typo? yea, right, that should have been 2008

06/15/2008 06:02:58 PM	Michael Rubinsky	Taken from Horde Developers State ⇒ Resolved

06/15/2008 06:02:09 PM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/trean/edit.php?r1=1.54&r2=1.55&ty=u

06/15/2008 05:36:50 PM	Michael Rubinsky	Assigned to Horde Developers Assigned to Michael Rubinsky State ⇒ Assigned

06/15/2008 05:35:07 PM	Michael Rubinsky	Comment #2	Reply to this comment
Using trean-cvs from 2006.06.14 I'm assuming that the 2006 is a typo?

06/15/2008 05:10:35 PM	felilx (dot) leimbach (at) gmx (dot) net	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ trean: Cannot delete folders with subfolders Queue ⇒ Trean Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Using trean-cvs from 2006.06.14 it is not possible to delete folders which do have subfolders. No message is displayed. The page is just reloaded and the folder still exists. Deleting folders without subfolders works. I tracked down the execution path to edit.php where in line 268 the delete() method of the $folder object is called as one would expect. The problem must be later on in the delete method.