From c2783b9b54ecae8299da0dc593c3813c2bd26fe2 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 26 Jul 2012 11:17:28 +0200 Subject: 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 --- install/ui/association.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'install/ui/association.js') 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(); -- cgit