studieren-ohne-grenzen / dashboard

app/sog/Dashboard/GuestControllerProvider.php

Braces +8 added lines, -5 removed lines

@@ -85,7 +85,9 @@ 
             $ownedGroups = $app['ldap']->getOwnedGroups($user->getAttributes()['dn'])->toArray();
 
             $selGroup = $request->request->get('ou');
-            if (!isset($selGroup)) $selGroup = $ownedGroups[0]['ou'][0];
+            if (!isset($selGroup)) {
+                $selGroup = $ownedGroups[0]['ou'][0];
+            }
             $selGroupDN = sprintf('ou=%s,%s', $selGroup, $app['config']['ldap.subtrees']['groups']);
 
             foreach ($ownedGroups as $og) {
@@ -151,9 +153,10 @@ 
     private function retrieveGuestByMail($mail)
     {
         $info = $this->app['ldap']->getMemberByMail($mail, 'mail');
-        if (is_array($info))
-            return $info['dn'];
-        else
-            return false;
+        if (is_array($info)) {
+                    return $info['dn'];
+        } else {
+                    return false;
+        }
     }
 }
\ No newline at end of file

app/sog/Dashboard/LdapAdapter.php

Braces +7 added lines, -5 removed lines

@@ -32,7 +32,8 @@ 
      * @param $subtrees
      * @throws LdapException
      */
-    public function __construct($options, $subtrees) {
+    public function __construct($options, $subtrees)
+    {
         parent::__construct($options);
         $this->subtrees = $subtrees;
     }
@@ -200,10 +201,11 @@ 
             ['dn', 'uid', 'cn', 'displayName', $attribute],
             'dn'
         );
-        if ($results->count() > 0)
-            return $results->getFirst();
-        else
-            return false;
+        if ($results->count() > 0) {
+                    return $results->getFirst();
+        } else {
+                    return false;
+        }
     }
 
     /**

public/index.php

Braces +31 added lines, -11 removed lines

@@ -66,7 +66,8 @@ 
             $groupDN = sprintf('ou=%s,%s', $groupOU, $app['config']['ldap.subtrees']['groups']);
             $groupAttr = $app['ldap']->getEntry($groupDN, ['cn', 'owner']);
 
-            switch ($action) {
+            switch ($action)
+            {
                 case 'quit':
                     try {
                         if (in_array($userDN, $groupAttr['owner'])) {
@@ -113,15 +114,23 @@ 
         $groupList = [];
         foreach ($groups as $g) {
             $roles = [];
-            if (isset($g['owner']) && in_array($userDN, $g['owner'])) $roles[] = 'owner';
-            if (isset($g['member']) && in_array($userDN, $g['member'])) $roles[] = 'member';
-            if (isset($g['pending']) && in_array($userDN, $g['pending'])) $roles[] = 'pending';
+            if (isset($g['owner']) && in_array($userDN, $g['owner'])) {
+                $roles[] = 'owner';
+            }
+            if (isset($g['member']) && in_array($userDN, $g['member'])) {
+                $roles[] = 'member';
+            }
+            if (isset($g['pending']) && in_array($userDN, $g['pending'])) {
+                $roles[] = 'pending';
+            }
 
             $owners = [];
             if (isset($g['owner'])) {
                 for ($j = 0; $j < count($g['owner']); $j++) {
                     $o = $app['ldap']->getEntry($g['owner'][$j], ['cn', 'mail']);
-                    if (isset($o)) $owners[] = $o;
+                    if (isset($o)) {
+                        $owners[] = $o;
+                    }
                 }
             }
 
@@ -156,7 +165,9 @@ 
             return new \Symfony\Component\HttpFoundation\RedirectResponse('/members/Benutzerdaten');
         }
 
-        if (!isset($selGroup)) $selGroup = $ownedGroups[0]['ou'][0];
+        if (!isset($selGroup)) {
+            $selGroup = $ownedGroups[0]['ou'][0];
+        }
         $selGroupDN = sprintf('ou=%s,%s', $selGroup, $app['config']['ldap.subtrees']['groups']);
 
         $action = $request->request->get('manage-action');
@@ -179,7 +190,8 @@ 
 
                 $groupAttr = $app['ldap']->getEntry($selGroupDN, ['owner']);
 
-                switch ($action) {
+                switch ($action)
+                {
                     case 'activate':
                         try {
                             $app['ldap']->activateMember($userID);
@@ -232,11 +244,19 @@ 
         $memberList = [];
         foreach ($allUsers as $u) {
             $roles = [];
-            if (isset($groupAttr['owner']) && in_array($u['dn'], $groupAttr['owner'])) $roles[] = 'owner';
-            if (isset($groupAttr['member']) && in_array($u['dn'], $groupAttr['member'])) $roles[] = 'member';
-            if (isset($groupAttr['pending']) && in_array($u['dn'], $groupAttr['pending'])) $roles[] = 'pending';
+            if (isset($groupAttr['owner']) && in_array($u['dn'], $groupAttr['owner'])) {
+                $roles[] = 'owner';
+            }
+            if (isset($groupAttr['member']) && in_array($u['dn'], $groupAttr['member'])) {
+                $roles[] = 'member';
+            }
+            if (isset($groupAttr['pending']) && in_array($u['dn'], $groupAttr['pending'])) {
+                $roles[] = 'pending';
+            }
             // we handle inactive members like the other cases, the UI is just so similar
-            if (strstr($u['dn'], 'ou=inactive')) $roles[] = 'inactive';
+            if (strstr($u['dn'], 'ou=inactive')) {
+                $roles[] = 'inactive';
+            }
 
             $listentry = array(
                 'name' => $u['cn'][0],