[Tickets #6962] Re: recursive delete

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/6962
------------------------------------------------------------------------------
  Ticket             | 6962
  Updated By         | Jan Schneider <jan at horde.org>
  Summary            | recursive delete
  Queue              | Trean
  Type               | Enhancement
  State              | Accepted
  Priority           | 2. Medium
  Milestone          | 1.0
  Patch              |
  Owners             |
------------------------------------------------------------------------------


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

>> If designed properly, the Horde 3.2.1 Share drivers would simply
>> ignore the parameter and return an error if we try to delete a folder
>> which contains subfolders. 3.2.2 drivers can do the recursive
>> deletion.
>
> Yea. Good point.  I guess we could also put a not-as-efficient
> recursive delete in Trean as well to keep the behavior consistent, if
> the Share driver returns the error.

No, better not. That would only be required for BC for one or two  
Horde releases. That's not worth it. Plus, if there really is an error  
deleting folders and you try to do it recursively in Trean, you would  
probably end up with an error message for each folder, which could be  
a lot.

> Also, I think I'd be more
> comfortable if there was an additional confirmation, or at the least,
> tweak the text of the existing confirmation to indicate your about to
> possibly nuke more than you realize.

Yes.