fix checking boolean parameters

darcs-hash:20080529151602-84dde-f6bc3c41846d92c94008f721fa5115907fd9a7d3.gz

diff --git a/actions/register.php b/actions/register.php
index 42b46bcc35a4a0c11083d3bfbe7e023233a311b1..d2e54b634596e8ecf3f5077910fafbba65c32d60 100644 (file)
--- a/actions/register.php
+++ b/actions/register.php
@@ -34,7 +34,6 @@ class RegisterAction extends Action {
        }
 
        function try_register() {
-               print_r($this->args);
                $nickname = $this->trimmed('nickname');
                $email = $this->trimmed('email');
                

diff --git a/lib/action.php b/lib/action.php
index 67eaf9ed7a65fbaad400a16794687a600fb37215..dae50e6e319eb4baa37ebe1bec530c8d57676cdd 100644 (file)
--- a/lib/action.php
+++ b/lib/action.php
@@ -50,12 +50,12 @@ class Action { // lawsuit
        function boolean($key, $def=false) {
                $arg = $this->arg($key);
                return (is_null($arg)) ? $def : 
-               (strcasecmp($arg, 'true')) ? true :
-               (strcasecmp($arg, 'yes')) ? true : 
-               (strcasecmp($arg, '1')) ? true :
-               (strcasecmp($arg, 'false')) ? false :
-               (strcasecmp($arg, 'no')) ? false : 
-               (strcasecmp($arg, '0')) ? false :
+               (strcasecmp($arg, 'true') == 0) ? true :
+               (strcasecmp($arg, 'yes') == 0) ? true : 
+               (strcasecmp($arg, '1') == 0) ? true :
+               (strcasecmp($arg, 'false') == 0) ? false :
+               (strcasecmp($arg, 'no') == 0) ? false : 
+               (strcasecmp($arg, '0') == 0) ? false :
                $def;
        }
 }