r8857: please don't get fancy with embedded boolean statements in js
authorAndrew Tridgell <tridge@samba.org>
Sat, 30 Jul 2005 02:02:52 +0000 (02:02 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 18:30:12 +0000 (13:30 -0500)
code. Especially as this is a new language for most Samba developers,
it is far better to err strongly on the side of readability rather
than trying to save a line of code by using fancy tricks
(This used to be commit 3228644cf898cc9b3386675f40f2f7e52f69e5c0)

source4/setup/provision

index dc542f59f042fb398a1dd807994c33366f54367e..a9bbfa6a265fed23718224af4114c2d3caab107f 100755 (executable)
@@ -106,8 +106,10 @@ for (r in options) {
        subobj[key] = options[r];
 }
 
+var blank = (options["blank"] != undefined);
+
 message("Provisioning for %s in realm %s\n", subobj.DOMAIN, subobj.REALM);
 message("Using administrator password: %s\n", subobj.ADMINPASS);
-provision(subobj, message, options["blank"] != undefined);
+provision(subobj, message, blank);
 message("All OK\n");
 return 0;