Test Failed
Push — master ( b8acf2...71c020 )
by Maja
17:47
created
web/admin/edit_hotspot.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
  */
27 27
 ?>
28 28
 <?php
29
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
29
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
30 30
 
31 31
 $deco = new \web\lib\admin\PageDecoration();
32 32
 $validator = new \web\lib\common\InputValidation();
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 if (!isset($_GET['deployment_id'])) {
39 39
     if (isset($_POST['consortium']) && count($myfed->getAttributes("fed:openroaming")) > 0) {
40 40
         $my_inst->newDeployment(\core\AbstractDeployment::DEPLOYMENTTYPE_MANAGED, $_POST['consortium']);
41
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier);
41
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier);
42 42
         exit(0);
43 43
     } else {
44 44
         throw new Exception("Desired consortium for Managed SP needs to be specified, and allowed!");
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
         if (in_array('OK', $response)) {
54 54
             $deployment->deactivate();
55 55
         }
56
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier . '&' . urldecode(http_build_query($response)));
56
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier.'&'.urldecode(http_build_query($response)));
57 57
         exit(0);
58 58
     }
59 59
 
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
         if (in_array('OK', $response)) {
63 63
             $deployment->activate();
64 64
         }
65
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier . '&' . urldecode(http_build_query($response)));
65
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier.'&'.urldecode(http_build_query($response)));
66 66
         exit(0);
67 67
     }
68 68
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
         } else {
85 85
             $response = ['NOOP', 'NOOP'];
86 86
         }
87
-        header("Location: overview_org.php?inst_id=" . $my_inst->identifier . '&' . urldecode(http_build_query($response)));
87
+        header("Location: overview_org.php?inst_id=".$my_inst->identifier.'&'.urldecode(http_build_query($response)));
88 88
         exit(0);
89 89
     }
90 90
 }
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
     echo $uiElements->instLevelInfoBoxes($my_inst);
113 113
     $deploymentOptions = $deployment->getAttributes();
114 114
     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'>
115
-                <input type='hidden' name='MAX_FILE_SIZE' value='" . \config\Master::MAX_UPLOAD_SIZE . "'>";
115
+                <input type='hidden' name='MAX_FILE_SIZE' value='".\config\Master::MAX_UPLOAD_SIZE."'>";
116 116
     $optionDisplay = new \web\lib\admin\OptionDisplay($deploymentOptions, \core\Options::LEVEL_PROFILE);
117 117
     ?>
118 118
     <fieldset class='option_container' id='managedsp_override'>
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
                 <!-- input for VLAN identifier for home users-->
145 145
                 <td>
146 146
                     <span id='vlan_label'>
147
-<?php echo sprintf(_("VLAN tag for own users%s:"), ($vlan === NULL ? "" : " " . _("(unset with '0')"))); ?>
147
+<?php echo sprintf(_("VLAN tag for own users%s:"), ($vlan === NULL ? "" : " "._("(unset with '0')"))); ?>
148 148
                     </span>
149 149
                 </td>
150 150
                 <td>
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
     </fieldset>
165 165
 
166 166
     <?php
167
-    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>";
167
+    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>";
168 168
     echo $deco->footer();
169 169
 
170 170
     
171 171
\ No newline at end of file
Please login to merge, or discard this patch.
web/admin/overview_org.php 1 patch
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
  */
27 27
 ?>
28 28
 <?php
29
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
29
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
30 30
 
31 31
 $deco = new \web\lib\admin\PageDecoration();
32 32
 $validator = new \web\lib\common\InputValidation();
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 } else {
39 39
     $link = 'http://';
40 40
 }
41
-$link .= $_SERVER['SERVER_NAME'] . $_SERVER['SCRIPT_NAME'];
41
+$link .= $_SERVER['SERVER_NAME'].$_SERVER['SCRIPT_NAME'];
42 42
 $link = htmlspecialchars($link);
43 43
 
44 44
 echo $deco->defaultPagePrelude(sprintf(_("%s: %s Dashboard"), \config\Master::APPEARANCE['productname'], $uiElements->nomenclatureParticipant));
@@ -173,14 +173,14 @@  discard block
 block discarded – undo
173 173
                                     <ul style='margin:1px'>
174 174
                                     <?php
175 175
                                     foreach ($completeness as $missing_attrib) {
176
-                                        echo "<li>" . $uiElements->displayName($missing_attrib) . "</li>";
176
+                                        echo "<li>".$uiElements->displayName($missing_attrib)."</li>";
177 177
                                     }
178 178
                                     ?>
179 179
                                     </ul>
180 180
                                 </div>
181 181
                 <?php
182 182
             } else {
183
-                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);
183
+                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);
184 184
             }
185 185
             ?>
186 186
                             <br/>
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
             if ($readonly === FALSE) {
190 190
                 ?>
191 191
                                 <form action='edit_silverbullet.php?inst_id=<?php echo $my_inst->identifier; ?>&amp;profile_id=<?php echo $profile_list->identifier; ?>' method='POST'>
192
-                                    <button <?php echo ( is_array($completeness) ? "disabled" : "" ); ?> type='submit' name='sb_action' value='sb_edit'><?php echo _("Manage User Base"); ?></button>
192
+                                    <button <?php echo (is_array($completeness) ? "disabled" : ""); ?> type='submit' name='sb_action' value='sb_edit'><?php echo _("Manage User Base"); ?></button>
193 193
                                 </form>
194 194
                 <?php
195 195
             }
@@ -214,21 +214,21 @@  discard block
 block discarded – undo
214 214
                                 $has_overrides = TRUE;
215 215
                             }
216 216
                         }
217
-                        $buffer_eaptypediv = "<div style='margin-bottom:40px; float:left;'>" . _("<strong>EAP Types</strong> (in order of preference):") . "<br/>";
217
+                        $buffer_eaptypediv = "<div style='margin-bottom:40px; float:left;'>"._("<strong>EAP Types</strong> (in order of preference):")."<br/>";
218 218
                         $typelist = $profile_list->getEapMethodsinOrderOfPreference();
219 219
                         $allcomplete = TRUE;
220 220
                         foreach ($typelist as $eaptype) {
221 221
                             $buffer_eaptypediv .= $eaptype->getPrintableRep();
222 222
                             $completeness = $profile_list->isEapTypeDefinitionComplete($eaptype);
223 223
                             if ($completeness === true) {
224
-                                $buffer_eaptypediv .= " <div class='acceptable'>" . _("OK") . "</div>";
224
+                                $buffer_eaptypediv .= " <div class='acceptable'>"._("OK")."</div>";
225 225
                             } else {
226 226
                                 $buffer_eaptypediv .= " <div class='notacceptable'>";
227 227
                                 $buffer_eaptypediv .= _("Information needed!");
228 228
                                 if (is_array($completeness)) {
229 229
                                     $buffer_eaptypediv .= "<ul style='margin:1px'>";
230 230
                                     foreach ($completeness as $missing_attrib) {
231
-                                        $buffer_eaptypediv .= "<li>" . $uiElements->displayName($missing_attrib) . "</li>";
231
+                                        $buffer_eaptypediv .= "<li>".$uiElements->displayName($missing_attrib)."</li>";
232 232
                                     }
233 233
                                     $buffer_eaptypediv .= "</ul>";
234 234
                                 }
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
                             foreach ($attribs as $attrib) {
241 241
                                 if ($attrib['level'] == \core\Options::LEVEL_METHOD && !preg_match("/^internal:/", $attrib['name']) && !$justOnce) {
242 242
                                     $justOnce = TRUE;
243
-                                    $buffer_eaptypediv .= "<img src='../resources/images/icons/Letter-E-blue-icon.png' alt='" . _("Options on EAP Method/Device level are in effect.") . "'>";
243
+                                    $buffer_eaptypediv .= "<img src='../resources/images/icons/Letter-E-blue-icon.png' alt='"._("Options on EAP Method/Device level are in effect.")."'>";
244 244
                                 }
245 245
                             }
246 246
                             $buffer_eaptypediv .= "<br/>";
@@ -278,10 +278,10 @@  discard block
 block discarded – undo
278 278
                             if (\config\Master::FUNCTIONALITY_LOCATIONS['DIAGNOSTICS'] == "LOCAL") {
279 279
                                 $diagUrl = "../diag/";
280 280
                             } else {
281
-                                $diagUrl = \config\Master::FUNCTIONALITY_LOCATIONS['DIAGNOSTICS'] . "/diag/";
281
+                                $diagUrl = \config\Master::FUNCTIONALITY_LOCATIONS['DIAGNOSTICS']."/diag/";
282 282
                             }
283 283
                             ?>
284
-                                <form action='<?php echo $diagUrl . "action_realmcheck.php?inst_id=" . $my_inst->identifier . "&profile_id=" . $profile_list->identifier ?>' method='post' accept-charset='UTF-8'>
284
+                                <form action='<?php echo $diagUrl."action_realmcheck.php?inst_id=".$my_inst->identifier."&profile_id=".$profile_list->identifier ?>' method='post' accept-charset='UTF-8'>
285 285
                                     <input type='hidden' name='comefrom' id='comefrom-<?php echo $profilecount; ?>' value='<?php echo $link; ?>'/>
286 286
                                     <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!"); ?>'>
287 287
                 <?php echo _("Check realm reachability"); ?>
@@ -315,9 +315,9 @@  discard block
 block discarded – undo
315 315
                 ?>
316 316
                                 <div style='display: flex;'>
317 317
                                 <?php
318
-                                $idpLevelUrl = ( isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == "on" ? 'https://' : 'http://') . $_SERVER['SERVER_NAME'] . dirname(dirname($_SERVER['SCRIPT_NAME'])) . "?idp=" . $my_inst->identifier;
319
-                                $displayurl = $idpLevelUrl . "&amp;profile=" . $profile_list->identifier;
320
-                                $QRurl = $idpLevelUrl . "&profile=" . $profile_list->identifier;
318
+                                $idpLevelUrl = (isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == "on" ? 'https://' : 'http://').$_SERVER['SERVER_NAME'].dirname(dirname($_SERVER['SCRIPT_NAME']))."?idp=".$my_inst->identifier;
319
+                                $displayurl = $idpLevelUrl."&amp;profile=".$profile_list->identifier;
320
+                                $QRurl = $idpLevelUrl."&profile=".$profile_list->identifier;
321 321
                                 $qrCode = new \chillerlan\QRCode\QRCode(new \chillerlan\QRCode\QROptions([
322 322
                                             'outputType' => \chillerlan\QRCode\QRCode::OUTPUT_IMAGE_PNG,
323 323
                                             'eccLevel' => \chillerlan\QRCode\QRCode::ECC_H,
@@ -329,9 +329,9 @@  discard block
 block discarded – undo
329 329
                                 if (empty($rawQr)) {
330 330
                                     throw new Exception("Something went seriously wrong during QR code generation!");
331 331
                                 }
332
-                                $uri = "data:image/png;base64," . base64_encode($uiElements->pngInjectConsortiumLogo($rawQr, web\lib\admin\UIElements::QRCODE_PIXELS_PER_SYMBOL));
332
+                                $uri = "data:image/png;base64,".base64_encode($uiElements->pngInjectConsortiumLogo($rawQr, web\lib\admin\UIElements::QRCODE_PIXELS_PER_SYMBOL));
333 333
                                 $size = getimagesize($uri);
334
-                                echo "<img width='" . ($size[0] / 4) . "' height='" . ($size[1] / 4) . "' src='$uri' alt='QR-code'/>";
334
+                                echo "<img width='".($size[0] / 4)."' height='".($size[1] / 4)."' src='$uri' alt='QR-code'/>";
335 335
 
336 336
                                 //echo "<nobr>$displayurl</nobr></a>";
337 337
                                 echo "<p>$displayurl</p></a>";
@@ -381,7 +381,7 @@  discard block
 block discarded – undo
381 381
                     <div>
382 382
                         <input type="hidden" name="consortium" value="eduroam"/>
383 383
                         <button type='submit' <?php echo ($hasMail > 0 ? "" : "disabled"); ?> name='profile_action' value='new'>
384
-        <?php echo sprintf(_("Add %s deployment ..."), \config\ConfAssistant::CONSORTIUM['name'] . " " . \core\DeploymentManaged::PRODUCTNAME); ?>
384
+        <?php echo sprintf(_("Add %s deployment ..."), \config\ConfAssistant::CONSORTIUM['name']." ".\core\DeploymentManaged::PRODUCTNAME); ?>
385 385
                         </button>
386 386
 
387 387
                     </div>
@@ -407,7 +407,7 @@  discard block
 block discarded – undo
407 407
         <?php
408 408
         $hotspotProfiles = $my_inst->listDeployments();
409 409
         if (count($hotspotProfiles) == 0) { // no profiles yet.
410
-            echo "<h2>" . sprintf(_("There are not yet any known deployments for your %s."), $uiElements->nomenclatureHotspot) . "</h2>";
410
+            echo "<h2>".sprintf(_("There are not yet any known deployments for your %s."), $uiElements->nomenclatureHotspot)."</h2>";
411 411
         }
412 412
 
413 413
         foreach ($hotspotProfiles as $counter => $deploymentObject) {
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
                 <h2><?php
429 429
     switch ($deploymentObject->consortium) {
430 430
         case "eduroam":
431
-            $displayname = config\ConfAssistant::CONSORTIUM['name'] . " " . core\DeploymentManaged::PRODUCTNAME;
431
+            $displayname = config\ConfAssistant::CONSORTIUM['name']." ".core\DeploymentManaged::PRODUCTNAME;
432 432
             break;
433 433
         case "OpenRoaming":
434 434
             $displayname = "OpenRoaming ANP";
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
         default:
437 437
             throw new Exception("We are supposed to operate on a roaming consortium we don't know.");
438 438
     }
439
-    echo $displayname . " (<span style='color:" . ( $deploymentObject->status == \core\AbstractDeployment::INACTIVE ? "red;'>" . _("inactive") : "green;'>" . _("active") ) . "</span>)";
439
+    echo $displayname." (<span style='color:".($deploymentObject->status == \core\AbstractDeployment::INACTIVE ? "red;'>"._("inactive") : "green;'>"._("active"))."</span>)";
440 440
             ?></h2>
441 441
                 <table>
442 442
                     <caption><?php echo _("Deployment Details"); ?></caption>
@@ -450,13 +450,13 @@  discard block
 block discarded – undo
450 450
                         <td><strong><?php echo _("Your primary RADIUS server") ?></strong><br/>
451 451
     <?php
452 452
     if ($deploymentObject->host1_v4 !== NULL) {
453
-        echo _("IPv4") . ": " . $deploymentObject->host1_v4;
453
+        echo _("IPv4").": ".$deploymentObject->host1_v4;
454 454
     }
455 455
     if ($deploymentObject->host1_v4 !== NULL && $deploymentObject->host1_v6 !== NULL) {
456 456
         echo "<br/>";
457 457
     }
458 458
     if ($deploymentObject->host1_v6 !== NULL) {
459
-        echo _("IPv6") . ": " . $deploymentObject->host1_v6;
459
+        echo _("IPv6").": ".$deploymentObject->host1_v6;
460 460
     }
461 461
     ?>
462 462
                         </td>
@@ -464,9 +464,9 @@  discard block
 block discarded – undo
464 464
                         <td><?php echo $deploymentObject->port1; ?></td>
465 465
                         <td>
466 466
     <?php
467
-    echo "<img src='" . $radiusMessages[$deploymentObject->radius_status_1]['icon'] .
468
-    "' alt='" . $radiusMessages[$deploymentObject->radius_status_1]['text'] .
469
-    "' title='" . $radiusMessages[$deploymentObject->radius_status_1]['text'] . "'>";
467
+    echo "<img src='".$radiusMessages[$deploymentObject->radius_status_1]['icon'].
468
+    "' alt='".$radiusMessages[$deploymentObject->radius_status_1]['text'].
469
+    "' title='".$radiusMessages[$deploymentObject->radius_status_1]['text']."'>";
470 470
     ?>
471 471
                         </td>
472 472
                     </tr>
@@ -474,22 +474,22 @@  discard block
 block discarded – undo
474 474
                         <td><strong><?php echo _("Your backup RADIUS server") ?><br/></strong>
475 475
     <?php
476 476
     if ($deploymentObject->host2_v4 !== NULL) {
477
-        echo _("IPv4") . ": " . $deploymentObject->host2_v4;
477
+        echo _("IPv4").": ".$deploymentObject->host2_v4;
478 478
     }
479 479
     if ($deploymentObject->host2_v4 !== NULL && $deploymentObject->host2_v6 !== NULL) {
480 480
         echo "<br/>";
481 481
     }
482 482
     if ($deploymentObject->host2_v6 !== NULL) {
483
-        echo _("IPv6") . ": " . $deploymentObject->host2_v6;
483
+        echo _("IPv6").": ".$deploymentObject->host2_v6;
484 484
     }
485 485
     ?></td>
486 486
                         <td><?php echo _("RADIUS port number: ") ?></td>
487 487
                         <td><?php echo $deploymentObject->port2; ?></td>
488 488
                         <td>
489 489
     <?php
490
-    echo "<img src='" . $radiusMessages[$deploymentObject->radius_status_2]['icon'] .
491
-    "' alt='" . $radiusMessages[$deploymentObject->radius_status_2]['text'] .
492
-    "' title='" . $radiusMessages[$deploymentObject->radius_status_2]['text'] . "'>";
490
+    echo "<img src='".$radiusMessages[$deploymentObject->radius_status_2]['icon'].
491
+    "' alt='".$radiusMessages[$deploymentObject->radius_status_2]['text'].
492
+    "' title='".$radiusMessages[$deploymentObject->radius_status_2]['text']."'>";
493 493
     ?>
494 494
                         </td>
495 495
                     </tr>
@@ -541,12 +541,12 @@  discard block
 block discarded – undo
541 541
                                     if (array_key_exists('FAILURE', $res) && $res['FAILURE'] > 0) {
542 542
                                         echo '<br>';
543 543
                                         if ($res['FAILURE'] == 2) {
544
-                                            echo ' <span style="color: red;">' . _("Activation failure.") . '</span>';
544
+                                            echo ' <span style="color: red;">'._("Activation failure.").'</span>';
545 545
                                         } else {
546 546
                                             if (isset($_GET['res'][1]) && $_GET['res']['1'] == 'FAILURE') {
547
-                                                echo ' <span style="color: red;">' . _("Activation failure for your primary RADIUS server.") . '</span>';
547
+                                                echo ' <span style="color: red;">'._("Activation failure for your primary RADIUS server.").'</span>';
548 548
                                             } else {
549
-                                                echo ' <span style="color: red;">' . _("Activation failure for your backup RADIUS server.") . '</span>';
549
+                                                echo ' <span style="color: red;">'._("Activation failure for your backup RADIUS server.").'</span>';
550 550
                                             }
551 551
                                         }
552 552
                                     }
@@ -566,12 +566,12 @@  discard block
 block discarded – undo
566 566
                                     if ($res['FAILURE'] > 0) {
567 567
                                         echo '<br>';
568 568
                                         if ($res['FAILURE'] == 2) {
569
-                                            echo ' <span style="color: red;">' . _("Failure during deactivation, your request is queued for handling") . '</span>';
569
+                                            echo ' <span style="color: red;">'._("Failure during deactivation, your request is queued for handling").'</span>';
570 570
                                         } else {
571 571
                                             if (isset($_GET['res'][1]) && $_GET['res']['1'] == 'FAILURE') {
572
-                                                echo ' <span style="color: red;">' . _("Deactivation failure for your primary RADIUS server, your request is queued.") . '</span>';
572
+                                                echo ' <span style="color: red;">'._("Deactivation failure for your primary RADIUS server, your request is queued.").'</span>';
573 573
                                             } else {
574
-                                                echo ' <span style="color: red;">' . _("Deactivation failure for your backup RADIUS server, your request is queued.") . '</span>';
574
+                                                echo ' <span style="color: red;">'._("Deactivation failure for your backup RADIUS server, your request is queued.").'</span>';
575 575
                                             }
576 576
                                         }
577 577
                                     }
Please login to merge, or discard this patch.