Closed Bug 410937 Opened 17 years ago Closed 16 years ago

Make "choose app to open with" show ALWAYS appname not appname.app if files extensions are disable in finder [Mac]

Categories

(Firefox :: File Handling, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: NicolasWeb, Assigned: florian)

Details

Attachments

(3 files)

User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; fr; rv:1.9b2) Gecko/2007121014 Firefox/3.0b2 Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; fr; rv:1.9b2) Gecko/2007121014 Firefox/3.0b2 for different app Fx show appname.app or appname, there is no one behavior. The current behavior do not respect the finder pref to show or hide files extensions. Maybe this link with Bug #410930 Reproducible: Always Steps to Reproduce: 1.Click to download a file .doc 2.Choose MSoffice to open with 3.Click to download a file .sxw 4.Choose NeoOffice to open with Actual Results: With extension hided in finder : .doc show "Microsoft Word" .sxw show "NeoOffice.app" Expected Results: With extension hided in finder : .sxw should show "NeoOffice"
Version: unspecified → Trunk
Attached image Actual behavior
Status: UNCONFIRMED → NEW
Component: General → File Handling
Ever confirmed: true
QA Contact: general → file.handling
Attached patch patch v1Splinter Review
In nsIMIMEInfo.idl the comment above the "name" property says "Human readable name for the handler". The patch makes this property do what the comment says for nsILocalHandlerApp, and then similar code can be removed from various JS files. Later we will want to move this code to nsLocalFile{Mac,Win} in order to remove the 2 remaining JS instances (in FeedWriter.js and preferences.xml) of this code that aren't using it from an nsILocalHandlerApp object.
Assignee: nobody → florian
Status: NEW → ASSIGNED
Attachment #302720 - Flags: review?(dmose)
Attached patch testSplinter Review
Attachment #302726 - Flags: review?(dmose)
This is a duplicate of bug 422511. This patch has the right approach -- almost. Instead of ifdeffing all over the place, you should put the OS-specific code in the OS-specific subclasses. Maybe the patch over in the other bug (which actually fails a test, don't know why) can help you in the right direction.
Seems to be resolved in Fx 3b1...
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Attachment #302720 - Flags: review?(dmose)
Attachment #302726 - Flags: review?(dmose)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: