Passed
Push — master ( b32147...6275ed )
by Maja
06:20
created
web/user/API.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
  *
25 25
  * @package UserAPI
26 26
  */
27
-require dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
27
+require dirname(dirname(dirname(__FILE__)))."/config/_config.php";
28 28
 $API = new \core\UserNetAPI();
29 29
 $validator = new web\lib\common\InputValidation();
30 30
 $loggerInstance = new \core\common\Logging();
Please login to merge, or discard this patch.
web/user/tou.inc.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-        $Tou =  [
22
+        $Tou = [
23 23
     'title' => _("Terms of Use"),
24 24
     'subtitle' => _("(for legal reasons, the Terms of Use are in English only)"),
25 25
     'short' => "<p>The purpose of this service is to make eduroam configuration easier for end users and eduroam
Please login to merge, or discard this patch.
web/admin/edit_user.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
22
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
23 23
 
24 24
 $deco = new \web\lib\admin\PageDecoration();
25 25
 $uiElements = new web\lib\admin\UIElements();
Please login to merge, or discard this patch.
web/admin/inc/toggleRedirect.inc.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-require_once dirname(dirname(dirname(dirname(__FILE__)))) . "/config/_config.php";
22
+require_once dirname(dirname(dirname(dirname(__FILE__))))."/config/_config.php";
23 23
 
24 24
 $auth = new \web\lib\admin\Authentication();
25 25
 $loggerInstance = new \core\common\Logging();
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $remaining_attribs = $my_profile->beginFlushMethodLevelAttributes($eaptype->getIntegerRep(), NULL);
86 86
         $optionParser->processSubmittedFields($my_profile, $_POST, $_FILES, $eaptype->getIntegerRep(), NULL);
87 87
     }
88
-    $loggerInstance->writeAudit($_SESSION['user'], "MOD", "Profile " . $my_profile->identifier . " - device/EAP-Type settings changed");
88
+    $loggerInstance->writeAudit($_SESSION['user'], "MOD", "Profile ".$my_profile->identifier." - device/EAP-Type settings changed");
89 89
     header("Location: ../overview_installers.php?inst_id=$my_inst->identifier&profile_id=$my_profile->identifier");
90 90
     exit;
91 91
 }
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
     }
100 100
     $captiontext = sprintf(_("device <strong>%s</strong>"), $device['display']);
101 101
     $keyword = "device-specific";
102
-    $extrainput = "<input type='hidden' name='device' value='" . $device_key . "'/>";
102
+    $extrainput = "<input type='hidden' name='device' value='".$device_key."'/>";
103 103
 } elseif ($eaptype !== NULL) {
104 104
     foreach ($my_profile->getAttributes() as $attrib) {
105 105
         if (isset($attrib['eapmethod']) && $attrib['eapmethod'] == $eaptype->getArrayRep()) {
@@ -109,12 +109,12 @@  discard block
 block discarded – undo
109 109
 
110 110
     $captiontext = sprintf(_("EAP-Type <strong>%s</strong>"), $eaptype->getPrintableRep());
111 111
     $keyword = "eap-specific";
112
-    $extrainput = "<input type='hidden' name='eaptype' value='" . $eaptype->getIntegerRep() . "'>";
112
+    $extrainput = "<input type='hidden' name='eaptype' value='".$eaptype->getIntegerRep()."'>";
113 113
 } else {
114 114
     throw new Exception("previous type checks make it impossible to reach this code path.");
115 115
 }
116 116
 ?>
117
-<p><?php echo _("Fine-tuning options for ") . $captiontext; ?></p>
117
+<p><?php echo _("Fine-tuning options for ").$captiontext; ?></p>
118 118
 <hr/>
119 119
 
120 120
 <form action='inc/toggleRedirect.inc.php?inst_id=<?php echo $my_inst->identifier; ?>&amp;profile_id=<?php echo $my_profile->identifier; ?>' method='post' accept-charset='UTF-8'><?php echo $extrainput; ?>
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
     $interesting_attribs = [];
124 124
 
125 125
     foreach ($attribs as $attrib) {
126
-        if ($attrib['level'] == "Method" && preg_match('/^' . $keyword . ':/', $attrib['name'])) {
126
+        if ($attrib['level'] == "Method" && preg_match('/^'.$keyword.':/', $attrib['name'])) {
127 127
             $interesting_attribs[] = $attrib;
128 128
         }
129 129
     }
Please login to merge, or discard this patch.
web/admin/inc/logout.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
  */
21 21
 ?>
22 22
 <?php
23
-require_once dirname(dirname(dirname(dirname(__FILE__)))) . "/config/_config.php";
23
+require_once dirname(dirname(dirname(dirname(__FILE__))))."/config/_config.php";
24 24
 
25 25
 $auth = new \web\lib\admin\Authentication();
26 26
 
Please login to merge, or discard this patch.
web/admin/inc/click_button_js.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-require_once dirname(dirname(dirname(dirname(__FILE__)))) . "/config/_config.php";
22
+require_once dirname(dirname(dirname(dirname(__FILE__))))."/config/_config.php";
23 23
 ?>
24 24
 
25 25
 <!-- JQuery --> 
Please login to merge, or discard this patch.
web/admin/overview_installers.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-require_once dirname(dirname(__DIR__)) . "/config/_config.php";
22
+require_once dirname(dirname(__DIR__))."/config/_config.php";
23 23
 
24 24
 $deco = new \web\lib\admin\PageDecoration();
25 25
 $validator = new \web\lib\common\InputValidation();
@@ -53,10 +53,10 @@  discard block
 block discarded – undo
53 53
             <?php
54 54
             foreach ($preflist as $method) {
55 55
                 $escapedMethod = $method->getIntegerRep();
56
-                echo "<th style='min-width:200px'>" . $method->getPrintableRep() . "<br/>
56
+                echo "<th style='min-width:200px'>".$method->getPrintableRep()."<br/>
57 57
                         <form method='post' action='inc/toggleRedirect.inc.php?inst_id=$my_inst->identifier&amp;profile_id=$my_profile->identifier' onsubmit='popupRedirectWindow(this); return false;' accept-charset='UTF-8'>
58 58
                         <input type='hidden' name='eaptype' value='$escapedMethod'>
59
-                        <button class='redirect' type='submit'>" . _("EAP-Type-specific options...") . "</button>
59
+                        <button class='redirect' type='submit'>"._("EAP-Type-specific options...")."</button>
60 60
                         </form></th>";
61 61
             }
62 62
             ?>
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
         foreach (\devices\Devices::listDevices() as $index => $description) {
73 73
 
74 74
             echo "<tr>";
75
-            echo "<td align='center'><img src='../resources/images/vendorlogo/" . $description['group'] . ".png' alt='logo'></td><td>" . $description['display'] . "<br/>
75
+            echo "<td align='center'><img src='../resources/images/vendorlogo/".$description['group'].".png' alt='logo'></td><td>".$description['display']."<br/>
76 76
                         <form method='post' action='inc/toggleRedirect.inc.php?inst_id=$my_inst->identifier&amp;profile_id=$my_profile->identifier' onsubmit='popupRedirectWindow(this); return false;' accept-charset='UTF-8'>
77 77
                         <input type='hidden' name='device' value='$index'>
78
-                        <button class='redirect' type='submit'>" . _("Device-specific options...") . "</button>
78
+                        <button class='redirect' type='submit'>"._("Device-specific options...")."</button>
79 79
                         </form>
80 80
                         </td>";
81 81
             $factory = new \core\DeviceFactory($index);
@@ -84,10 +84,10 @@  discard block
 block discarded – undo
84 84
                 $footnotesForDevEapCombo = [];
85 85
                 $display_footnote = FALSE;
86 86
                 $langObject = new \core\common\Language();
87
-                $downloadform = "<form action='" . rtrim(dirname(dirname($_SERVER['SCRIPT_NAME'])), '/') . "/user/API.php?action=downloadInstaller&profile=$my_profile->identifier&lang=" . $langObject->getLang() . "' method='post' accept-charset='UTF-8'>
87
+                $downloadform = "<form action='".rtrim(dirname(dirname($_SERVER['SCRIPT_NAME'])), '/')."/user/API.php?action=downloadInstaller&profile=$my_profile->identifier&lang=".$langObject->getLang()."' method='post' accept-charset='UTF-8'>
88 88
                                        <input type='hidden' name='device' value='$index'/>
89 89
                                        <input type='hidden' name='generatedfor'  value='admin'/>
90
-                                       <button class='download'>" . _("Download") . "</button>
90
+                                       <button class='download'>"._("Download")."</button>
91 91
                                      ";
92 92
                 // first of all: if redirected, indicate by color
93 93
 
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
     </table>
159 159
     <?php
160 160
     if (count($distinctFootnotes)) {
161
-        echo "<p><strong>" . _("Footnotes:") . "</strong></p><table>";
161
+        echo "<p><strong>"._("Footnotes:")."</strong></p><table>";
162 162
         foreach ($distinctFootnotes as $number => $text) {
163 163
             echo "<tr><td>($number) - </td><td>$text</td></tr>";
164 164
         }
Please login to merge, or discard this patch.
web/admin/edit_profile_result.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
22
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
23 23
 
24 24
 $auth = new \web\lib\admin\Authentication();
25 25
 $deco = new \web\lib\admin\PageDecoration();
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
         $profile = $my_profile;
116 116
     } else {
117 117
         $profile = $my_inst->newProfile(core\AbstractProfile::PROFILETYPE_RADIUS);
118
-        $loggerInstance->writeAudit($_SESSION['user'], "NEW", "IdP " . $my_inst->identifier . " - Profile created");
118
+        $loggerInstance->writeAudit($_SESSION['user'], "NEW", "IdP ".$my_inst->identifier." - Profile created");
119 119
     }
120 120
 }
121 121
 
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
     $uiElements = new web\lib\admin\UIElements();
131 131
     // set realm info, if submitted
132 132
     if ($realm !== FALSE) {
133
-        $profile->setRealm($anonLocal . "@" . $realm);
133
+        $profile->setRealm($anonLocal."@".$realm);
134 134
         echo $uiElements->boxOkay(sprintf(_("Realm: <strong>%s</strong>"), $realm));
135 135
     } else {
136 136
         $profile->setRealm("");
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
             echo $uiElements->boxError(_("Realm check username cannot be configured: realm is missing!"));
154 154
         } else {
155 155
             $profile->setRealmcheckUser(true, $checkuser_name);
156
-            echo $uiElements->boxOkay(sprintf(_("Special username for realm check is <strong>%s</strong>, the value is <strong>%s</strong>"), _("ON"), $checkuser_name . "@" . $realm));
156
+            echo $uiElements->boxOkay(sprintf(_("Special username for realm check is <strong>%s</strong>, the value is <strong>%s</strong>"), _("ON"), $checkuser_name."@".$realm));
157 157
         }
158 158
     } else {
159 159
         $profile->setRealmCheckUser(false);
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
         } else {
167 167
             $profile->setInputVerificationPreference($verify, $hint);
168 168
             if ($hint !== FALSE) {
169
-                $extratext = " " . sprintf(_("and the input field will be prefilled with '<strong>@%s</strong>'."), $realm);
169
+                $extratext = " ".sprintf(_("and the input field will be prefilled with '<strong>@%s</strong>'."), $realm);
170 170
             } else {
171 171
                 $extratext = ".";
172 172
             }
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
         echo $uiElements->boxOkay(_("Redirection is <strong>OFF</strong>"));
204 204
     }
205 205
 
206
-    $loggerInstance->writeAudit($_SESSION['user'], "MOD", "Profile " . $profile->identifier . " - attributes changed");
206
+    $loggerInstance->writeAudit($_SESSION['user'], "MOD", "Profile ".$profile->identifier." - attributes changed");
207 207
 
208 208
     // re-instantiate $profile, we need to do completion checks and need fresh data for isEapTypeDefinitionComplete()
209 209
 
@@ -216,23 +216,23 @@  discard block
 block discarded – undo
216 216
         if ($a->getIntegerRep() == \core\common\EAP::INTEGER_SILVERBULLET) { // do not allow adding silverbullet via the backdoor
217 217
             continue;
218 218
         }
219
-        if (isset($_POST[$a->getPrintableRep()]) && isset($_POST[$a->getPrintableRep() . "-priority"]) && is_numeric($_POST[$a->getPrintableRep() . "-priority"])) {
220
-            $priority = (int) $_POST[$a->getPrintableRep() . "-priority"];
219
+        if (isset($_POST[$a->getPrintableRep()]) && isset($_POST[$a->getPrintableRep()."-priority"]) && is_numeric($_POST[$a->getPrintableRep()."-priority"])) {
220
+            $priority = (int) $_POST[$a->getPrintableRep()."-priority"];
221 221
             // add EAP type to profile as requested, but ...
222 222
             $profile->addSupportedEapMethod($a, $priority);
223
-            $loggerInstance->writeAudit($_SESSION['user'], "MOD", "Profile " . $profile->identifier . " - supported EAP types changed");
223
+            $loggerInstance->writeAudit($_SESSION['user'], "MOD", "Profile ".$profile->identifier." - supported EAP types changed");
224 224
             // see if we can enable the EAP type, or if info is missing
225 225
             $eapcompleteness = $profile->isEapTypeDefinitionComplete($a);
226 226
             if ($eapcompleteness === true) {
227
-                echo $uiElements->boxOkay(_("Supported EAP Type: ") . "<strong>" . $a->getPrintableRep() . "</strong>");
227
+                echo $uiElements->boxOkay(_("Supported EAP Type: ")."<strong>".$a->getPrintableRep()."</strong>");
228 228
             } else {
229 229
                 $warntext = "";
230 230
                 if (is_array($eapcompleteness)) {
231 231
                     foreach ($eapcompleteness as $item) {
232
-                        $warntext .= "<strong>" . $uiElements->displayName($item) . "</strong> ";
232
+                        $warntext .= "<strong>".$uiElements->displayName($item)."</strong> ";
233 233
                     }
234 234
                 }
235
-                echo $uiElements->boxWarning(sprintf(_("Supported EAP Type: <strong>%s</strong> is missing required information %s !"), $a->getPrintableRep(), $warntext) . "<br/>" . _("The EAP type was added to the profile, but you need to complete the missing information before we can produce installers for you."));
235
+                echo $uiElements->boxWarning(sprintf(_("Supported EAP Type: <strong>%s</strong> is missing required information %s !"), $a->getPrintableRep(), $warntext)."<br/>"._("The EAP type was added to the profile, but you need to complete the missing information before we can produce installers for you."));
236 236
             }
237 237
         }
238 238
     }
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
 <?php
249 249
 if (count($reloadedProfile->getEapMethodsinOrderOfPreference(1)) > 0) {
250 250
     echo "<form method='post' action='overview_installers.php?inst_id=$my_inst->identifier&profile_id=$reloadedProfile->identifier' accept-charset='UTF-8'>
251
-        <button type='submit'>" . _("Continue to Installer Fine-Tuning and Download") . "</button>
251
+        <button type='submit'>"._("Continue to Installer Fine-Tuning and Download")."</button>
252 252
     </form>";
253 253
 }
254 254
 echo $deco->footer();
Please login to merge, or discard this patch.
web/admin/112365365321.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  *          <base_url>/copyright.php after deploying the software
20 20
  */
21 21
 
22
-require_once dirname(dirname(__DIR__)) . '/config/_config.php';
22
+require_once dirname(dirname(__DIR__)).'/config/_config.php';
23 23
 
24 24
 $uiElements = new web\lib\admin\UIElements();
25 25
 
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
                     // filesystem cleanup
73 73
                     case web\lib\common\FormElements::BUTTON_DELETE:
74 74
                         $i = web\lib\admin\Maintenance::deleteObsoleteTempDirs();
75
-                        echo "<div class='ca-summary'><table>" . $uiElements->boxRemark(sprintf("Deleted %d cache directories.", $i), "Cache deleted") . "</table></div>";
75
+                        echo "<div class='ca-summary'><table>".$uiElements->boxRemark(sprintf("Deleted %d cache directories.", $i), "Cache deleted")."</table></div>";
76 76
                         break;
77 77
                     default:
78 78
                         break;
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
                     $admin_query = $dbHandle->exec("SELECT SUM(downloads_admin) AS admin, SUM(downloads_user) AS user, SUM(downloads_silverbullet) as silverbullet FROM downloads WHERE device_id = '$index'");
139 139
                     // SELECT -> mysqli_result, not boolean
140 140
                     while ($a = mysqli_fetch_object(/** @scrutinizer ignore-type */ $admin_query)) {
141
-                        echo "<td>" . $device_array['display'] . "</td><td>" . $a->admin . "</td><td>" . $a->user . "</td><td>" . $a->silverbullet . "</td><td>" . sprintf("%s", $a->user + $a->silverbullet) . "</td>";
141
+                        echo "<td>".$device_array['display']."</td><td>".$a->admin."</td><td>".$a->user."</td><td>".$a->silverbullet."</td><td>".sprintf("%s", $a->user + $a->silverbullet)."</td>";
142 142
                         $gross_admin = $gross_admin + $a->admin;
143 143
                         $gross_user = $gross_user + $a->user;
144 144
                         $gross_silverbullet = $gross_silverbullet + $a->silverbullet;
Please login to merge, or discard this patch.