summaryrefslogtreecommitdiffstats
path: root/install
diff options
context:
space:
mode:
authorEndi S. Dewata <edewata@redhat.com>2011-09-09 18:32:37 -0500
committerEndi S. Dewata <edewata@redhat.com>2011-09-13 05:52:05 +0000
commitb996a41177df9e41426155d3445dc20b0657c9d5 (patch)
tree2aafe20e60bc3105c28dbdd5f682d19677406f46 /install
parent1bdb5d04fe14c92c57a4ef151be9c2eda7ca705f (diff)
downloadfreeipa-b996a41177df9e41426155d3445dc20b0657c9d5.tar.gz
freeipa-b996a41177df9e41426155d3445dc20b0657c9d5.tar.xz
freeipa-b996a41177df9e41426155d3445dc20b0657c9d5.zip
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.
Diffstat (limited to 'install')
-rw-r--r--install/ui/details.js14
-rw-r--r--install/ui/dialog.js16
2 files changed, 19 insertions, 11 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));
}
}
};
diff --git a/install/ui/dialog.js b/install/ui/dialog.js
index 2d6d941c6..86aefeffc 100644
--- a/install/ui/dialog.js
+++ b/install/ui/dialog.js
@@ -162,17 +162,21 @@ IPA.dialog = function(spec) {
field.create(field_container);
- if (field.optional){
- field_container.css('display','none');
+ 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));
}
}