diff options
author | Petr Vobornik <pvoborni@redhat.com> | 2011-12-05 13:39:30 +0100 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2011-12-05 16:02:11 +0000 |
commit | 1ffbec9942ad90e00e28b05296d3233f52ce2dad (patch) | |
tree | db2226435f5a925a8073f482e626565c3d2e4375 /install/ui/ipa.js | |
parent | c5ca34f41d2aede667c19d2213a5d7e79a5e301c (diff) | |
download | freeipa-1ffbec9942ad90e00e28b05296d3233f52ce2dad.tar.gz freeipa-1ffbec9942ad90e00e28b05296d3233f52ce2dad.tar.xz freeipa-1ffbec9942ad90e00e28b05296d3233f52ce2dad.zip |
Removed usage of bitwise assignment operators in logical operations
JavaScript &= and |= are bitwise operators. They are shortened version of:
foo = foo & bar
foo = foo | bar
In some places they were used as shortened version of logical operation and assignment.
foo = foo && bar
It lead to type conversion to Number which is wrong (0 !== false).
This patch replaces such occurances with full version of logical operation and asignment.
https://fedorahosted.org/freeipa/ticket/2040
Diffstat (limited to 'install/ui/ipa.js')
-rw-r--r-- | install/ui/ipa.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/install/ui/ipa.js b/install/ui/ipa.js index 08d8722ce..8a7b29f75 100644 --- a/install/ui/ipa.js +++ b/install/ui/ipa.js @@ -725,8 +725,8 @@ IPA.concurrent_command = function(spec) { for(var i=0; i < that.commands.length; i++) { var command_info = that.commands[i]; - all_completed &= command_info.completed; - all_success &= command_info.success; + all_completed = all_completed && command_info.completed; + all_success = all_success && command_info.success; } if(all_completed) { |