GEANT / CAT

core/diag/Logopath.php

Braces +6 added lines, -4 removed lines

@@ -173,10 +173,11 @@ 
         ];
         
         // add exalted human-readable information to main mail body
-        foreach ($this->mailStack as $oneEntry)
-        if (isset($this->additionalFindings['INTERACTIVE_ENDUSER_AUTH_TIMESTAMP'])) {
+        foreach ($this->mailStack as $oneEntry) {
+                if (isset($this->additionalFindings['INTERACTIVE_ENDUSER_AUTH_TIMESTAMP'])) {
             $oneEntry .= _("Authentication/Attempt Timestamp of user session:") ." ".$this->additionalFindings['INTERACTIVE_ENDUSER_AUTH_TIMESTAMP']."\n";
         }
+        }
         if (isset($this->additionalFindings['INTERACTIVE_ENDUSER_MAC'])) {
             $oneEntry .= _("MAC address of end user in question:") ." ".$this->additionalFindings['INTERACTIVE_ENDUSER_MAC']."\n";
         }
@@ -246,12 +247,13 @@ 
         if (in_array(AbstractTest::INFRA_IDP_ADMIN_DETERMINED_FORCED, $this->possibleFailureReasons)) {
             $this->mailQueue[] = Logopath::IDP_SUSPECTED_PROBLEM_INTERACTIVE_FORCED;
         }
-        if(in_array($needle, $haystack))
-// after collecting all the conditions, find the target entities in all
+        if(in_array($needle, $haystack)) {
+            // after collecting all the conditions, find the target entities in all
 // the mails, and check if they resolve to a known mail address. If they
 // do not, this triggers more mails about missing contact info.
 
         $abstractRecipients = [];
+        }
         foreach ($this->mailQueue as $mail) {
             $abstractRecipients = array_unique(array_merge($this->mailStack[$mail]['to'], $this->mailStack[$mail]['cc'], $this->mailStack[$mail]['bcc'], $this->mailStack[$mail]['reply-to']));
         }