6.0.0-beta1
7/6/25

[#10802] dimp: folder view crashes
Summary dimp: folder view crashes
Queue IMP
Queue Version 5.0.15
Type Bug
State Resolved
Priority 2. Medium
Owners slusarz (at) horde (dot) org
Requester rsalmon (at) mbpgroup (dot) com
Created 11/25/2011 (4972 days ago)
Due
Updated 12/15/2011 (4952 days ago)
Assigned 11/30/2011 (4967 days ago)
Resolved 12/06/2011 (4961 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
12/15/2011 09:04:38 PM Michael Slusarz Comment #12 Reply to this comment
You are correct - we have already stripped off namespace info so we 
need to compare the special mailboxes, which contain the ns info, to 
the original mailbox name.  Fixed.

For some reason the commit message did not appear here:

http://git.horde.org/horde-git/-/commit/d7c32d1d9bf2b4d0df0b983aa81f42859f0f9f52


12/15/2011 04:52:28 PM rsalmon (at) mbpgroup (dot) com Comment #11 Reply to this comment
possible fix :


--$out = substr_replace($out, Horde_String::convertCharset($val, 
'UTF-8', 'UTF7-IMAP'), 0, $len);
++$out = substr_replace($this->_mbox, 
Horde_String::convertCharset($val, 'UTF-8', 'UTF7-IMAP'), 0, $len);
12/15/2011 02:07:27 PM rsalmon (at) mbpgroup (dot) com Comment #10
New Attachment: hordelog_sent.txt Download
Reply to this comment
I see this issue using either traditional or dynamic view.

attached the trace.

The name changes to the wrong value over here :
imp/lib/Mailbox.php:protected function _getDisplay
...

horde::debug($this);
horde::debug($sub);
         foreach ($sub as $key => $val) {
             if (strlen($key) &&
                 (($key != 'INBOX') || ($this->_mbox == $out)) &&
                 strpos($this->_mbox, $key) === 0) {
                 $len = strlen($key);
                 if ((strlen($this->_mbox) == $len) || 
($this->_mbox[$len] == (is_null($ns_info) ? '' : 
$ns_info['delimiter']))) {
                     horde::debug($out);
                     $out = substr_replace($out, 
Horde_String::convertCharset($val, 'UTF-8', 'UTF7-IMAP'), 0, $len);
                     horde::debug($out);
                     break;
                 }
             }
         }

12/12/2011 10:52:44 PM Michael Slusarz Comment #9 Reply to this comment
You'll need to track this down.
12/07/2011 09:34:33 AM rsalmon (at) mbpgroup (dot) com Comment #8 Reply to this comment
I don't use multiple identities

$_prefs['subscribe']['value'] = 0;
$_prefs['subscribe']['locked'] = true;


12/07/2011 09:31:45 AM rsalmon (at) mbpgroup (dot) com Comment #7
New Attachment: screnshot.jpg Download
Reply to this comment
Closing, as I was able to reproduce and then verify that this patch 
fixed things.
I don't have the javascript errors anymore, but the labels are wrong 
and only for folders under .Sent, see screenshot. I have the following 
folders :

.Sent
.Sent.test
.Sent.2010


I still don't see any errors/warnings.


12/06/2011 06:49:11 AM Michael Slusarz Comment #6
State ⇒ Resolved
Reply to this comment
Closing, as I was able to reproduce and then verify that this patch 
fixed things.
12/02/2011 10:15:51 PM Michael Slusarz Comment #5
Assigned to Michael Slusarz
Priority ⇒ 2. Medium
Reply to this comment
Finally reproduced.  Try this commit.
12/02/2011 10:15:31 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git for this ticket:

[mms] Don't list parent mailboxes after child mailboxes (Bug #10802).

  3 files changed, 3 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/90b42223e0735a4823c81b2b4cfd24a100915aea
12/01/2011 08:31:05 AM rsalmon (at) mbpgroup (dot) com Comment #3
New Attachment: screenshot.png Download
Reply to this comment
I don't see this, and I have this mailbox layout:

Sent (special mailbox)
   - Horde (special mailbox)
   - Test (regular mailbox)

I correctly see Sent and Horde in the top part, and a Sent container 
and Sent.Test on the bottom.
I don't see any error in any log file.
IMP seems to be getting the right info :

S: * LIST (\HasNoChildren) "." "INBOX.Sent.test"


But, I'm still getting the javascript error in DIMP, and in IMP the 
folder tree doesn't look right (see attached screenshot)





11/30/2011 05:18:42 PM Jan Schneider State ⇒ Feedback
 
11/28/2011 07:08:07 AM Michael Slusarz Comment #2 Reply to this comment
I don't see this, and I have this mailbox layout:

Sent (special mailbox)
   - Horde (special mailbox)
   - Test (regular mailbox)

I correctly see Sent and Horde in the top part, and a Sent container 
and Sent.Test on the bottom.
11/25/2011 02:54:55 PM rsalmon (at) mbpgroup (dot) com Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ dimp: folder view crashes
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
I get the following javascript error :
doActionComplete
TypeError: $(this.getSubMboxId(this.getMboxId(ob.pa))) is null
  ? $(this.getSubMboxId(this.getMboxId(ob.pa))).down()  dimpbase.js 
(ligne 2948)


prototype.js:1825 element = document.getElementById(element);    returns null

I get the same error using git (up to date)
I use courier-imap 4.9.3

This happens when a user has folders under Sent: eg :

.Sent
.Sent.2010
.Sent.2011

Saved Queries