From c2783b9b54ecae8299da0dc593c3813c2bd26fe2 Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvoborni@redhat.com>
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/aci.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'install/ui/aci.js')

diff --git a/install/ui/aci.js b/install/ui/aci.js
index 63181efa..039e6332 100644
--- a/install/ui/aci.js
+++ b/install/ui/aci.js
@@ -492,7 +492,7 @@ IPA.attributes_widget = function(spec) {
                 type: "checkbox",
                 click: function() {
                     $('.aci-attribute', that.table).
-                        attr('checked', $(this).attr('checked'));
+                        prop('checked', $(this).prop('checked'));
                     that.value_changed.notify([], that);
                 }
             })
-- 
cgit