simplesamlphp / simplesamlphp-module-metarefresh

lib/MetaLoader.php

Braces +18 added lines, -9 removed lines

@@ -167,8 +167,10 @@ 
             /* Do we have an attribute whitelist? */
             if (isset($source['attributewhitelist']) && !empty($source['attributewhitelist'])) {
                 $idpmetadata = $entity->getMetadata20IdP();
-                if (!isset($idpmetadata)) /* Skip non-IdPs */
+                if (!isset($idpmetadata)) {
+                    /* Skip non-IdPs */
                     continue;
+                }
 
                 /* Do a recursive comparison for each whitelist of the attributewhitelist with the idpmetadata for this IdP. At least one of these whitelists should match */
                 $match=0;
@@ -178,8 +180,10 @@ 
                         break;
                     }
                 }
-                if ($match==0) /* No match found -> next IdP */
+                if ($match==0) {
+                    /* No match found -> next IdP */
                     continue;
+                }
                 Logger::debug('Whitelisted entityID: '. $entity->getEntityID());
             }
 
@@ -237,8 +241,9 @@ 
                     /* key is number, check that the key appears as one
                      * of the destination keys: if not, then src has
                      * more keys than dst */
-                    if (!array_key_exists($srckey, $dst))
-                        return false;
+                    if (!array_key_exists($srckey, $dst)) {
+                                            return false;
+                    }
 
                     /* loop over dest keys, to find value: we don't know
                      * whether they are in the same order */
@@ -249,13 +254,15 @@ 
                             break;
                         }
                     }
-                    if ($submatch == 0)
-                        return false;
+                    if ($submatch == 0) {
+                                            return false;
+                    }
                 } else {
                     /* key is regexp: find matching keys */
                     $matchingdstkeys=preg_grep($srckey, $dstkeys);
-                    if (!is_array($matchingdstkeys))
-                        return false;
+                    if (!is_array($matchingdstkeys)) {
+                                            return false;
+                    }
 
                     $match=0;
                     foreach ($matchingdstkeys as $dstkey) {
@@ -265,8 +272,10 @@ 
                             break;
                         }
                     }
-                    if ($match==0) /* none of the keys has a matching value */
+                    if ($match==0) {
+                        /* none of the keys has a matching value */
                         return false;
+                    }
                 }
             }
             /* each src key/value matches */