Closed Bug 333277 Opened 19 years ago Closed 18 years ago

null description in helper version info leaves Download Actions blank

Categories

(Thunderbird :: Preferences, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3

People

(Reporter: philor, Assigned: philor)

Details

(Keywords: fixed1.8.1.5)

Attachments

(1 file)

Thunderbird port of bug 308204: Windows helper apps like Azureus that have version info, but have a blank description field in it, cause nsLocalFileWin::getVersionInfoValue to throw, which leaves the whole list blank when it's uncaught. You get the fix for the Change Action dialog from toolkit in bug 308204, but not for the main list. STR: 1. Install Azureus on Windows. 2. Mail yourself an attached .torrent 3. Tell Thunderbird to always open with Azureus 4. Open Tools > Options > Attachements > View & Edit Options, the list will be blank
Attached patch fix (deleted) — Splinter Review
Attachment #217712 - Flags: review?(mscott)
Comment on attachment 217712 [details] [diff] [review] fix thanks Phil.
Attachment #217712 - Flags: review?(mscott) → review+
Whiteboard: [checkin needed]
mozilla/mail/components/preferences/downloadactions.js 1.5
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3
Whiteboard: [checkin needed]
Comment on attachment 217712 [details] [diff] [review] fix This simple fix should probably land on the branch for TB2, Mike Cowperthwaite noticed it there.
Attachment #217712 - Flags: approval-thunderbird2?
Ping -- this would be really great for TB 2 -- see bug 364594, which is Mac-only but creates a problem that the user can't get out of, without manually editing mimeTypes.rdf, unless this bug is fixed.
Weird, this approval request doesn't show up in my bugzilla query for pending tb2 approvals and I'm not cc'ed on the bug so I never saw this. Sorry we didn't get it in tb2. I'm not sure why it is evading my query. Looking at my query, I see I did not select the 'Preferences' component in Thunderbird. That's too bad. Sorry guys. We could try to get this in for a point release.
Comment on attachment 217712 [details] [diff] [review] fix Gah, this poor thing just keeps getting forgotten. Should be safe as houses - the same change to the same code in Firefox has been on 1.8 for a few days over a year now.
Attachment #217712 - Flags: approval-thunderbird2? → approval1.8.1.5?
Comment on attachment 217712 [details] [diff] [review] fix approved for 1.8.1.5, a=dveditz for release-drivers
Attachment #217712 - Flags: approval1.8.1.5? → approval1.8.1.5+
mail/components/preferences/downloadactions.js 1.1.4.3
Keywords: fixed1.8.1.5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: