recursive delete of folder - change (fatal) error message

If you will try to delete a not empty folder and you did not

Fri, 01 Jun 2012 13:47:20 +0000

If you will try to delete a not empty folder and you did not change your settings to allow recursive delets you get a fatal error page. I would say this error report/page is "wrong" since it is not a software bug or any other serious problem. I would prefere to just send a note to the user that the folder is not empty and so it will not be deleted. Maybe add a note about changing this behavior in the user settings.... Regards, Stephan

And what is the (full) fatal error?

Mon, 04 Jun 2012 12:12:39 +0000

And what *is* the (full) fatal error?

> And what is the (full) fatal error? ? "Unable to del

Mon, 04 Jun 2012 12:19:50 +0000

> And what *is* the (full) fatal error? ? "Unable to delete "/Folder", the directory is not empty" - everybody knows I thought... see attached recursive_delete-Horde_Fatal_Error.html

Makes sense to me. The Gollem:: class currently leaks Horde_

Fri, 12 Oct 2012 13:40:37 +0000

Makes sense to me. The Gollem:: class currently leaks Horde_Vfs_Exceptions that are not caught in the controller scripts. Don't know if this was intentional or not, but we need to either always throw Gollem_Exceptions, or catch the Vfs_Exception in the calling code.

Changes have been made in Git (master): commit 63f19ac6dd4d

Mon, 26 Nov 2012 10:33:57 +0000

Changes have been made in Git (master): commit 63f19ac6dd4db646bf2433fedf557a597cca6acd Author: Jan Schneider Date: Mon Nov 26 11:33:24 2012 +0100 Consistently throw Gollem_Exceptions from Gollem:: class (Bug #11208). gollem/lib/Gollem.php | 134 ++++++++++++++++++++++++++++++++----------------- gollem/manager.php | 14 +++--- 2 files changed, 94 insertions(+), 54 deletions(-) http://git.horde.org/horde-git/-/commit/63f19ac6dd4db646bf2433fedf557a597cca6acd