Fixed missing optional field.

The optional uid field in user's adder dialog did not appear when
the link is clicked to show the field. This is a regression introduced
in the patch for ticket #1648.

The click handler for the link field has been moved into a new closure
so that the variables point to the correct elements.

Note: the duplicate code in IPA.details_table_section.create() and
IPA.dialog.create() will be addressed separately in ticket #1394.

1 files changed, 9 insertions, 5 deletions

diff --git a/install/ui/details.js b/install/ui/details.js
index 23f217afb..f7831114d 100644
--- a/install/ui/details.js
+++ b/install/ui/details.js
@@ -213,17 +213,21 @@ IPA.details_table_section = function(spec) {
 
             field.create(field_container);
 
-            if (field.optional){
+            if (field.optional) {
                 field_container.css('display', 'none');
+
                 var link = $('<a/>', {
                     text: IPA.messages.widget.optional,
-                    href: '',
-                    click: function() {
+                    href: ''
+                }).appendTo(td);
+
+                link.click(function(field_container, link) {
+                    return function() {
                         field_container.css('display', 'inline');
                         link.css('display', 'none');
                         return false;
-                    }
-                }).appendTo(td);
+                    };
+                }(field_container, link));
             }
         }
     };