114325 - Update callers for hasAttribute work

Status: RESOLVED INCOMPLETE

(SeaMonkey :: General, defect)

Description

[jag (Peter Annema)]

There are a bunch of places that still e.g. .setAttribute("checked", "false"),
.setAttribute("checked", someCondition), or .getAttribute("checked") == "false".
This use clashes with the use of .hasAttribute("checked") for the checked
property on checkbox.

Please fix these sites, or revert the logic in checkbox.xml until you have a
chance to fix them.

Comment 1

[jag (Peter Annema)]

This may (in part) be causing bug 113482 and bug 113335.

I also suspect this bug is the reason why, when you go to preferences, when you
have all the checkboxes on the navigator panel unchecked, select another panel,
then select the navigator panel again, the checkboxes will now all be checked.

Comment 2

[Stephen Moehle]

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

Comment 3

[Blake Ross]

Attachment: patch

Comment 4

[Blake Ross]

I only found a few places that should be affected by the changes I've made so 
far, and only one was related to checkbox...

Comment 5

[Joe Hewitt (gone)]

Comment on attachment 61031 [details] [diff] [review]
patch

sr=hewitt

Comment 6

[Boris Zbarsky [:bzbarsky]]

Please make sure that

http://lxr.mozilla.org/seamonkey/search?string=checked%3D%22false%22

are a totally separate issue... (they look like they should be)

Comment 7

[jag (Peter Annema)]

And then there's the other two possibilities I pointed out in my original 
comment.

E.g. |setAttribute("checked", a!=b)| and |var foo = el.getAttribute("checked"); 
if (foo != "false") { ... } else { ... }| (there's some of this somewhere in 
wallet code, twice, iirc).

Comment 8

[Blake Ross]

I went through get/setAttribute calls and didn't find any more that would break 
as a result of my changes.  The one in wallet that I think you're referring to 
is commented out, but it doesn't refer to a checkbox anyways, it's a command.  

Comment 9

[Vedran Miletic]

retargeting

Comment 10

[Boris Zbarsky [:bzbarsky]]

Neil, do you have the time to wrap this up?

Comment 11

[Robert Kaiser]

MASS-CHANGE:
This bug report is registered in the SeaMonkey product, but has been without a comment since the inception of the SeaMonkey project. This means that it was logged against the old Mozilla suite and we cannot determine that it's still valid for the current SeaMonkey suite. Because of this, we are setting it to an UNCONFIRMED state.

If you can confirm that this report still applies to current SeaMonkey 2.x nightly builds, please set it back to the NEW state along with a comment on how you reproduced it on what Build ID, or if it's an enhancement request, why it's still worth implementing and in what way.
If you can confirm that the report doesn't apply to current SeaMonkey 2.x nightly builds, please set it to the appropriate RESOLVED state (WORKSFORME, INVALID, WONTFIX, or similar).
If no action happens within the next few months, we move this bug report to an EXPIRED state.

Query tag for this change: mass-UNCONFIRM-20090614

Comment 12

[Robert Kaiser]

Marking this INCOMPLETE, as it's not sure this work has been completed, but then, if problems have not been fixed, they have been lurking for quite some time and very possibly have been rewritten and replaced since then anyhow.