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/

Reply via email to