68105 - mimetype entry disappears after editing Handled By

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect)

Description

[db]

Start with new empty profile. In the preferences->helperapplications there is
only one entry for text/html. Add a new type and enter the values

Postscript
ps
application/postscript
/usr/X11R6/bin/ghostview

then press OK and you can se the new entry next to text/html. Now edit the new
entry and change the path to something else. Press OK. And now the entry has
disappeared.

Even if I don't change the entry so that it doesn't disappear, when I add
entries for both postscript and PDF I can not get it to work.

It's simply very broken.

This is on build 2001020712, on a Linux Redhat 6.2 system.

Comment 1

[Tobias Burnus]

I saw the same with 2001-02-07-06/SuSE-x86-Linux-7.1b6.

Comment 2

[Daniel Veditz [:dveditz]]

This is prefs front end (and probably a duplicate)

Comment 3

[sairuh (rarely reading bugmail)]

shrir, you see this? if so, should mscott or bill get it?

Comment 4

[sairuh (rarely reading bugmail)]

confirming. and i see this on winNT and Mac -> all.

not sure if this is the result of an existing bug [shrir?]... also, i noticed
that i cannot edit the Application field of an existing helper app [same as this
bug? another one?] --clicking OK in the edit dialog doesn't dismiss it
[reminiscent of an older bug], but after hitting cancel to dismiss the edit
dialog, then bringing it up again reveals that the Application field is suddenly
empty. hrm.

tentatively handing to mscott --punt as needed. :)

Comment 5

[Scott MacGregor]

probably related to the changed paul checked in tuesday night.

re-assigning.

Comment 6

[sairuh (rarely reading bugmail)]

noticed that this also occurs when changing the Handled By radiobutton choice.
resummarizing.

another thing i noticed --pls lemme know if this is the same bug: i create a new
helper app entry *without* specifying a MIME Type, it'll appear as a blank
listing. then i edit it, and give it a MIME Type --after i click OK in the edit
dialog, *another* entry is added to the list, along with the blank one. [this
might be an existing bug, but i cannot recall its number offhand... lemme know
if i should file this. thx!]

Comment 7

[Dave]

If I edit a helper app entry and hit enter, even if I don't make any changes,
the entry dissappears (2001020909/Linux).

Comment 8

[sairuh (rarely reading bugmail)]

*** Bug 68362 has been marked as a duplicate of this bug. ***

Comment 9

[Paul Chen]

Attachment: move deletion of old mapping, also delete mapping we're going to override

Comment 10

[Paul Chen]

Just attached proposed fix. For some reason, the prefs code is adding a helper 
app mapping with a null mime type. That's why the mapping disappears, we delete 
the old version, then the new version is added, which is bogus. Sigh.

Moving the deletion higher up in the code seems to help, but I'm kinda weary of 
this solution. Would like more testing. While I'm here, also fix case where you 
change the mimetype to override another mapping, need to delete the one you're 
about to override before you save the new one.

In other words, you have a pref for "application/ps" and "application/pdf". You 
edit the mimetype of application/ps to application/pdf. After putting up the 
warning dialog, we wouldn't delete the current application/pdf pref before saving 
the pref you're editing.

Comment 11

[Scott MacGregor]

sr=mscott 

Comment 12

[Asa Dotzler [:asa]]

can we get this checked into the branch as well. a=asa

Comment 13

[Brendan Eich [:brendan]]

How about an r= first?

/be

Comment 14

[Alec Flett]

r=alecf
this was on the xpapp's group's "top 10 annoying features" so adding the
"dogfood" keyword

Comment 15

[shrirang khanzode]

*** Bug 68124 has been marked as a duplicate of this bug. ***

Comment 16

[sairuh (rarely reading bugmail)]

*** Bug 68708 has been marked as a duplicate of this bug. ***

Comment 17

[Paul Chen]

Checked into trunk and mozilla0.8 branch. I wouldn't be the least bit surprised 
if there were still issues with editing.

Comment 18

[shrirang khanzode]

Thisi is working fine on the trunk 0219 (win,mac, linux). Also verified on the 
0215 moz 0.8 builds on win/mac/linux. Sairuh, I see the issue that you mentioned 
in 'Comments From sairuh (se) 2001-02-08 13:33 '. Pls file a bug for 
that.Thanks! Marknig this bug verified.

Comment 19

[sairuh (rarely reading bugmail)]

okay, shrir, i filed bug 69708 for the blank entry issue.