Removed useless "function_exists" checks

[friendica.git] / src / Model / Group.php

diff --git a/src/Model/Group.php b/src/Model/Group.php
index 51b709f5363191b708fac4e51b0f88e7202d4806..857b972d62b4c81c4f1d02ff64e54b689f0355a8 100644 (file)
--- a/src/Model/Group.php
+++ b/src/Model/Group.php
@@ -62,21 +62,13 @@ class Group extends BaseObject
         */
        private static function getByContactIdForUserId($uid, $cid)
        {
-               $stmt = dba::p('SELECT `id`
-                       FROM `group`
-                       INNER JOIN `group_member`
-                               ON `group_member`.`gid` = `group`.`id`
-                       WHERE `group`.`uid` = ?
-                       AND `group_member`.`contact-id` = ?',
-                       $uid,
-                       $cid
-               );
+               $condition = ['uid' => $uid, 'contact-id' => $cid];
+               $stmt = dba::select('group_member', ['gid'], $condition);
 
                $return = [];
-               if (DBM::is_result($stmt)) {
-                       while($group = dba::fetch($stmt)) {
-                               $return[] = $group['id'];
-                       }
+
+               while ($group = dba::fetch($stmt)) {
+                       $return[] = $group['gid'];
                }
 
                return $return;
@@ -125,7 +117,7 @@ class Group extends BaseObject
         */
        public static function getIdByName($uid, $name)
        {
-               if ((! $uid) || (! strlen($name))) {
+               if (!$uid || !strlen($name)) {
                        return false;
                }
 
@@ -209,7 +201,7 @@ class Group extends BaseObject
         */
        public static function addMember($gid, $cid)
        {
-               if (!($gid && $cid)) {
+               if (!$gid || !$cid) {
                        return false;
                }
 
@@ -233,11 +225,7 @@ class Group extends BaseObject
         */
        public static function removeMember($gid, $cid)
        {
-               if (!$gid) {
-                       return false;
-               }
-
-               if (!($gid && $cid)) {
+               if (!$gid || !$cid) {
                        return false;
                }
 
@@ -275,7 +263,7 @@ class Group extends BaseObject
         */
        public static function expand($group_ids, $check_dead = false, $use_gcontact = false)
        {
-               if (!(is_array($group_ids) && count($group_ids))) {
+               if (!is_array($group_ids) || !count($group_ids)) {
                        return [];
                }
 
@@ -292,11 +280,9 @@ class Group extends BaseObject
                        $stmt = dba::select('group_member', ['contact-id'], $condition_array);
                }
 
-               $return = array();
-               if (DBM::is_result($stmt)) {
-                       while($group_member = dba::fetch($stmt)) {
-                               $return[] = $group_member['contact-id'];
-                       }
+               $return = [];
+               while($group_member = dba::fetch($stmt)) {
+                       $return[] = $group_member['contact-id'];
                }
 
                if ($check_dead && !$use_gcontact) {
@@ -318,7 +304,7 @@ class Group extends BaseObject
        {
                $o = '';
 
-               $groups = dba::select('group', [], ['deleted' => 0, 'uid' => $uid], ['order' => ['name' => 'ASC']]);
+               $stmt = dba::select('group', [], ['deleted' => 0, 'uid' => $uid], ['order' => ['name']]);
 
                $display_groups = [
                        [
@@ -327,7 +313,7 @@ class Group extends BaseObject
                                'selected' => ''
                        ]
                ];
-               foreach ($groups as $group) {
+               while ($group = dba::fetch($stmt)) {
                        $display_groups[] = [
                                'name' => $group['name'],
                                'id' => $group['id'],
@@ -377,36 +363,34 @@ class Group extends BaseObject
                        ]
                ];
 
-               $groups = dba::select('group', [], ['deleted' => 0, 'uid' => local_user()], ['order' => ['name' => 'ASC']]);
+               $stmt = dba::select('group', [], ['deleted' => 0, 'uid' => local_user()], ['order' => ['name']]);
 
                $member_of = array();
                if ($cid) {
                        $member_of = self::getByContactIdForUserId(local_user(), $cid);
                }
 
-               if (DBM::is_result($groups)) {
-                       foreach ($groups as $group) {
-                               $selected = (($group_id == $group['id']) ? ' group-selected' : '');
-
-                               if ($editmode == 'full') {
-                                       $groupedit = [
-                                               'href' => 'group/' . $group['id'],
-                                               'title' => t('edit'),
-                                       ];
-                               } else {
-                                       $groupedit = null;
-                               }
+               while ($group = dba::fetch($stmt)) {
+                       $selected = (($group_id == $group['id']) ? ' group-selected' : '');
 
-                               $display_groups[] = [
-                                       'id'   => $group['id'],
-                                       'cid'  => $cid,
-                                       'text' => $group['name'],
-                                       'href' => $each . '/' . $group['id'],
-                                       'edit' => $groupedit,
-                                       'selected' => $selected,
-                                       'ismember' => in_array($group['id'], $member_of),
+                       if ($editmode == 'full') {
+                               $groupedit = [
+                                       'href' => 'group/' . $group['id'],
+                                       'title' => t('edit'),
                                ];
+                       } else {
+                               $groupedit = null;
                        }
+
+                       $display_groups[] = [
+                               'id'   => $group['id'],
+                               'cid'  => $cid,
+                               'text' => $group['name'],
+                               'href' => $each . '/' . $group['id'],
+                               'edit' => $groupedit,
+                               'selected' => $selected,
+                               'ismember' => in_array($group['id'], $member_of),
+                       ];
                }
 
                $tpl = get_markup_template('group_side.tpl');