Label could conceivably be 0, so can't use empty()

horde · May 19, 2014 · 58dde95 · 58dde95
1 parent 7232063
commit 58dde95
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/imp/lib/Ajax/Application/Handler/RemotePrefs.php b/imp/lib/Ajax/Application/Handler/RemotePrefs.php
@@ -61,7 +61,7 @@ public function autoconfigAccount()
 
                 $res->mconfig = (object)$mconfig;
                 $res->mconfig->imap = $imap;
-                if (empty($res->mconfig->label)) {
+                if (!strlen($res->mconfig->label)) {
                     $res->mconfig->label = $email->bare_address;
                 }
                 $res->success = true;