Error message display if virtual mailbox selected

[toony]

If a virtual mailbox is selected in folder tree, roundcube display an STATUS error message (mailbox does not exist).

Reproduce this:

• got rights on a share folder or mailbox
• select this share folder or mailbox in folder tree
• collapse virtual folder or mailbox tree

[rosali]

I can confirm the issue, but the patch does not work for me. I'm on hMailserver and there are Public Folders like '#Public'. I don't see anything in the IMAP logs which could help to decide if the selected folder is a virtual folder or not. Perhaps we can have config value to setup Roundcube according to hMailserver settings for virtual folders. I think about something like:

$rcmail_config['virtual_folders'] = array(
  '#Public',
  '#Foo',
  '#Bar',
);

[alec]

The question is, why this folder is "virtual"? Enable imap_debug and provide the log.

[toony]

Replying to toony:

If a virtual mailbox is selected in folder tree, roundcube display an STATUS error message (mailbox does not exist).

Reproduce this:

• got rights on a share folder or mailbox
• select this share folder or mailbox in folder tree
• collapse virtual folder or mailbox tree

I'm using Cyrus IMAPd 2.4.13

[toony]

Replying to rosali:

I can confirm the issue, but the patch does not work for me. I'm on hMailserver and there are Public Folders like '#Public'. I don't see anything in the IMAP logs which could help to decide if the selected folder is a virtual folder or not. Perhaps we can have config value to setup Roundcube according to hMailserver settings for virtual folders. I think about something like:

$rcmail_config['virtual_folders'] = array(
  '#Public',
  '#Foo',
  '#Bar',
);

Did you mean that you can select this entry in roundcube folder tree ?

[alec]

This is about proper handling of folder collapse action. It is not possible to select virtual folder by clicking it's name, but when a folder which is a child of virtual folder is selected we can collapse the virtual folder. In that situation normally we are selecting collapsed parent, but in case of virtual folder we have to:

• do not change the selection
  
• select INBOX folder

[toony]

Replying to alec:

This is about proper handling of folder collapse action. It is not possible to select virtual folder by clicking it's name, but when a folder which is a child of virtual folder is selected we can collapse the virtual folder. In that situation normally we are selecting collapsed parent, but in case of virtual folder we have to:

• do not change the selection
  
• select INBOX folder

Propose an improved patch (​http://trac.roundcube.net/attachment/ticket/1488346/client_-_dontRefreshFolderContentIfVirtual.patch).
Replace other. Client do stuff.

I prefer to unchange selection, because it's more "user compliant" I think.
Changing selected folder for INBOX or collapsed parent folder may confuse users.

[alec]

Fixed in [1837c330] (trunk) and r5952/svn (0.7).