User Agent: Mozilla/5.0 (Windows NT 6.1; rv:7.0.1) Gecko/20100101 Firefox/7.0.1 Build ID: 20110928134238 Steps to reproduce: Checking a large (1000 message) IMAP folder for junk mail. Selected folder, selected Tools->Run Junk Mail Controls on Folder Actual results: the first approximately 20 messages were marked junk (correctly) in approximately one second. It then stopped checking and the remaining 980 messages were not checked to be junk or not. Error Console: gMessengerBundle is not defined chrome://messenger/content/junkCommands.js Line: 276 Expected results: The whole folder should have been checked for junk.
I can confirm I'm getting this as well.
I'm also seeing this. Hopefully it will stay reproducible so I can track it down. I'm seeing it mostly on Mark/Run Junk Mail Controls
(In reply to Kent James (:rkent) from comment #2) > I'm also seeing this. Hopefully it will stay reproducible so I can track it > down. I'm seeing it mostly on Mark/Run Junk Mail Controls My money is on an ill-advised NS_ENSURE_SUCCESS(rv, rv); ;-)
Created attachment 580699 [details] [diff] [review] fix David, you could loose the money this time ;-) This fixed it for me.
Can somebody look at the patch? This may even be nice for TB9/10. It should be really low risk for a high reward (unbreaking a function exposed in menu).
Comment on attachment 580699 [details] [diff] [review] fix thx very much for the patch; sorry for the delay.
This was a quick fix, however there is a comment at the top of the file, that gMessengerBundle is a global prerequisite. It may be worth investigating why it is no longer defined here.
Our junk command tests didn't catch this because they complete too soon for the statusDisplayInterval. :(
Comment on attachment 580699 [details] [diff] [review] fix a=Standard8 for Aurora, too late for TB 9.
Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/962c4118375a
(In reply to Siddharth Agarwal [:sid0] from comment #9) > Our junk command tests didn't catch this because they complete too soon for > the statusDisplayInterval. :( needs a follow up bug for tests? http://getsatisfaction.com/mozilla_messaging/topics/run_junk_mail_controls_on_folder_mostly_inoperable_in_versions_6_7 reports this as happening at least as early as version 6 looks like this made the merge for v10 beta which is dated 12/22, correct? ftp://ftp.mozilla.org/pub/mozilla.org/thunderbird/releases/10.0b1/win32/en-US/
Per comment 10 and comment 11 this should be in TB 10 (Aurora at that time, now Beta). I asked for backport to sooner releases than TB11. TB9 didn't make it as it had only 1 day before release.