sairuh (rarely reading bugmail) Reporter
	Description • 24 years ago

found using 2001.06.29.06-trunk comm bits on win98. not sure if this is 
already known. pls reassign as needed...

summary: if i already have a helper app [defined in the pref panel] for 
application/foo, then create another one with same mime --but with different 
description, extension, *or* application-- i'm prompted as to whether i want to 
overwrite the existing one. if i hit Cancel, however, the existing one is 
overwritten anyhow.

recipe:
1. in the helper applications preferences panel, create a helper app with the 
following settings [click "New Type"]:
  description: Foo files
  extension: foo
  mimetype: application/foo
  handled by: winzip.exe [selected path to .exe on my machine]
2. click OK, helper app is created just fine.
2. create another helper app with the following settings [again, click "New 
Type"]:
  description: Zub files
  extension: zub
  mimetype: application/foo
  handled by: [leave blank]
3. click OK, result: get a dialog sayinging "A helper alrady exists for MIME 
type 'application/foo'. Do you want to replace it?"
4. click Cancel.
5. in the pref panel, select the application/foo item in the listbox.

results:
* "Extension" now has two entries: foo zub (interesting way to get multiple 
extensions ;).
* the helper app ["Handled by"] entry is now gone.

	sairuh (rarely reading bugmail) Reporter
	Comment 1 • 24 years ago

well, implied dataloss, since a setting which was cancelled was really 
overwritten.

will check later on other platforms...

	Scott MacGregor Assignee
	Comment 2 • 24 years ago

over to xp-apps / pref's helper app panel owners...

	Paul Chen
	Comment 3 • 24 years ago

nav triage team:

marking nsbeta1+, p3, and mozilla0.9.3

	Viswanath Ramachandran
	Comment 4 • 24 years ago

nav triage -> m1.0, not a frequent enuff user action to be a stopper at this 
time. 

	sairuh (rarely reading bugmail) Reporter
	Comment 5 • 24 years ago

this might not be a frequently-used feature, but the behavior is incorrect! when
you cancel, *nothing* should be done --and in this case, something *is* done and
it changes user's setting. rather deceptive, if you ask me...

adding helpwanted...

	Scott MacGregor Assignee
	Comment 6 • 24 years ago

the following code must be failing for some reason:

  if (exists) {
    var titleMsg = gPrefApplicationsBundle.getString("handlerExistsTitle");
    var dialogMsg = gPrefApplicationsBundle.getString("handlerExists");
    dialogMsg = dialogMsg.replace(/%mime%/g, gMIMEField.value);
    var promptService =
Components.classes["@mozilla.org/embedcomp/prompt-service;1"].getService(Components.interfaces.nsIPromptService);
    var replace = promptService.confirm(window, titleMsg, dialogMsg);
    if (!replace)
      window.close();
  }

this is in pref-application-new.js

I bet we just need a return in front of that window.close. o.t. we'll keep
processing after the if clause and end up replacing the content. I'll post the fix.

	Scott MacGregor Assignee
	Comment 8 • 24 years ago

we should really try to get this one line change into rtm. o.t. we introduce
data loss.

	Scott MacGregor Assignee
	Comment 9 • 24 years ago

sr=blake for this bug.

	Scott MacGregor Assignee
	Comment 10 • 24 years ago

this 0 risk fix has been checked into the trunk. 

	sairuh (rarely reading bugmail) Reporter
	Comment 11 • 24 years ago

->all, since i see this w/2001.07.03.16-trunk bits on linux.

will test once my debug finishes building.

	sairuh (rarely reading bugmail) Reporter
	Comment 12 • 24 years ago

this is fixed on my linux debug.

	scottputterman
	Comment 13 • 24 years ago

When the branch is open, please check this into it today.

	sairuh (rarely reading bugmail) Reporter
	Comment 14 • 24 years ago

vrfy'd fixed using the following comm trunk builds:

linux 2001.07.05.08
winnt 2001.07.05.09
mac 2001.07.05.08

removing vtrunk kw.

	Scott MacGregor Assignee
	Comment 15 • 24 years ago

fixed on the branch.

	sairuh (rarely reading bugmail) Reporter
	Comment 16 • 24 years ago

vrfy fixed on the branch using the following commercial branch bits:

linux - 2001.07.09.04
winnt - 2001.07.09.05
mac - 2001.07.09.03