[Pkg-owncloud-commits] [owncloud] 19/59: Use is_null() instead of empty() when checking the return value of GroupManager::get().

David Prévot taffit at moszumanska.debian.org
Fri Jul 18 16:19:26 UTC 2014


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository owncloud.

commit a4a0ebf9db2ed2cd845d92bb27d99ba412fb7d39
Author: Stephan Peijnik <speijnik at anexia-it.com>
Date:   Tue Jul 15 10:10:43 2014 +0200

    Use is_null() instead of empty() when checking the return value of GroupManager::get().
    
    Additionally, $grp was renamed to $group inside
     \OC\Group\MetaData::fetchGroups().
    
    Signed-off-by: Stephan Peijnik <speijnik at anexia-it.com>
---
 lib/private/group/metadata.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/private/group/metadata.php b/lib/private/group/metadata.php
index b22fde3..2d8ed37 100644
--- a/lib/private/group/metadata.php
+++ b/lib/private/group/metadata.php
@@ -176,9 +176,9 @@ class MetaData {
 			*/
 			$this->groups = array();
 			foreach($groupIds as $gid) {
-				$grp = $this->groupManager->get($gid);
-				if (!empty($grp)) {
-					$this->groups[$gid] = $grp;
+				$group = $this->groupManager->get($gid);
+				if (!is_null($group)) {
+					$this->groups[$gid] = $group;
 				}
 			}
 		}

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud.git



More information about the Pkg-owncloud-commits mailing list