Thanks for looking into this problem and making suggestions. However, can we please move this to android-platform, because that is a better forum for discussion of changes & work on the platform and the platform apps.
We're ramping up on accepting changes (slowly, but surely) and patches are welcome. When you work on this problem, please take a look at the existing unit tests (for Email) and be sure that any changes you make are matched by new unit tests as well. Finally, to be honest, 292 is a lot of folders, and there may be far more issues than simply improving the IMAP protocol aspects of this problem. Thanks, Andy On Thu, Jan 22, 2009 at 8:01 PM, [email protected] <[email protected]> wrote: > > This code is found in: > src/com/android/email/mail/store/ImapStore.java > > > Okay, so I have a beef with the E-mail program provided by default and > I see room for change. My problem is that they use the IMAP command > LIST to list each folder, the problem with this is that, if you are > like me, with a large organization, you have a plethora of IMAP > folders. I haven't counted, but I'd estimate that we have around 292 > and when I use the IMAP client built in, it loads all these folders, > and it takes it's sweet time. > > connection.executeSimpleCommand(String.format("LIST \"\" \"%s* > \"",mPathPrefix == null ? "" : mPathPrefix)); > > Is the line that the code uses, which has a problem, because it's > telling the imap server it wants to list all the folders, whether you > are subscribed or not... > > A simple change to this would be > > connection.executeSimpleCommand(String.format("LSUB \"\" \"%s* > \"",mPathPrefix == null ? "" : mPathPrefix)); > > The LSUB command will only display the folders you are subscribed to, > instead of every folder. A better version would be the > > connection.executeSimpleCommand(String.format("LSUB \"\" \"%s% > \"",mPathPrefix == null ? "" : mPathPrefix)); > > so that it would fold the folders first, then if a user click on it, > it can expand, swapping the % for a *. > > > > If they are worried about not being able to see unsubscribed folders, > a new option can be coded to allow for > > connection.executeSimpleCommand(String.format("LIST \"\" \"%s* > \"",mPathPrefix == null ? "" : mPathPrefix)); > > and simple have the user pick which folders they'd like to subscribe > to, if they have the permissions. > > > I have not yet git the entire tree and attempted to recompile the > emulator to reflect this change, but I will to test out my change and > I'll let you all know if this works, just wanted to get some feedback > on this problem. > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Android Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/android-developers?hl=en -~----------~----~----~----~------~----~------~--~---
