]> git.mxchange.org Git - friendica.git/commitdiff
apply coding standards to lockview module
authorrabuzarus <rabuzarus@t-online.de>
Wed, 5 Sep 2018 12:24:51 +0000 (14:24 +0200)
committerrabuzarus <rabuzarus@t-online.de>
Wed, 5 Sep 2018 12:24:51 +0000 (14:24 +0200)
mod/lockview.php

index 7617d6f7367aec0a5f790ba13cd3d02a91190499..0d87e1551034d24739e867f79388d185f762cd10 100644 (file)
@@ -8,24 +8,27 @@ use Friendica\Core\L10n;
 use Friendica\Database\DBA;
 use Friendica\Model\Item;
 
-function lockview_content(App $a) {
-
+function lockview_content(App $a)
+{
        $type = (($a->argc > 1) ? $a->argv[1] : 0);
        if (is_numeric($type)) {
                $item_id = intval($type);
-               $type='item';
+               $type = 'item';
        } else {
                $item_id = (($a->argc > 2) ? intval($a->argv[2]) : 0);
        }
 
-       if (!$item_id)
+       if (!$item_id) {
                killme();
+       }
 
-       if (!in_array($type, ['item','photo','event']))
+       if (!in_array($type, ['item','photo','event'])) {
                killme();
+       }
 
        $fields = ['uid', 'allow_cid', 'allow_gid', 'deny_cid', 'deny_gid'];
        $condition = ['id' => $item_id];
+
        if ($type != 'item') {
                $item = DBA::selectFirst($type, $fields, $condition);
        } else {
@@ -44,18 +47,20 @@ function lockview_content(App $a) {
                killme();
        }
 
-
-       if (($item['private'] == 1) && empty($item['allow_cid']) && empty($item['allow_gid'])
-               && empty($item['deny_cid']) && empty($item['deny_gid'])) {
-
+       if ($item['private'] == 1
+               && empty($item['allow_cid'])
+               && empty($item['allow_gid'])
+               && empty($item['deny_cid'])
+               && empty($item['deny_gid']))
+       {
                echo L10n::t('Remote privacy information not available.') . '<br />';
                killme();
        }
 
-       $allowed_users = expand_acl($item['allow_cid']);
+       $allowed_users  = expand_acl($item['allow_cid']);
        $allowed_groups = expand_acl($item['allow_gid']);
-       $deny_users = expand_acl($item['deny_cid']);
-       $deny_groups = expand_acl($item['deny_gid']);
+       $deny_users     = expand_acl($item['deny_cid']);
+       $deny_groups    = expand_acl($item['deny_gid']);
 
        $o = L10n::t('Visible to:') . '<br />';
        $l = [];
@@ -64,36 +69,44 @@ function lockview_content(App $a) {
                $r = q("SELECT `name` FROM `group` WHERE `id` IN ( %s )",
                        DBA::escape(implode(', ', $allowed_groups))
                );
-               if (DBA::isResult($r))
-                       foreach($r as $rr)
+               if (DBA::isResult($r)) {
+                       foreach ($r as $rr) {
                                $l[] = '<b>' . $rr['name'] . '</b>';
+                       }
+               }
        }
+
        if (count($allowed_users)) {
                $r = q("SELECT `name` FROM `contact` WHERE `id` IN ( %s )",
-                       DBA::escape(implode(', ',$allowed_users))
+                       DBA::escape(implode(', ', $allowed_users))
                );
-               if (DBA::isResult($r))
-                       foreach($r as $rr)
+               if (DBA::isResult($r)) {
+                       foreach ($r as $rr) {
                                $l[] = $rr['name'];
-
+                       }
+               }
        }
 
        if (count($deny_groups)) {
                $r = q("SELECT `name` FROM `group` WHERE `id` IN ( %s )",
                        DBA::escape(implode(', ', $deny_groups))
                );
-               if (DBA::isResult($r))
-                       foreach($r as $rr)
+               if (DBA::isResult($r)) {
+                       foreach ($r as $rr) {
                                $l[] = '<b><strike>' . $rr['name'] . '</strike></b>';
+                       }
+               }
        }
+
        if (count($deny_users)) {
                $r = q("SELECT `name` FROM `contact` WHERE `id` IN ( %s )",
-                       DBA::escape(implode(', ',$deny_users))
+                       DBA::escape(implode(', ', $deny_users))
                );
-               if (DBA::isResult($r))
-                       foreach($r as $rr)
+               if (DBA::isResult($r)) {
+                       foreach ($r as $rr) {
                                $l[] = '<strike>' . $rr['name'] . '</strike>';
-
+                       }
+               }
        }
 
        echo $o . implode(', ', $l);