Bug 358121 fixed a similar problem by adding a try/catch around the line folder = aUrl.folder; but the line immediately after includes a test for null 'folder' before testing for the foldertype being 'nntp' or 'rss' -- because 'folder' is null, when you open a .EML file. Bug 328749 replaced that code with a rewrite; the analogous try/catch block was maintained in the new code, but the null-folder check was not. This is now causing an error every time you open a .EML file. That's what this bug is about. Bug 358247 ported the 358121 code to Seamonkey and put the test within the try block, allowing the null check to be discarded. Seamonkey is still running with the older code. (Is it really necessary that this file is forked?)
xref bug 358121 comment 5
OS: Windows 2000 → All
Hardware: PC → All
Version: Trunk → 2.0
Created attachment 256488 [details] [diff] [review] diff -uw vs. 2b2-0221 distro This fixes the problem by moving the test inside the try block, per Neil's comment. This fixes the bug and re-allows the .EML file to be detected as a scam, altho the "Not a Scam" button does nothing when clicked in that case. Maybe it should be disabled.
Created attachment 256490 [details] [diff] [review] diff -u Sorry, I'm an idiot; you don't need a whitespace patch here.
Comment on attachment 256490 [details] [diff] [review] diff -u thanks Mike.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
V with TB 2.0pre-0227, Win2K.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.