Hi,

I have a fix for #1484681 (see patch in the ticket) but I'm not sure if I missed something.

The function subscribe() in program/include/rcube_imap.inc splits mailboxes into an array with a comma as delimiter. That's the source of the problem in this ticket.

It splits into an array because that's what _change_subscription() wants. that's OK.

The array should be splittable from a text string because change_subscription() in apps.js may send more than one folder. Understandable.

But: change_subscription() in apps.js is never called. Ever.

Thus I removed it, and made sure subscribe() and unsubscribe() call _change_subscription() with an array as first argument.

Is this okay? Is change_subscription() an old artefact, or is it needed somewhere else in the future?

Robin _______________________________________________ List info: http://lists.roundcube.net/dev/