diff options
author | Christopher Smith <chris@jalakai.co.uk> | 2014-03-05 21:58:46 +0000 |
---|---|---|
committer | Christopher Smith <chris@jalakai.co.uk> | 2014-03-05 21:58:46 +0000 |
commit | 0e80bb5e347ff00c6f81627d8e39dafaaa923bc5 (patch) | |
tree | 19e8450aeef463ffe7278e3fb5e8888940113b4a /lib | |
parent | 17dd401e94c682034b7f3d9164ac3523ab01d825 (diff) | |
download | rpg-0e80bb5e347ff00c6f81627d8e39dafaaa923bc5.tar.gz rpg-0e80bb5e347ff00c6f81627d8e39dafaaa923bc5.tar.bz2 |
use empty() where array values might not be set
Diffstat (limited to 'lib')
-rw-r--r-- | lib/exe/mediamanager.php | 4 | ||||
-rw-r--r-- | lib/plugins/acl/admin.php | 2 | ||||
-rw-r--r-- | lib/plugins/usermanager/admin.php | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/lib/exe/mediamanager.php b/lib/exe/mediamanager.php index d94a24c74..7044232ce 100644 --- a/lib/exe/mediamanager.php +++ b/lib/exe/mediamanager.php @@ -57,7 +57,7 @@ } // give info on PHP caught upload errors - if($_FILES['upload']['error']){ + if(!empty($_FILES['upload']['error'])){ switch($_FILES['upload']['error']){ case 1: case 2: @@ -71,7 +71,7 @@ } // handle upload - if($_FILES['upload']['tmp_name']){ + if(!empty($_FILES['upload']['tmp_name'])){ $JUMPTO = media_upload($NS,$AUTH); if($JUMPTO) $NS = getNS($JUMPTO); } diff --git a/lib/plugins/acl/admin.php b/lib/plugins/acl/admin.php index 6c7c28ff6..00bf9969f 100644 --- a/lib/plugins/acl/admin.php +++ b/lib/plugins/acl/admin.php @@ -488,7 +488,7 @@ class admin_plugin_acl extends DokuWiki_Admin_Plugin { function _html_list_acl($item){ $ret = ''; // what to display - if($item['label']){ + if(!empty($item['label'])){ $base = $item['label']; }else{ $base = ':'.$item['id']; diff --git a/lib/plugins/usermanager/admin.php b/lib/plugins/usermanager/admin.php index eadfb76ad..b67d91b36 100644 --- a/lib/plugins/usermanager/admin.php +++ b/lib/plugins/usermanager/admin.php @@ -53,7 +53,7 @@ class admin_plugin_usermanager extends DokuWiki_Admin_Plugin { } // attempt to retrieve any import failures from the session - if ($_SESSION['import_failures']){ + if (!empty($_SESSION['import_failures'])){ $this->_import_failures = $_SESSION['import_failures']; } } |