Test Setup Failed
Push — release_2_1 ( 12ba21...8865c9 )
by Stefan
07:36
created
devices/Devices-template.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -411,7 +411,7 @@  discard block
 block discarded – undo
411 411
                     'hs20' => 1,
412 412
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
413 413
                             "geteduroam",
414
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=app.eduroam.geteduroam'>Google Play</a>, <a target='_blank' href='geteduroam-stable.apk'>" . _("as local download") . "</a>"),
414
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=app.eduroam.geteduroam'>Google Play</a>, <a target='_blank' href='geteduroam-stable.apk'>"._("as local download")."</a>"),
415 415
                 ],
416 416
             ],            
417 417
 
@@ -426,7 +426,7 @@  discard block
 block discarded – undo
426 426
                     'mime' => 'application/eap-config',
427 427
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
428 428
                             "geteduroam",
429
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=app.eduroam.geteduroam'>Google Play</a>, <a target='_blank' href='geteduroam-stable.apk'>" . _("as local download") . "</a>"),
429
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=app.eduroam.geteduroam'>Google Play</a>, <a target='_blank' href='geteduroam-stable.apk'>"._("as local download")."</a>"),
430 430
                 ],
431 431
             ],              
432 432
 
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
                     'mime' => 'application/eap-config',
441 441
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
442 442
                             "eduroamCAT",
443
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
443
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
444 444
                 ],
445 445
             ],            
446 446
             
@@ -455,7 +455,7 @@  discard block
 block discarded – undo
455 455
                     'mime' => 'application/eap-config',
456 456
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
457 457
                             "eduroamCAT",
458
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
458
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
459 459
                 ],
460 460
             ],
461 461
             'android_pie' => [
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
                     'mime' => 'application/eap-config',
470 470
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
471 471
                             "eduroamCAT",
472
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
472
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
473 473
                 ],
474 474
             ],
475 475
             'android_oreo' => [
@@ -483,7 +483,7 @@  discard block
 block discarded – undo
483 483
                     'mime' => 'application/eap-config',
484 484
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
485 485
                             "eduroamCAT",
486
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
486
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
487 487
                 ],
488 488
             ],
489 489
             'android_nougat' => [
@@ -497,7 +497,7 @@  discard block
 block discarded – undo
497 497
                     'mime' => 'application/eap-config',
498 498
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
499 499
                             "eduroamCAT",
500
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
500
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
501 501
                 ],
502 502
             ],
503 503
             'android_marshmallow' => [
@@ -511,7 +511,7 @@  discard block
 block discarded – undo
511 511
                     'mime' => 'application/eap-config',
512 512
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
513 513
                             "eduroamCAT",
514
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
514
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
515 515
                 ],
516 516
             ],
517 517
             'android_lollipop' => [
@@ -525,7 +525,7 @@  discard block
 block discarded – undo
525 525
                     'mime' => 'application/eap-config',
526 526
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
527 527
                             "eduroamCAT",
528
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
528
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
529 529
                 ],
530 530
             ],
531 531
             'android_kitkat' => [
@@ -539,7 +539,7 @@  discard block
 block discarded – undo
539 539
                     'mime' => 'application/eap-config',
540 540
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
541 541
                             "eduroamCAT",
542
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
542
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
543 543
                 ],
544 544
             ],
545 545
             'android_43' => [
@@ -553,7 +553,7 @@  discard block
 block discarded – undo
553 553
                     'mime' => 'application/eap-config',
554 554
                     'message' => sprintf(_("Before you proceed with installation on Android systems, please make sure that you have installed the %s application. This application is available from these sites: %s and will use the configuration file downloaded from CAT to create all necessary settings."),
555 555
                             "eduroamCAT",
556
-                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>" . _("as local download") . "</a>"),
556
+                            "<a target='_blank' href='https://play.google.com/store/apps/details?id=uk.ac.swansea.eduroamcat'>Google Play</a>, <a target='_blank' href='https://www.amazon.com/dp/B01EACCX0S/'>Amazon Appstore</a>, <a target='_blank' href='eduroamCAT-stable.apk'>"._("as local download")."</a>"),
557 557
                 ],
558 558
             ],
559 559
 
Please login to merge, or discard this patch.
web/admin/overview_installers.php 1 patch
Spacing   +9 added lines, -9 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();
@@ -55,10 +55,10 @@  discard block
 block discarded – undo
55 55
             <?php
56 56
             foreach ($preflist as $method) {
57 57
                 $escapedMethod = $method->getIntegerRep();
58
-                echo "<th  scope='col' style='min-width:200px'>" . $method->getPrintableRep() . "<br/>
58
+                echo "<th  scope='col' style='min-width:200px'>".$method->getPrintableRep()."<br/>
59 59
                         <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'>
60 60
                         <input type='hidden' name='eaptype' value='$escapedMethod'>
61
-                        <button class='redirect' type='submit'>" . _("EAP-Type-specific options...") . "</button>
61
+                        <button class='redirect' type='submit'>"._("EAP-Type-specific options...")."</button>
62 62
                         </form></th>";
63 63
             }
64 64
             ?>
@@ -78,10 +78,10 @@  discard block
 block discarded – undo
78 78
                 continue;
79 79
             }
80 80
             echo "<tr>";
81
-            echo "<td align='center'><img src='../resources/images/vendorlogo/" . $description['group'] . ".png' alt='logo'></td><td>" . $description['display'] . "<br/>
81
+            echo "<td align='center'><img src='../resources/images/vendorlogo/".$description['group'].".png' alt='logo'></td><td>".$description['display']."<br/>
82 82
                         <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'>
83 83
                         <input type='hidden' name='device' value='$index'>
84
-                        <button class='redirect' type='submit'>" . _("Device-specific options...") . "</button>
84
+                        <button class='redirect' type='submit'>"._("Device-specific options...")."</button>
85 85
                         </form>
86 86
                         </td>";
87 87
             $defaultisset = FALSE;
@@ -90,13 +90,13 @@  discard block
 block discarded – undo
90 90
                 $display_footnote = FALSE;
91 91
                 $langObject = new \core\common\Language();
92 92
                 $downloadform = "<span style='display:ruby;'>";
93
-                $formDiffs = ["" => sprintf(_("%s<br/>Installer"), config\ConfAssistant::CONSORTIUM['display_name'])];  /* eduroam */
93
+                $formDiffs = ["" => sprintf(_("%s<br/>Installer"), config\ConfAssistant::CONSORTIUM['display_name'])]; /* eduroam */
94 94
                 if (sizeof($my_profile->getAttributes("media:openroaming")) > 0 && isset($factory->device->options['hs20']) && $factory->device->options['hs20'] == 1) {
95 95
                     $formDiffs["<input type='hidden' name='openroaming'  value='1'/>"] = sprintf(_("%s + OpenRoaming<br/>Installer"), config\ConfAssistant::CONSORTIUM['display_name']); /* eduroam + OpenRoaming */
96 96
                 }
97 97
 
98 98
                 foreach ($formDiffs as $inputField => $text) {
99
-                    $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'>
99
+                    $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'>
100 100
                                        <input type='hidden' name='device' value='$index'/>
101 101
                                        <input type='hidden' name='generatedfor'  value='admin'/>
102 102
                                        $inputField
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
                                     $distinctFootnotes[$num_footnotes] = $oneFootnote;
151 151
                                 }
152 152
                                 $numberToDisplay = array_keys($distinctFootnotes, $oneFootnote);
153
-                                echo "(" . $numberToDisplay[0] . ")";
153
+                                echo "(".$numberToDisplay[0].")";
154 154
                             }
155 155
                         }
156 156
                         echo "</td>";
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
     </table>
180 180
     <?php
181 181
     if (count($distinctFootnotes)) {
182
-        echo "<p><strong>" . _("Footnotes:") . "</strong></p><table>";
182
+        echo "<p><strong>"._("Footnotes:")."</strong></p><table>";
183 183
         foreach ($distinctFootnotes as $number => $text) {
184 184
             echo "<tr><td>($number) - </td><td>$text</td></tr>";
185 185
         }
Please login to merge, or discard this patch.
web/lib/user/TextTemplates.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 const DOWNLOAD_FOR_MESSAGE = 1055;
53 53
 const SB_GO_AWAY = 1060;
54 54
 const SB_FRONTPAGE_BIGDOWNLOADBUTTON = 1061;
55
-const SB_FRONTPAGE_ROLLER_CUSTOMBUILT= 1062;
55
+const SB_FRONTPAGE_ROLLER_CUSTOMBUILT = 1062;
56 56
 
57 57
 
58 58
 /**
@@ -104,13 +104,13 @@  discard block
 block discarded – undo
104 104
         ];
105 105
         $this->templates[WELCOME_ABOARD_TERMS] = "";
106 106
         foreach ($this->templates[NETWORK_TERMS_AND_PRIV] as $consortium => $terms) {
107
-            $this->templates[WELCOME_ABOARD_TERMS] .= sprintf("<p>" . _("Please remember that when connecting to %s hotspots, the following <a href='%s'>Terms and Conditions</a> and <a href='%s'>Privacy Notice</a> apply.") . "</p>", $consortium, $terms['TOU_LINK'], $terms['PRIV_LINK']);
107
+            $this->templates[WELCOME_ABOARD_TERMS] .= sprintf("<p>"._("Please remember that when connecting to %s hotspots, the following <a href='%s'>Terms and Conditions</a> and <a href='%s'>Privacy Notice</a> apply.")."</p>", $consortium, $terms['TOU_LINK'], $terms['PRIV_LINK']);
108 108
         }
109 109
     //    $this->templates[WELCOME_ABOARD_TERMS] .= "<p>"._("I agree to be bound by these Terms and Conditions.")."</p>";
110 110
         $this->templates[WELCOME_ABOARD_BACKTODOWNLOADS] = _("Back to downloads");
111 111
         $this->templates[EDUROAM_WELCOME_ADVERTISING] = sprintf(_("We would like to warmly welcome you among the several million users of %s! From now on, you will be able to use internet access resources on thousands of universities, research centres and other places all over the globe. All of this completely free of charge!"), \config\ConfAssistant::CONSORTIUM['display_name']);
112 112
         $this->templates[HEADING_TOPLEVEL_GREET] = sprintf(_("Welcome to %s"), \config\Master::APPEARANCE['productname']);
113
-        $this->templates[HEADING_TOPLEVEL_PURPOSE] = sprintf(_("Connect your device to %s"),\config\ConfAssistant::CONSORTIUM['display_name']);
113
+        $this->templates[HEADING_TOPLEVEL_PURPOSE] = sprintf(_("Connect your device to %s"), \config\ConfAssistant::CONSORTIUM['display_name']);
114 114
         $this->templates[FRONTPAGE_ROLLER_EASY] = sprintf(_("%s installation made easy:"), \config\ConfAssistant::CONSORTIUM['display_name']);
115 115
         $this->templates[FRONTPAGE_ROLLER_CUSTOMBUILT] = _("Custom built for your organisation");
116 116
         $this->templates[FRONTPAGE_BIGDOWNLOADBUTTON] = sprintf(_("Click here to download your %s installer"), \config\ConfAssistant::CONSORTIUM['display_name'], \config\ConfAssistant::CONSORTIUM['display_name']);
Please login to merge, or discard this patch.
web/skins/modern/diag/js/diag_js.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
  * License: see the web/copyright.inc.php file in the file structure or
20 20
  *          <base_url>/copyright.php after deploying the software
21 21
  */
22
- ?>
22
+    ?>
23 23
 <?php
24 24
 $Gui = new \web\lib\user\Gui();
25 25
 $Gui->languageInstance->setTextDomain("diagnostics");
Please login to merge, or discard this patch.
core/common/OutsideComm.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
             $loggerInstance->debug(4, "OutsideComm::mailAddressValidSecure: no MX.");
124 124
             return OutsideComm::MAILDOMAIN_NO_MX;
125 125
         }
126
-        $loggerInstance->debug(5, "Domain: $domain MX: " . /** @scrutinizer ignore-type */ print_r($mx, TRUE));
126
+        $loggerInstance->debug(5, "Domain: $domain MX: "./** @scrutinizer ignore-type */ print_r($mx, TRUE));
127 127
         // create a pool of A and AAAA records for all the MXes
128 128
         $ipAddrs = [];
129 129
         foreach ($mx as $onemx) {
@@ -133,14 +133,14 @@  discard block
 block discarded – undo
133 133
                 $ipAddrs[] = $oneipv4['ip'];
134 134
             }
135 135
             foreach ($v6list as $oneipv6) {
136
-                $ipAddrs[] = "[" . $oneipv6['ipv6'] . "]";
136
+                $ipAddrs[] = "[".$oneipv6['ipv6']."]";
137 137
             }
138 138
         }
139 139
         if (count($ipAddrs) == 0) {
140 140
             $loggerInstance->debug(4, "OutsideComm::mailAddressValidSecure: no mailserver hosts.");
141 141
             return OutsideComm::MAILDOMAIN_NO_HOST;
142 142
         }
143
-        $loggerInstance->debug(5, "Domain: $domain Addrs: " . /** @scrutinizer ignore-type */ print_r($ipAddrs, TRUE));
143
+        $loggerInstance->debug(5, "Domain: $domain Addrs: "./** @scrutinizer ignore-type */ print_r($ipAddrs, TRUE));
144 144
         // connect to all hosts. If all can't connect, return MAILDOMAIN_NO_CONNECT. 
145 145
         // If at least one does not support STARTTLS or one of the hosts doesn't connect
146 146
         // , return MAILDOMAIN_NO_STARTTLS (one which we can't connect to we also
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
         switch (\config\ConfAssistant::SMSSETTINGS['provider']) {
194 194
             case 'Nexmo':
195 195
                 // taken from https://docs.nexmo.com/messaging/sms-api
196
-                $url = 'https://rest.nexmo.com/sms/json?' . http_build_query(
196
+                $url = 'https://rest.nexmo.com/sms/json?'.http_build_query(
197 197
                                 [
198 198
                                     'api_key' => \config\ConfAssistant::SMSSETTINGS['username'],
199 199
                                     'api_secret' => \config\ConfAssistant::SMSSETTINGS['password'],
@@ -224,14 +224,14 @@  discard block
 block discarded – undo
224 224
                     $loggerInstance->debug(2, 'Problem with SMS invitation: no message was sent!');
225 225
                     return OutsideComm::SMS_NOTSENT;
226 226
                 }
227
-                $loggerInstance->debug(2, 'Total of ' . $messageCount . ' messages were attempted to send.');
227
+                $loggerInstance->debug(2, 'Total of '.$messageCount.' messages were attempted to send.');
228 228
 
229 229
                 $totalFailures = 0;
230 230
                 foreach ($decoded_response['messages'] as $message) {
231 231
                     if ($message['status'] == 0) {
232
-                        $loggerInstance->debug(2, $message['message-id'] . ": Success");
232
+                        $loggerInstance->debug(2, $message['message-id'].": Success");
233 233
                     } else {
234
-                        $loggerInstance->debug(2, $message['message-id'] . ": Failed (failure code = " . $message['status'] . ")");
234
+                        $loggerInstance->debug(2, $message['message-id'].": Failed (failure code = ".$message['status'].")");
235 235
                         $totalFailures++;
236 236
                     }
237 237
                 }
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
             $proto = "https://";
301 301
         }
302 302
         // then, send out the mail
303
-        $message = _("Hello,") . "\n\n" . wordwrap($introTexts[$introtext] . " " . $validity, 72) . "\n\n";
303
+        $message = _("Hello,")."\n\n".wordwrap($introTexts[$introtext]." ".$validity, 72)."\n\n";
304 304
         // default means we don't have a Reply-To.
305 305
         $replyToMessage = wordwrap(_("manually. Please do not reply to this mail; this is a send-only address."));
306 306
 
@@ -308,8 +308,8 @@  discard block
 block discarded – undo
308 308
             // see if we are supposed to add a custom message
309 309
             $customtext = $federation->getAttributes('fed:custominvite');
310 310
             if (count($customtext) > 0) {
311
-                $message .= wordwrap(sprintf(_("Additional message from your %s administrator:"), Entity::$nomenclature_fed), 72) . "\n---------------------------------" .
312
-                        wordwrap($customtext[0]['value'], 72) . "\n---------------------------------\n\n";
311
+                $message .= wordwrap(sprintf(_("Additional message from your %s administrator:"), Entity::$nomenclature_fed), 72)."\n---------------------------------".
312
+                        wordwrap($customtext[0]['value'], 72)."\n---------------------------------\n\n";
313 313
             }
314 314
             // and add Reply-To already now
315 315
             foreach ($federation->listFederationAdmins() as $fedadmin_id) {
@@ -325,19 +325,19 @@  discard block
 block discarded – undo
325 325
         }
326 326
         $productname = \config\Master::APPEARANCE['productname'];
327 327
         $consortium = \config\ConfAssistant::CONSORTIUM['display_name'];
328
-        $message .= wordwrap(sprintf(_("To enlist as an administrator for that %s, please click on the following link:"), Entity::$nomenclature_participant), 72) . "\n\n" .
329
-                $proto . $_SERVER['SERVER_NAME'] . \config\Master::PATHS['cat_base_url'] . "admin/action_enrollment.php?token=$newtoken\n\n" .
330
-                wordwrap(sprintf(_("If clicking the link doesn't work, you can also go to the %s Administrator Interface at"), $productname), 72) . "\n\n" .
331
-                $proto . $_SERVER['SERVER_NAME'] . \config\Master::PATHS['cat_base_url'] . "admin/\n\n" .
332
-                _("and enter the invitation token") . "\n\n" .
333
-                $newtoken . "\n\n$replyToMessage\n\n" .
334
-                wordwrap(_("Do NOT forward the mail before the token has expired - or the recipients may be able to consume the token on your behalf!"), 72) . "\n\n" .
335
-                wordwrap(sprintf(_("We wish you a lot of fun with the %s."), $productname), 72) . "\n\n" .
328
+        $message .= wordwrap(sprintf(_("To enlist as an administrator for that %s, please click on the following link:"), Entity::$nomenclature_participant), 72)."\n\n".
329
+                $proto.$_SERVER['SERVER_NAME'].\config\Master::PATHS['cat_base_url']."admin/action_enrollment.php?token=$newtoken\n\n".
330
+                wordwrap(sprintf(_("If clicking the link doesn't work, you can also go to the %s Administrator Interface at"), $productname), 72)."\n\n".
331
+                $proto.$_SERVER['SERVER_NAME'].\config\Master::PATHS['cat_base_url']."admin/\n\n".
332
+                _("and enter the invitation token")."\n\n".
333
+                $newtoken."\n\n$replyToMessage\n\n".
334
+                wordwrap(_("Do NOT forward the mail before the token has expired - or the recipients may be able to consume the token on your behalf!"), 72)."\n\n".
335
+                wordwrap(sprintf(_("We wish you a lot of fun with the %s."), $productname), 72)."\n\n".
336 336
                 sprintf(_("Sincerely,\n\nYour friendly folks from %s Operations"), $consortium);
337 337
 
338 338
 
339 339
 // who to whom?
340
-        $mail->FromName = \config\Master::APPEARANCE['productname'] . " Invitation System";
340
+        $mail->FromName = \config\Master::APPEARANCE['productname']." Invitation System";
341 341
 
342 342
         if (isset(\config\Master::APPEARANCE['invitation-bcc-mail']) && \config\Master::APPEARANCE['invitation-bcc-mail'] !== NULL) {
343 343
             $mail->addBCC(\config\Master::APPEARANCE['invitation-bcc-mail']);
Please login to merge, or discard this patch.
devices/ms/MsLanProfile.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
         $dom = new \DOMDocument('1.0', 'utf-8');
31 31
         $root = $dom->createElement($rootname);
32 32
         $dom->appendChild($root);
33
-        $ns = $dom->createAttributeNS( null, 'xmlns' );
33
+        $ns = $dom->createAttributeNS(null, 'xmlns');
34 34
         $ns->value = "http://www.microsoft.com/networking/LAN/profile/v1";
35 35
         $root->appendChild($ns);        
36 36
         \core\DeviceXMLmain::marshalObject($dom, $root, 'WLANprofile', $this->getLANprofile(), '', true);
Please login to merge, or discard this patch.
devices/ms/WindowsCommon.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 
103 103
         $this->loggerInstance->debug(5, "translateFile($source_name, $output_name)\n");
104 104
         ob_start();
105
-        $this->loggerInstance->debug(5, $this->module_path . '/Files/' . $this->device_id . '/' . $source_name . "\n");
105
+        $this->loggerInstance->debug(5, $this->module_path.'/Files/'.$this->device_id.'/'.$source_name."\n");
106 106
         $source = $this->findSourceFile($source_name);
107 107
 
108 108
         if ($source !== false) { // if there is no file found, don't attempt to include an uninitialised variable
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
229 229
         $out .= sprintf(_("%s installer will be in the form of an EXE file. It will configure %s on your device, by creating wireless network profiles.<p>When you click the download button, the installer will be saved by your browser. Copy it to the machine you want to configure and execute."), \config\ConfAssistant::CONSORTIUM['display_name'], \config\ConfAssistant::CONSORTIUM['display_name']);
230 230
         $out .= "<p>";
231 231
         if ($networksCount > $configCount) {
232
-            $out .= sprintf(ngettext("In addition to <strong>%s</strong> the installer will also configure access to:", "In addition to <strong>%s</strong> the installer will also configure access to the following networks:", $networksCount - $configCount), implode(', ', $configNetworkList)) . " ";
233
-            $out .= '<strong>' . join('</strong>, <strong>', array_diff($networkList, $configNetworkList)) . '</strong>';
232
+            $out .= sprintf(ngettext("In addition to <strong>%s</strong> the installer will also configure access to:", "In addition to <strong>%s</strong> the installer will also configure access to the following networks:", $networksCount - $configCount), implode(', ', $configNetworkList))." ";
233
+            $out .= '<strong>'.join('</strong>, <strong>', array_diff($networkList, $configNetworkList)).'</strong>';
234 234
             $out .= "<p>";
235 235
         }
236 236
 // TODO - change this below
@@ -339,14 +339,14 @@  discard block
 block discarded – undo
339 339
      */
340 340
     protected function signInstaller()
341 341
     {
342
-        $fileName = $this->installerBasename . '.exe';
342
+        $fileName = $this->installerBasename.'.exe';
343 343
         if (!$this->sign) {
344 344
             rename("installer.exe", $fileName);
345 345
             return $fileName;
346 346
         }
347 347
         $retval = 0;
348 348
         // are actually signing
349
-        $outputFromSigning = system($this->sign . " installer.exe '$fileName' > /dev/null", $retval);
349
+        $outputFromSigning = system($this->sign." installer.exe '$fileName' > /dev/null", $retval);
350 350
         $this->loggerInstance->debug(4, $retval, "Output from Windows signing:", "==\n");
351 351
         if ($retval !== 0 || $outputFromSigning === false) {
352 352
             $this->loggerInstance->debug(2, "Signing the WindowsCommon installer $fileName FAILED!\n");
@@ -364,12 +364,12 @@  discard block
 block discarded – undo
364 364
      */
365 365
     protected function compileNSIS()
366 366
     {
367
-        $makensis = \config\ConfAssistant::PATHS['makensis'] . " -INPUTCHARSET UTF8";
367
+        $makensis = \config\ConfAssistant::PATHS['makensis']." -INPUTCHARSET UTF8";
368 368
         $lcAll = getenv("LC_ALL");
369 369
         putenv("LC_ALL=en_US.UTF-8");
370
-        $command = $makensis . ' -V4 cat.NSI > nsis.log 2>&1';
370
+        $command = $makensis.' -V4 cat.NSI > nsis.log 2>&1';
371 371
         system($command);
372
-        putenv("LC_ALL=" . $lcAll);
372
+        putenv("LC_ALL=".$lcAll);
373 373
         $this->loggerInstance->debug(4, "compileNSIS:$command\n");
374 374
     }
375 375
 
@@ -386,10 +386,10 @@  discard block
 block discarded – undo
386 386
             'email' => 'SUPPORT',
387 387
             'url' => 'URL',
388 388
         ];
389
-        $s = "support_" . $type . "_substitute";
389
+        $s = "support_".$type."_substitute";
390 390
         $substitute = $this->translateString($this->$s);
391
-        $returnValue = !empty($attr['support:' . $type][0]) ? $attr['support:' . $type][0] : $substitute;
392
-        return '!define ' . $supportString[$type] . ' "' . $returnValue . '"' . "\n";
391
+        $returnValue = !empty($attr['support:'.$type][0]) ? $attr['support:'.$type][0] : $substitute;
392
+        return '!define '.$supportString[$type].' "'.$returnValue.'"'."\n";
393 393
     }
394 394
 
395 395
     /**
@@ -402,18 +402,18 @@  discard block
 block discarded – undo
402 402
     {
403 403
         $fcontents = '';
404 404
         if ($attr['internal:profile_count'][0] > 1) {
405
-            $fcontents .= "\n" . '!define USER_GROUP "' . $this->translateString(str_replace('"', '$\\"', $attr['profile:name'][0])) . '"
405
+            $fcontents .= "\n".'!define USER_GROUP "'.$this->translateString(str_replace('"', '$\\"', $attr['profile:name'][0])).'"
406 406
 ';
407 407
         }
408 408
         $fcontents .= '
409
-Caption "' . $this->translateString(sprintf(WindowsCommon::sprintNsis(_("%s installer for %s")), \config\ConfAssistant::CONSORTIUM['display_name'], $attr['general:instname'][0])) . '"
410
-!define APPLICATION "' . $this->translateString(sprintf(WindowsCommon::sprintNsis(_("%s installer for %s")), \config\ConfAssistant::CONSORTIUM['display_name'], $attr['general:instname'][0])) . '"
411
-!define VERSION "' . \core\CAT::VERSION_MAJOR . '.' . \core\CAT::VERSION_MINOR . '"
409
+Caption "' . $this->translateString(sprintf(WindowsCommon::sprintNsis(_("%s installer for %s")), \config\ConfAssistant::CONSORTIUM['display_name'], $attr['general:instname'][0])).'"
410
+!define APPLICATION "' . $this->translateString(sprintf(WindowsCommon::sprintNsis(_("%s installer for %s")), \config\ConfAssistant::CONSORTIUM['display_name'], $attr['general:instname'][0])).'"
411
+!define VERSION "' . \core\CAT::VERSION_MAJOR.'.'.\core\CAT::VERSION_MINOR.'"
412 412
 !define INSTALLER_NAME "installer.exe"
413
-!define LANG "' . $this->lang . '"
414
-!define LOCALE "' . preg_replace('/\..*$/', '', \config\Master::LANGUAGES[$this->languageInstance->getLang()]['locale']) . '"
413
+!define LANG "' . $this->lang.'"
414
+!define LOCALE "' . preg_replace('/\..*$/', '', \config\Master::LANGUAGES[$this->languageInstance->getLang()]['locale']).'"
415 415
 ;--------------------------------
416
-!define ORGANISATION "' . $this->translateString($attr['general:instname'][0]) . '"
416
+!define ORGANISATION "' . $this->translateString($attr['general:instname'][0]).'"
417 417
 ';
418 418
         $fcontents .= $this->getSupport($attr, 'email');
419 419
         $fcontents .= $this->getSupport($attr, 'url');
@@ -421,18 +421,18 @@  discard block
 block discarded – undo
421 421
             $fcontents .= '!define WIRED
422 422
         ';
423 423
         }
424
-        $fcontents .= '!define PROVIDERID "urn:UUID:' . $this->deviceUUID . '"
424
+        $fcontents .= '!define PROVIDERID "urn:UUID:'.$this->deviceUUID.'"
425 425
 ';
426 426
         if (!empty($attr['internal:realm'][0])) {
427
-            $fcontents .= '!define REALM "' . $attr['internal:realm'][0] . '"
427
+            $fcontents .= '!define REALM "'.$attr['internal:realm'][0].'"
428 428
 ';
429 429
         }
430 430
         if (!empty($attr['internal:hint_userinput_suffix'][0]) && $attr['internal:hint_userinput_suffix'][0] == 1) {
431
-            $fcontents .= '!define HINT_USER_INPUT "' . $attr['internal:hint_userinput_suffix'][0] . '"
431
+            $fcontents .= '!define HINT_USER_INPUT "'.$attr['internal:hint_userinput_suffix'][0].'"
432 432
 ';
433 433
         }
434 434
         if (!empty($attr['internal:verify_userinput_suffix'][0]) && $attr['internal:verify_userinput_suffix'][0] == 1) {
435
-            $fcontents .= '!define VERIFY_USER_REALM_INPUT "' . $attr['internal:verify_userinput_suffix'][0] . '"
435
+            $fcontents .= '!define VERIFY_USER_REALM_INPUT "'.$attr['internal:verify_userinput_suffix'][0].'"
436 436
 ';
437 437
         }
438 438
         $fcontents .= $this->msInfoFile($attr);
@@ -453,7 +453,7 @@  discard block
 block discarded – undo
453 453
             $out .= '!define EXTERNAL_INFO "';
454 454
 //  $this->loggerInstance->debug(4,"Info file type ".$attr['support:info_file'][0]['mime']."\n");
455 455
             if ($attr['internal:info_file'][0]['mime'] == 'rtf') {
456
-                $out = '!define LICENSE_FILE "' . $attr['internal:info_file'][0]['name'];
456
+                $out = '!define LICENSE_FILE "'.$attr['internal:info_file'][0]['name'];
457 457
             } elseif ($attr['internal:info_file'][0]['mime'] == 'txt') {
458 458
                 $infoFile = file_get_contents($attr['internal:info_file'][0]['name']);
459 459
                 if ($infoFile === false) {
@@ -466,7 +466,7 @@  discard block
 block discarded – undo
466 466
                     $out = '!define LICENSE_FILE " info_f.txt';
467 467
                 }
468 468
             } else {
469
-                $out = '!define EXTERNAL_INFO "' . $attr['internal:info_file'][0]['name'];
469
+                $out = '!define EXTERNAL_INFO "'.$attr['internal:info_file'][0]['name'];
470 470
             }
471 471
 
472 472
             $out .= "\"\n";
Please login to merge, or discard this patch.
web/skins/modern/index.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -50,11 +50,11 @@  discard block
 block discarded – undo
50 50
     print "vendorlogo ='';\n";
51 51
 }
52 52
 if ($operatingSystem) {
53
-    print "recognisedOS = '".$operatingSystem['device'] . "';\n";
54
-    print "recognisedOShs20 = '".$operatingSystem['hs20'] . "';\n";
53
+    print "recognisedOS = '".$operatingSystem['device']."';\n";
54
+    print "recognisedOShs20 = '".$operatingSystem['hs20']."';\n";
55 55
 }
56 56
 
57
-print 'downloadMessage = "'.$Gui->textTemplates->templates[\web\lib\user\DOWNLOAD_MESSAGE] . '";';
57
+print 'downloadMessage = "'.$Gui->textTemplates->templates[\web\lib\user\DOWNLOAD_MESSAGE].'";';
58 58
 //TODO modify this based on OS detection
59 59
 $userAgent = $_SERVER['HTTP_USER_AGENT'] ?? "";
60 60
 if (preg_match('/Android/', $userAgent)) {
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
     <?php echo $divs->divHeading($visibility); ?>
84 84
     <div id="main_page">
85 85
         <div id="loading_ico">
86
-          <?php echo _("Authenticating") . "..." ?><br><img src="<?php echo $Gui->skinObject->findResourceUrl("IMAGES", "icons/loading51.gif"); ?>" alt="Authenticating ..."/>
86
+          <?php echo _("Authenticating")."..." ?><br><img src="<?php echo $Gui->skinObject->findResourceUrl("IMAGES", "icons/loading51.gif"); ?>" alt="Authenticating ..."/>
87 87
         </div>
88 88
         <div id="info_overlay"> <!-- device info -->
89 89
             <div id="info_window"></div>
Please login to merge, or discard this patch.
devices/ms/DeviceW8W10.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -326,9 +326,9 @@
 block discarded – undo
326 326
         $otherTlsName = \core\common\Entity::getAttributeValue($this->attributes, 'eap-specific:tls_use_other_id', 0) === 'on' ? 'true' : 'false';
327 327
         if (isset(\core\common\Entity::getAttributeValue($this->attributes, 'device-specific:geantlink', $this->device_id)[0]) &&
328 328
              \core\common\Entity::getAttributeValue($this->attributes, 'device-specific:geantlink', $this->device_id)[0] === 'on') {
329
-             $this->useGeantLink = true;
329
+                $this->useGeantLink = true;
330 330
         } else { 
331
-             $this->useGeantLink = false;
331
+                $this->useGeantLink = false;
332 332
         }
333 333
         $eapConfig = $this->setEapObject();
334 334
         $eapConfig->setInnerType($this->selectedEap['INNER']);
Please login to merge, or discard this patch.