Tickets :: [#13008] Basic view: Crash on mailbox delete / rename

6.0.0-alpha14

7/2/25

Summary	Basic view: Crash on mailbox delete / rename
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	thomas.jarosch (at) intra2net (dot) com
Created	03/03/2014 (4139 days ago)
Due
Updated	03/04/2014 (4138 days ago)
Assigned	03/03/2014 (4139 days ago)
Resolved	03/04/2014 (4138 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

03/04/2014 10:49:12 AM	Thomas Jarosch	Comment #7	Reply to this comment
was fixed by the earlier commit for ~~#13000~~ rubbish, the bug number should have been ~~#12998~~ double-fail: The correct one is ~~#12999~~ Just a guess though. It's fixed anyway :)

03/04/2014 10:47:50 AM	Thomas Jarosch	Comment #6	Reply to this comment
was fixed by the earlier commit for ~~#13000~~ rubbish, the bug number should have been ~~#12998~~

03/04/2014 10:46:54 AM	Thomas Jarosch	Comment #5 State ⇒ Resolved	Reply to this comment
[Show Quoted Text - 13 lines][Hide Quoted Text] Though I get an error message from the IMAP server (INBOX/xxx not found) when I try to delete a folder. Can't reproduce. Rename is even funnier, it always shows a different folder name than I selected in the popup. I was actually getting a fatal error, so I couldn't reproduce this behavior. So I guess a more thorough fix is needed than to add "global $injector" ;) That's all I needed to fix (at least for delete). Rename and delete work now for me, too. May be the "delete" problem was fixed by the earlier commit for ~~#13000~~ Thanks!

03/03/2014 07:36:14 PM	Michael Slusarz	Comment #4 Assigned to Michael Slusarz State ⇒ Feedback	Reply to this comment
Though I get an error message from the IMAP server (INBOX/xxx not found) when I try to delete a folder. Can't reproduce. Rename is even funnier, it always shows a different folder name than I selected in the popup. I was actually getting a fatal error, so I couldn't reproduce this behavior. So I guess a more thorough fix is needed than to add "global $injector" ;) That's all I needed to fix (at least for delete).

03/03/2014 07:33:47 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit ae66256416ddd4db3ffeff474a8f3c104ddd96d9 Author: Michael M Slusarz <slusarz@horde.org> Date: Mon Mar 3 12:24:45 2014 -0700 ~~Bug #13008~~: Fix rename imp/lib/Ftree.php \| 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/ae66256416ddd4db3ffeff474a8f3c104ddd96d9

03/03/2014 07:33:45 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git (master): commit b3a1098975068d5b4e1c5b8a51afa820f3dae149 Author: Michael M Slusarz <slusarz@horde.org> Date: Mon Mar 3 12:17:21 2014 -0700 ~~Bug #13008~~: fix global usage imp/lib/Imap.php \| 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/b3a1098975068d5b4e1c5b8a51afa820f3dae149

03/03/2014 10:38:23 AM	Thomas Jarosch	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Basic view: Crash on mailbox delete / rename Queue ⇒ IMP Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Hi, the mailbox delete / rename folder action is currently crashing: [Mon Mar 03 11:29:59 2014] [error] [client 172.16.1.253] PHP Fatal error: Call to a member function getInstance() on a non-object in /datastore/DEVEL/horde/imp/lib/Imap.php on line 731, referer: https://172.16.1.123/horde/imp/basic.php?Horde=jf96nm905ohak09ercalh13fq6&page=folders&token=1ONAcdrxcYCqXA9mSJkmxA2 -> "global $injector" is missing in the function. Though I get an error message from the IMAP server (INBOX/xxx not found) when I try to delete a folder. Rename is even funnier, it always shows a different folder name than I selected in the popup. So I guess a more thorough fix is needed than to add "global $injector" ;) Thomas