diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2012-07-26 11:17:28 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2012-08-02 10:27:17 +0200 |
| commit | c2783b9b54ecae8299da0dc593c3813c2bd26fe2 (patch) | |
| tree | 8d43ff1b23545ea58a20bbc911f82f23a455acd2 /install/ui/association.js | |
| parent | 1a94109f4ad5bcb304c4c23af868c3cf07ce92ec (diff) | |
| download | freeipa.git-c2783b9b54ecae8299da0dc593c3813c2bd26fe2.tar.gz freeipa.git-c2783b9b54ecae8299da0dc593c3813c2bd26fe2.tar.xz freeipa.git-c2783b9b54ecae8299da0dc593c3813c2bd26fe2.zip | |
Replace use of attr with prop for booleans
Recommened way of setting boolean HTML attributes is by $.prop(boolean) method not $.attr(boolean) because it sets DOM object property not an attribute. Latter works because of jquery's backward compatibility. This patch makes things clearer.
Some info about prop and attr: http://stackoverflow.com/a/5876747
https://fedorahosted.org/freeipa/ticket/2817
Diffstat (limited to 'install/ui/association.js')
| -rw-r--r-- | install/ui/association.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/install/ui/association.js b/install/ui/association.js index 0594ea76..be8b8459 100644 --- a/install/ui/association.js +++ b/install/ui/association.js @@ -1038,9 +1038,9 @@ IPA.association_facet = function (spec, no_init) { that.refresh = function() { if (that.association_type == 'direct') { - if (that.direct_radio) that.direct_radio.attr('checked', true); + if (that.direct_radio) that.direct_radio.prop('checked', true); } else { - if (that.indirect_radio) that.indirect_radio.attr('checked', true); + if (that.indirect_radio) that.indirect_radio.prop('checked', true); } var pkey = that.entity.get_primary_key(); |