Tickets :: Comment on [#11983] limit synced mailboxes by user preferences

* Your Email Address
* Spam protection	Enter the letters below: . ..__.. .. . __. \|\ \|\| \|\|__\|\| \|(__ \| \\|\|__\|\| \|\|/\\|.__)
Comment	> >> If I had to call _getMailFolder() for every folder returned by >> _imap->getMailboxes(), wouldn't this get too expensive? My list may >> well be longer than 1k entries... each tme creating an instance of >> Horde_ActiveSync_Message_Folder, calling getSpecialMailboxes() etc, >> and then dumping the result in all but a handful of cases. > > Well, getSpecialMailboxes() already caches the list so we only hit > the imap server once. > >> Purely functionally speaking, the quickest approach would be to call >> _imap->getSpecialMailboxes() only once, re-map to an "folder >> name"-based array, then check every "_imap->getMailboxes()" result >> against that new array. Does that sound acceptable to you? > > Feel free to submit a patch and I'll review.
Attachment
Watch this ticket