GEANT / CAT

web/admin/overview_org.php

Switch Indentation +53 added lines, -53 removed lines

@@ -207,70 +207,70 @@ 
                     </div>
                     <?php
                     break;
-                case "core\ProfileRADIUS":
-                    ?>
-                    <div style='padding-bottom:20px;'>
+                    case "core\ProfileRADIUS":
+                        ?>
+                        <div style='padding-bottom:20px;'>
                         <?php $profile_name = $profile_list->name; ?>
-                        <h2 style='overflow:auto; display:inline; padding-bottom: 10px;'><?php printf(_("Profile: %s"), $profile_name); ?></h2>
+                            <h2 style='overflow:auto; display:inline; padding-bottom: 10px;'><?php printf(_("Profile: %s"), $profile_name); ?></h2>
                         <?php
-                        // see if there are any profile-level overrides
-                        $attribs = $profile_list->getAttributes();
-                        // write things into a buffer; we need some function calls to determine
-                        // readiness - but want to display it before!
-                        $has_overrides = FALSE;
-                        foreach ($attribs as $attrib) {
-                            if ($attrib['level'] == \core\Options::LEVEL_PROFILE && !preg_match("/^(internal:|profile:name|profile:description|eap:)/", $attrib['name'])) {
-                                $has_overrides = TRUE;
+                            // see if there are any profile-level overrides
+                            $attribs = $profile_list->getAttributes();
+                            // write things into a buffer; we need some function calls to determine
+                            // readiness - but want to display it before!
+                            $has_overrides = FALSE;
+                            foreach ($attribs as $attrib) {
+                                if ($attrib['level'] == \core\Options::LEVEL_PROFILE && !preg_match("/^(internal:|profile:name|profile:description|eap:)/", $attrib['name'])) {
+                                    $has_overrides = TRUE;
+                                }
                             }
-                        }
-                        $buffer_eaptypediv = "<div style='margin-bottom:40px; float:left;'>" . _("<strong>EAP Types</strong> (in order of preference):") . "<br/>";
-                        $typelist = $profile_list->getEapMethodsinOrderOfPreference();
-                        $allcomplete = TRUE;
-                        foreach ($typelist as $eaptype) {
-                            $buffer_eaptypediv .= $eaptype->getPrintableRep();
-                            $completeness = $profile_list->isEapTypeDefinitionComplete($eaptype);
-                            if ($completeness === true) {
-                                $buffer_eaptypediv .= " <div class='acceptable'>" . _("OK") . "</div>";
-                            } else {
-                                $buffer_eaptypediv .= " <div class='notacceptable'>";
-                                $buffer_eaptypediv .= _("Information needed!");
-                                if (is_array($completeness)) {
-                                    $buffer_eaptypediv .= "<ul style='margin:1px'>";
-                                    foreach ($completeness as $missing_attrib) {
-                                        $buffer_eaptypediv .= "<li>" . $uiElements->displayName($missing_attrib) . "</li>";
+                            $buffer_eaptypediv = "<div style='margin-bottom:40px; float:left;'>" . _("<strong>EAP Types</strong> (in order of preference):") . "<br/>";
+                            $typelist = $profile_list->getEapMethodsinOrderOfPreference();
+                            $allcomplete = TRUE;
+                            foreach ($typelist as $eaptype) {
+                                $buffer_eaptypediv .= $eaptype->getPrintableRep();
+                                $completeness = $profile_list->isEapTypeDefinitionComplete($eaptype);
+                                if ($completeness === true) {
+                                    $buffer_eaptypediv .= " <div class='acceptable'>" . _("OK") . "</div>";
+                                } else {
+                                    $buffer_eaptypediv .= " <div class='notacceptable'>";
+                                    $buffer_eaptypediv .= _("Information needed!");
+                                    if (is_array($completeness)) {
+                                        $buffer_eaptypediv .= "<ul style='margin:1px'>";
+                                        foreach ($completeness as $missing_attrib) {
+                                            $buffer_eaptypediv .= "<li>" . $uiElements->displayName($missing_attrib) . "</li>";
+                                        }
+                                        $buffer_eaptypediv .= "</ul>";
                                     }
-                                    $buffer_eaptypediv .= "</ul>";
+                                    $buffer_eaptypediv .= "</div>";
+                                    $allcomplete = FALSE;
                                 }
-                                $buffer_eaptypediv .= "</div>";
-                                $allcomplete = FALSE;
-                            }
-                            $attribs = $profile_list->getAttributes();
-                            $justOnce = FALSE;
-                            foreach ($attribs as $attrib) {
-                                if ($attrib['level'] == \core\Options::LEVEL_METHOD && !preg_match("/^internal:/", $attrib['name']) && !$justOnce) {
-                                    $justOnce = TRUE;
-                                    $buffer_eaptypediv .= "<img src='../resources/images/icons/Letter-E-blue-icon.png' alt='" . _("Options on EAP Method/Device level are in effect.") . "'>";
+                                $attribs = $profile_list->getAttributes();
+                                $justOnce = FALSE;
+                                foreach ($attribs as $attrib) {
+                                    if ($attrib['level'] == \core\Options::LEVEL_METHOD && !preg_match("/^internal:/", $attrib['name']) && !$justOnce) {
+                                        $justOnce = TRUE;
+                                        $buffer_eaptypediv .= "<img src='../resources/images/icons/Letter-E-blue-icon.png' alt='" . _("Options on EAP Method/Device level are in effect.") . "'>";
+                                    }
                                 }
+                                $buffer_eaptypediv .= "<br/>";
                             }
-                            $buffer_eaptypediv .= "<br/>";
-                        }
-                        $buffer_eaptypediv .= "</div>";
+                            $buffer_eaptypediv .= "</div>";
 
-                        $buffer_headline = "<span style='float:right;'>";
-                        $readiness = $profile_list->readinessLevel();
-                        if ($has_overrides) {
-                            $buffer_headline .= $uiElements->boxRemark("", _("Option override on profile level is in effect."), TRUE);
-                        }
-                        if (!$allcomplete) {
-                            $buffer_headline .= $uiElements->boxError("", _("The information in this profile is incomplete."), TRUE);
-                        }
-                        switch ($readiness) {
+                            $buffer_headline = "<span style='float:right;'>";
+                            $readiness = $profile_list->readinessLevel();
+                            if ($has_overrides) {
+                                $buffer_headline .= $uiElements->boxRemark("", _("Option override on profile level is in effect."), TRUE);
+                            }
+                            if (!$allcomplete) {
+                                $buffer_headline .= $uiElements->boxError("", _("The information in this profile is incomplete."), TRUE);
+                            }
+                            switch ($readiness) {
                             case core\AbstractProfile::READINESS_LEVEL_SHOWTIME:
                                 $buffer_headline .= $uiElements->boxOkay("", _("This profile is shown on the user download interface."), TRUE);
                                 break;
                             case core\AbstractProfile::READINESS_LEVEL_SUFFICIENTCONFIG:
                                 $buffer_headline .= $uiElements->boxWarning("", sprintf(_("This profile is NOT shown on the user download interface, even though we have enough information to show. To enable the profile, add the attribute \"%s\" and tick the corresponding box."), $uiElements->displayName("profile:production")), TRUE);
-                        }
+                            }
 
                         $buffer_headline .= "</span></div>";
 
@@ -353,8 +353,8 @@ 
                     </div>
                     <?php
                     break;
-                default:
-                    throw new Exception("We were asked to operate on something that is neither a RADIUS nor Silverbullet profile!");
+                    default:
+                        throw new Exception("We were asked to operate on something that is neither a RADIUS nor Silverbullet profile!");
             }
             ?>
             <!-- dummy width to keep a little distance -->

Spacing +36 added lines, -36 removed lines

@@ -26,7 +26,7 @@ 
  */
 ?>
 <?php
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
 
 $deco = new \web\lib\admin\PageDecoration();
 $validator = new \web\lib\common\InputValidation();
@@ -38,7 +38,7 @@ 
 } else {
     $link = 'http://';
 }
-$link .= $_SERVER['SERVER_NAME'] . $_SERVER['SCRIPT_NAME'];
+$link .= $_SERVER['SERVER_NAME'].$_SERVER['SCRIPT_NAME'];
 $link = htmlspecialchars($link);
 
 echo $deco->defaultPagePrelude(sprintf(_("%s: %s Dashboard"), \config\Master::APPEARANCE['productname'], $uiElements->nomenclatureParticipant));
@@ -182,14 +182,14 @@ 
                                     <ul style='margin:1px'>
                                         <?php
                                         foreach ($completeness as $missing_attrib) {
-                                            echo "<li>" . $uiElements->displayName($missing_attrib) . "</li>";
+                                            echo "<li>".$uiElements->displayName($missing_attrib)."</li>";
                                         }
                                         ?>
                                     </ul>
                                 </div>
                                 <?php
                             } else {
-                                echo sprintf(_("You can create up to %d users."), $maxusers[0]['value']) . "<br/>" . sprintf(_("Their credentials will carry the name <strong>%s</strong>."), $profile_list->realm);
+                                echo sprintf(_("You can create up to %d users."), $maxusers[0]['value'])."<br/>".sprintf(_("Their credentials will carry the name <strong>%s</strong>."), $profile_list->realm);
                             }
                             ?>
                             <br/>
@@ -198,7 +198,7 @@ 
                             if ($readonly === FALSE) {
                                 ?>
                                 <form action='edit_silverbullet.php?inst_id=<?php echo $my_inst->identifier; ?>&amp;profile_id=<?php echo $profile_list->identifier; ?>' method='POST'>
-                                    <button <?php echo ( is_array($completeness) ? "disabled" : "" ); ?> type='submit' name='sb_action' value='sb_edit'><?php echo _("Manage User Base"); ?></button>
+                                    <button <?php echo (is_array($completeness) ? "disabled" : ""); ?> type='submit' name='sb_action' value='sb_edit'><?php echo _("Manage User Base"); ?></button>
                                 </form>
                                 <?php
                             }
@@ -223,21 +223,21 @@ 
                                 $has_overrides = TRUE;
                             }
                         }
-                        $buffer_eaptypediv = "<div style='margin-bottom:40px; float:left;'>" . _("<strong>EAP Types</strong> (in order of preference):") . "<br/>";
+                        $buffer_eaptypediv = "<div style='margin-bottom:40px; float:left;'>"._("<strong>EAP Types</strong> (in order of preference):")."<br/>";
                         $typelist = $profile_list->getEapMethodsinOrderOfPreference();
                         $allcomplete = TRUE;
                         foreach ($typelist as $eaptype) {
                             $buffer_eaptypediv .= $eaptype->getPrintableRep();
                             $completeness = $profile_list->isEapTypeDefinitionComplete($eaptype);
                             if ($completeness === true) {
-                                $buffer_eaptypediv .= " <div class='acceptable'>" . _("OK") . "</div>";
+                                $buffer_eaptypediv .= " <div class='acceptable'>"._("OK")."</div>";
                             } else {
                                 $buffer_eaptypediv .= " <div class='notacceptable'>";
                                 $buffer_eaptypediv .= _("Information needed!");
                                 if (is_array($completeness)) {
                                     $buffer_eaptypediv .= "<ul style='margin:1px'>";
                                     foreach ($completeness as $missing_attrib) {
-                                        $buffer_eaptypediv .= "<li>" . $uiElements->displayName($missing_attrib) . "</li>";
+                                        $buffer_eaptypediv .= "<li>".$uiElements->displayName($missing_attrib)."</li>";
                                     }
                                     $buffer_eaptypediv .= "</ul>";
                                 }
@@ -249,7 +249,7 @@ 
                             foreach ($attribs as $attrib) {
                                 if ($attrib['level'] == \core\Options::LEVEL_METHOD && !preg_match("/^internal:/", $attrib['name']) && !$justOnce) {
                                     $justOnce = TRUE;
-                                    $buffer_eaptypediv .= "<img src='../resources/images/icons/Letter-E-blue-icon.png' alt='" . _("Options on EAP Method/Device level are in effect.") . "'>";
+                                    $buffer_eaptypediv .= "<img src='../resources/images/icons/Letter-E-blue-icon.png' alt='"._("Options on EAP Method/Device level are in effect.")."'>";
                                 }
                             }
                             $buffer_eaptypediv .= "<br/>";
@@ -287,10 +287,10 @@ 
                                 if (\config\Master::FUNCTIONALITY_LOCATIONS['DIAGNOSTICS'] == "LOCAL") {
                                     $diagUrl = "../diag/";
                                 } else {
-                                    $diagUrl = \config\Master::FUNCTIONALITY_LOCATIONS['DIAGNOSTICS'] . "/diag/";
+                                    $diagUrl = \config\Master::FUNCTIONALITY_LOCATIONS['DIAGNOSTICS']."/diag/";
                                 }
                                 ?>
-                                <form action='<?php echo $diagUrl . "action_realmcheck.php?inst_id=" . $my_inst->identifier . "&profile_id=" . $profile_list->identifier ?>' method='post' accept-charset='UTF-8'>
+                                <form action='<?php echo $diagUrl."action_realmcheck.php?inst_id=".$my_inst->identifier."&profile_id=".$profile_list->identifier ?>' method='post' accept-charset='UTF-8'>
                                     <input type='hidden' name='comefrom' id='comefrom-<?php echo $profilecount; ?>' value='<?php echo $link; ?>'/>
                                     <button type='submit' name='profile_action' value='check' <?php echo ($has_realm ? "" : "disabled='disabled'"); ?> title='<?php echo _("The realm can only be checked if you configure the realm!"); ?>'>
                                         <?php echo _("Check realm reachability"); ?>
@@ -324,9 +324,9 @@ 
                                 ?>
                                 <div style='display: flex;'>
                                     <?php
-                                    $idpLevelUrl = ( isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == "on" ? 'https://' : 'http://') . $_SERVER['SERVER_NAME'] . dirname(dirname($_SERVER['SCRIPT_NAME'])) . "?idp=" . $my_inst->identifier;
-                                    $displayurl = $idpLevelUrl . "&amp;profile=" . $profile_list->identifier;
-                                    $QRurl = $idpLevelUrl . "&profile=" . $profile_list->identifier;
+                                    $idpLevelUrl = (isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == "on" ? 'https://' : 'http://').$_SERVER['SERVER_NAME'].dirname(dirname($_SERVER['SCRIPT_NAME']))."?idp=".$my_inst->identifier;
+                                    $displayurl = $idpLevelUrl."&amp;profile=".$profile_list->identifier;
+                                    $QRurl = $idpLevelUrl."&profile=".$profile_list->identifier;
                                     $qrCode = new \chillerlan\QRCode\QRCode(new \chillerlan\QRCode\QROptions([
                                                 'outputType' => \chillerlan\QRCode\QRCode::OUTPUT_IMAGE_PNG,
                                                 'eccLevel' => \chillerlan\QRCode\QRCode::ECC_H,
@@ -338,9 +338,9 @@ 
                                     if (empty($rawQr)) {
                                         throw new Exception("Something went seriously wrong during QR code generation!");
                                     }
-                                    $uri = "data:image/png;base64," . base64_encode($uiElements->pngInjectConsortiumLogo($rawQr, web\lib\admin\UIElements::QRCODE_PIXELS_PER_SYMBOL));
+                                    $uri = "data:image/png;base64,".base64_encode($uiElements->pngInjectConsortiumLogo($rawQr, web\lib\admin\UIElements::QRCODE_PIXELS_PER_SYMBOL));
                                     $size = getimagesize($uri);
-                                    echo "<img width='" . ($size[0] / 4) . "' height='" . ($size[1] / 4) . "' src='$uri' alt='QR-code'/>";
+                                    echo "<img width='".($size[0] / 4)."' height='".($size[1] / 4)."' src='$uri' alt='QR-code'/>";
 
                                     //echo "<nobr>$displayurl</nobr></a>";
                                     echo "<p>$displayurl</p></a>";
@@ -390,7 +390,7 @@ 
                     <div>
                         <input type="hidden" name="consortium" value="eduroam"/>
                         <button type='submit' <?php echo ($hasMail > 0 ? "" : "disabled"); ?> name='profile_action' value='new'>
-                            <?php echo sprintf(_("Add %s deployment ..."), \config\ConfAssistant::CONSORTIUM['name'] . " " . \core\DeploymentManaged::PRODUCTNAME); ?>
+                            <?php echo sprintf(_("Add %s deployment ..."), \config\ConfAssistant::CONSORTIUM['name']." ".\core\DeploymentManaged::PRODUCTNAME); ?>
                         </button>
 
                     </div>
@@ -416,7 +416,7 @@ 
     <?php
     $hotspotProfiles = $my_inst->listDeployments();
     if (count($hotspotProfiles) == 0) { // no profiles yet.
-        echo "<h2>" . sprintf(_("There are not yet any known deployments for your %s."), $uiElements->nomenclatureHotspot) . "</h2>";
+        echo "<h2>".sprintf(_("There are not yet any known deployments for your %s."), $uiElements->nomenclatureHotspot)."</h2>";
     }
 
     foreach ($hotspotProfiles as $counter => $deploymentObject) {
@@ -437,7 +437,7 @@ 
                 <h2><?php
                     switch ($deploymentObject->consortium) {
                         case "eduroam":
-                            $displayname = config\ConfAssistant::CONSORTIUM['name'] . " " . core\DeploymentManaged::PRODUCTNAME;
+                            $displayname = config\ConfAssistant::CONSORTIUM['name']." ".core\DeploymentManaged::PRODUCTNAME;
                             break;
                         case "OpenRoaming":
                             $displayname = "OpenRoaming ANP";
@@ -445,7 +445,7 @@ 
                         default:
                             throw new Exception("We are supposed to operate on a roaming consortium we don't know.");
                     }
-                    echo $displayname . " (<span style='color:" . ( $deploymentObject->status == \core\AbstractDeployment::INACTIVE ? "red;'>" . _("inactive") : "green;'>" . _("active") ) . "</span>)";
+                    echo $displayname." (<span style='color:".($deploymentObject->status == \core\AbstractDeployment::INACTIVE ? "red;'>"._("inactive") : "green;'>"._("active"))."</span>)";
                     ?></h2>
                 <table>
                     <caption><?php echo _("Deployment Details"); ?></caption>
@@ -459,13 +459,13 @@ 
                         <td><strong><?php echo _("Your primary RADIUS server") ?></strong><br/>
                             <?php
                             if ($deploymentObject->host1_v4 !== NULL) {
-                                echo _("IPv4") . ": " . $deploymentObject->host1_v4;
+                                echo _("IPv4").": ".$deploymentObject->host1_v4;
                             }
                             if ($deploymentObject->host1_v4 !== NULL && $deploymentObject->host1_v6 !== NULL) {
                                 echo "<br/>";
                             }
                             if ($deploymentObject->host1_v6 !== NULL) {
-                                echo _("IPv6") . ": " . $deploymentObject->host1_v6;
+                                echo _("IPv6").": ".$deploymentObject->host1_v6;
                             }
                             ?>
                         </td>
@@ -473,9 +473,9 @@ 
                         <td><?php echo $deploymentObject->port1; ?></td>
                         <td>
                             <?php
-                            echo "<img src='" . $radiusMessages[$deploymentObject->radius_status_1]['icon'] .
-                            "' alt='" . $radiusMessages[$deploymentObject->radius_status_1]['text'] .
-                            "' title='" . $radiusMessages[$deploymentObject->radius_status_1]['text'] . "'>";
+                            echo "<img src='".$radiusMessages[$deploymentObject->radius_status_1]['icon'].
+                            "' alt='".$radiusMessages[$deploymentObject->radius_status_1]['text'].
+                            "' title='".$radiusMessages[$deploymentObject->radius_status_1]['text']."'>";
                             ?>
                         </td>
                     </tr>
@@ -483,22 +483,22 @@ 
                         <td><strong><?php echo _("Your backup RADIUS server") ?><br/></strong>
                             <?php
                             if ($deploymentObject->host2_v4 !== NULL) {
-                                echo _("IPv4") . ": " . $deploymentObject->host2_v4;
+                                echo _("IPv4").": ".$deploymentObject->host2_v4;
                             }
                             if ($deploymentObject->host2_v4 !== NULL && $deploymentObject->host2_v6 !== NULL) {
                                 echo "<br/>";
                             }
                             if ($deploymentObject->host2_v6 !== NULL) {
-                                echo _("IPv6") . ": " . $deploymentObject->host2_v6;
+                                echo _("IPv6").": ".$deploymentObject->host2_v6;
                             }
                             ?></td>
                         <td><?php echo _("RADIUS port number: ") ?></td>
                         <td><?php echo $deploymentObject->port2; ?></td>
                         <td>
                             <?php
-                            echo "<img src='" . $radiusMessages[$deploymentObject->radius_status_2]['icon'] .
-                            "' alt='" . $radiusMessages[$deploymentObject->radius_status_2]['text'] .
-                            "' title='" . $radiusMessages[$deploymentObject->radius_status_2]['text'] . "'>";
+                            echo "<img src='".$radiusMessages[$deploymentObject->radius_status_2]['icon'].
+                            "' alt='".$radiusMessages[$deploymentObject->radius_status_2]['text'].
+                            "' title='".$radiusMessages[$deploymentObject->radius_status_2]['text']."'>";
                             ?>
                         </td>
                     </tr>
@@ -550,12 +550,12 @@ 
                                 if (array_key_exists('FAILURE', $res) && $res['FAILURE'] > 0) {
                                     echo '<br>';
                                     if ($res['FAILURE'] == 2) {
-                                        echo ' <span style="color: red;">' . _("Activation failure.") . '</span>';
+                                        echo ' <span style="color: red;">'._("Activation failure.").'</span>';
                                     } else {
                                         if (isset($_GET['res'][1]) && $_GET['res']['1'] == 'FAILURE') {
-                                            echo ' <span style="color: red;">' . _("Activation failure for your primary RADIUS server.") . '</span>';
+                                            echo ' <span style="color: red;">'._("Activation failure for your primary RADIUS server.").'</span>';
                                         } else {
-                                            echo ' <span style="color: red;">' . _("Activation failure for your backup RADIUS server.") . '</span>';
+                                            echo ' <span style="color: red;">'._("Activation failure for your backup RADIUS server.").'</span>';
                                         }
                                     }
                                 }
@@ -575,12 +575,12 @@ 
                                 if ($res['FAILURE'] > 0) {
                                     echo '<br>';
                                     if ($res['FAILURE'] == 2) {
-                                        echo ' <span style="color: red;">' . _("Failure during deactivation, your request is queued for handling") . '</span>';
+                                        echo ' <span style="color: red;">'._("Failure during deactivation, your request is queued for handling").'</span>';
                                     } else {
                                         if (isset($_GET['res'][1]) && $_GET['res']['1'] == 'FAILURE') {
-                                            echo ' <span style="color: red;">' . _("Deactivation failure for your primary RADIUS server, your request is queued.") . '</span>';
+                                            echo ' <span style="color: red;">'._("Deactivation failure for your primary RADIUS server, your request is queued.").'</span>';
                                         } else {
-                                            echo ' <span style="color: red;">' . _("Deactivation failure for your backup RADIUS server, your request is queued.") . '</span>';
+                                            echo ' <span style="color: red;">'._("Deactivation failure for your backup RADIUS server, your request is queued.").'</span>';
                                         }
                                     }
                                 }

web/admin/edit_hotspot.php

Indentation +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             && ( $_POST['consortium'] == "eduroam" 
                     ||   
                  ( $_POST['consortium'] == "OpenRoaming" && count($myfed->getAttributes("fed:openroaming")) > 0 )
-               )
+                )
             ) {
         $my_inst->newDeployment(\core\AbstractDeployment::DEPLOYMENTTYPE_MANAGED, $_POST['consortium']);
         header("Location: overview_org.php?inst_id=" . $my_inst->identifier);

Spacing +10 added lines, -10 removed lines

@@ -26,7 +26,7 @@ 
  */
 ?>
 <?php
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
 
 $deco = new \web\lib\admin\PageDecoration();
 $validator = new \web\lib\common\InputValidation();
@@ -37,13 +37,13 @@ 
 
 if (!isset($_GET['deployment_id'])) {
     if (isset($_POST['consortium']) 
-            && ( $_POST['consortium'] == "eduroam" 
+            && ($_POST['consortium'] == "eduroam" 
                     ||   
-                 ( $_POST['consortium'] == "OpenRoaming" && count($myfed->getAttributes("fed:openroaming")) > 0 )
+                 ($_POST['consortium'] == "OpenRoaming" && count($myfed->getAttributes("fed:openroaming")) > 0)
                )
             ) {
         $my_inst->newDeployment(\core\AbstractDeployment::DEPLOYMENTTYPE_MANAGED, $_POST['consortium']);
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier);
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier);
         exit(0);
     } else {
         throw new Exception("Desired consortium for Managed SP needs to be specified, and allowed!");
@@ -58,7 +58,7 @@ 
         if (in_array('OK', $response)) {
             $deployment->deactivate();
         }
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier . '&' . urldecode(http_build_query($response)));
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier.'&'.urldecode(http_build_query($response)));
         exit(0);
     }
 
@@ -67,7 +67,7 @@ 
         if (in_array('OK', $response)) {
             $deployment->activate();
         }
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier . '&' . urldecode(http_build_query($response)));
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier.'&'.urldecode(http_build_query($response)));
         exit(0);
     }
 
@@ -89,7 +89,7 @@ 
         } else {
             $response = ['NOOP', 'NOOP'];
         }
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier . '&' . urldecode(http_build_query($response)));
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier.'&'.urldecode(http_build_query($response)));
         exit(0);
     }
 }
@@ -117,7 +117,7 @@ 
     echo $uiElements->instLevelInfoBoxes($my_inst);
     $deploymentOptions = $deployment->getAttributes();
     echo "<form enctype='multipart/form-data' action='edit_hotspot.php?inst_id=$my_inst->identifier&amp;deployment_id=$deployment->identifier' method='post' accept-charset='UTF-8'>
-                <input type='hidden' name='MAX_FILE_SIZE' value='" . \config\Master::MAX_UPLOAD_SIZE . "'>";
+                <input type='hidden' name='MAX_FILE_SIZE' value='".\config\Master::MAX_UPLOAD_SIZE."'>";
     $optionDisplay = new \web\lib\admin\OptionDisplay($deploymentOptions, \core\Options::LEVEL_PROFILE);
     ?>
     <fieldset class='option_container' id='managedsp_override'>
@@ -149,7 +149,7 @@ 
                 <!-- input for VLAN identifier for home users-->
                 <td>
                     <span id='vlan_label'>
-<?php echo sprintf(_("VLAN tag for own users%s:"), ($vlan === NULL ? "" : " " . _("(unset with '0')"))); ?>
+<?php echo sprintf(_("VLAN tag for own users%s:"), ($vlan === NULL ? "" : " "._("(unset with '0')"))); ?>
                     </span>
                 </td>
                 <td>
@@ -169,7 +169,7 @@ 
     </fieldset>
 
     <?php
-    echo "<p><button type='submit' name='submitbutton' value='" . web\lib\common\FormElements::BUTTON_SAVE . "'>" . _("Save data") . "</button><button type='button' class='delete' name='abortbutton' value='abort' onclick='javascript:window.location = \"overview_org.php?inst_id=$my_inst->identifier\"'>" . _("Discard changes") . "</button></p></form>";
+    echo "<p><button type='submit' name='submitbutton' value='".web\lib\common\FormElements::BUTTON_SAVE."'>"._("Save data")."</button><button type='button' class='delete' name='abortbutton' value='abort' onclick='javascript:window.location = \"overview_org.php?inst_id=$my_inst->identifier\"'>"._("Discard changes")."</button></p></form>";
     echo $deco->footer();
 
     
\ No newline at end of file