Passed
Push — release_2_0 ( a1e5a0...377107 )
by Maja
06:22
created
web/diag/radius_tests.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(dirname(__FILE__))) . "/config/_config.php";
22
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
23 23
 
24 24
 $loggerInstance = new \core\common\Logging();
25 25
 $validator = new \web\lib\common\InputValidation();
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
  */
44 44
 function disp_name($eap) {
45 45
     $displayName = \core\common\EAP::eapDisplayName($eap);
46
-    return $displayName['OUTER'] . ( $displayName['INNER'] != '' ? '-' . $displayName['INNER'] : '');
46
+    return $displayName['OUTER'].($displayName['INNER'] != '' ? '-'.$displayName['INNER'] : '');
47 47
 }
48 48
 
49 49
 if (!isset($_REQUEST['test_type']) || !$_REQUEST['test_type']) {
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
     // contains port number; needs to be redacted for filter_var to work
83 83
     // in any case, it's a printable string, so filter it initially
84 84
     
85
-    $filteredHost = filter_input(INPUT_GET,'src', FILTER_SANITIZE_STRING) ?? filter_input(INPUT_POST,'src', FILTER_SANITIZE_STRING);
85
+    $filteredHost = filter_input(INPUT_GET, 'src', FILTER_SANITIZE_STRING) ?? filter_input(INPUT_POST, 'src', FILTER_SANITIZE_STRING);
86 86
     $hostonly1 = preg_replace('/:[0-9]*$/', "", $filteredHost);
87 87
     $hostonly2 = preg_replace('/^\[/', "", $hostonly1);
88 88
     $hostonly3 = preg_replace('/\]$/', "", $hostonly2);
@@ -158,10 +158,10 @@  discard block
 block discarded – undo
158 158
                             break;
159 159
                         case \core\common\Entity::L_REMARK:
160 160
                         case \core\common\Entity::L_WARN:
161
-                            $message = _("<strong>Test partially successful</strong>: authentication succeded.") . ' ' . $additional_message[$level];
161
+                            $message = _("<strong>Test partially successful</strong>: authentication succeded.").' '.$additional_message[$level];
162 162
                             break;
163 163
                         case \core\common\Entity::L_ERROR:
164
-                            $message = _("<strong>Test FAILED</strong>: authentication succeded.") . ' ' . $additional_message[$level];
164
+                            $message = _("<strong>Test FAILED</strong>: authentication succeded.").' '.$additional_message[$level];
165 165
                             break;
166 166
                     }
167 167
                     break;
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
             case \core\diag\RADIUSTests::RETVAL_CONVERSATION_REJECT:
213 213
                 $level = $returnarray['result'][$i]['level'];
214 214
                 if ($level > \core\common\Entity::L_OK) {
215
-                    $message = _("<strong>Test partially successful</strong>: a bidirectional RADIUS conversation with multiple round-trips was carried out, and ended in an Access-Reject as planned.") . ' ' . $additional_message[$level];
215
+                    $message = _("<strong>Test partially successful</strong>: a bidirectional RADIUS conversation with multiple round-trips was carried out, and ended in an Access-Reject as planned.").' '.$additional_message[$level];
216 216
                 } else {
217 217
                     $message = _("<strong>Test successful</strong>: a bidirectional RADIUS conversation with multiple round-trips was carried out, and ended in an Access-Reject as planned.");
218 218
                 }
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
                 $level = \core\common\Entity::L_ERROR;
236 236
                 break;
237 237
         }
238
-        $loggerInstance->debug(4, "SERVER=" . $returnarray['result'][$i]['server'] . "\n");
238
+        $loggerInstance->debug(4, "SERVER=".$returnarray['result'][$i]['server']."\n");
239 239
         $returnarray['result'][$i]['level'] = $level;
240 240
         $returnarray['result'][$i]['message'] = $message;
241 241
         break;
@@ -249,13 +249,13 @@  discard block
 block discarded – undo
249 249
         if (isset($rfc6614suite->TLS_CA_checks_result[$host])) {
250 250
             $returnarray['time_millisec'] = sprintf("%d", $rfc6614suite->TLS_CA_checks_result[$host]['time_millisec']);
251 251
             if (isset($rfc6614suite->TLS_CA_checks_result[$host]['cert_oddity']) && ($rfc6614suite->TLS_CA_checks_result[$host]['cert_oddity'] == \core\diag\RADIUSTests::CERTPROB_UNKNOWN_CA)) {
252
-                $returnarray['message'] = _("<strong>ERROR</strong>: the server presented a certificate which is from an unknown authority!") . ' (' . sprintf(_("elapsed time: %d"), $rfc6614suite->TLS_CA_checks_result[$host]['time_millisec']) . '&nbsp;ms)';
252
+                $returnarray['message'] = _("<strong>ERROR</strong>: the server presented a certificate which is from an unknown authority!").' ('.sprintf(_("elapsed time: %d"), $rfc6614suite->TLS_CA_checks_result[$host]['time_millisec']).'&nbsp;ms)';
253 253
                 $returnarray['level'] = \core\common\Entity::L_ERROR;
254 254
             } else {
255 255
                 $returnarray['message'] = $rfc6614suite->returnCodes[$rfc6614suite->TLS_CA_checks_result[$host]['status']]["message"];
256 256
                 $returnarray['level'] = \core\common\Entity::L_OK;
257 257
                 if ($rfc6614suite->TLS_CA_checks_result[$host]['status'] != \core\diag\RADIUSTests::RETVAL_CONNECTION_REFUSED) {
258
-                    $returnarray['message'] .= ' (' . sprintf(_("elapsed time: %d"), $rfc6614suite->TLS_CA_checks_result[$host]['time_millisec']) . '&nbsp;ms)';
258
+                    $returnarray['message'] .= ' ('.sprintf(_("elapsed time: %d"), $rfc6614suite->TLS_CA_checks_result[$host]['time_millisec']).'&nbsp;ms)';
259 259
                 } else {
260 260
                     $returnarray['level'] = \core\common\Entity::L_ERROR;
261 261
                 }
Please login to merge, or discard this patch.
web/diag/findRealm.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -34,9 +34,9 @@  discard block
 block discarded – undo
34 34
  * @param string $optSalt an optional salt value
35 35
  * @return boolean
36 36
  */
37
-function check_my_nonce($nonce, $optSalt='') {
37
+function check_my_nonce($nonce, $optSalt = '') {
38 38
     $remote = filter_input(INPUT_SERVER, 'REMOTE_ADDR');
39
-    $lasthour = date("G")-1<0 ? date('Ymd').'23' : date("YmdG")-1;
39
+    $lasthour = date("G") - 1 < 0 ? date('Ymd').'23' : date("YmdG") - 1;
40 40
     if (hash_hmac('sha256', session_id().$optSalt, date("YmdG").'1qaz2wsx3edc!QAZ@WSX#EDC'.$remote) == $nonce || 
41 41
         hash_hmac('sha256', session_id().$optSalt, $lasthour.'1qaz2wsx3edc!QAZ@WSX#EDC'.$remote) == $nonce) {
42 42
         return true;
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
         return false;
45 45
     }
46 46
 }
47
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
47
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
48 48
 
49 49
 // we are referring to $_SESSION later in the file
50 50
 CAT_session_start();
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
                 foreach ($details['admins'] as $admin) {
75 75
                     $admins[] = $admin['email'];
76 76
                 }
77
-                $details['admins'] = base64_encode(join(',',$admins));
77
+                $details['admins'] = base64_encode(join(',', $admins));
78 78
             } else {
79 79
                 $details['admins'] = '';
80 80
             }
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
                     $details = $cat->getExternalDBEntityDetails($realmOu);
109 109
                     if (!empty($details)) {
110 110
                         $returnArray['status'] = 1;
111
-                        $returnArray['realms'] = explode(',',$details['realmlist']);
111
+                        $returnArray['realms'] = explode(',', $details['realmlist']);
112 112
                     }   
113 113
                 }
114 114
             }
Please login to merge, or discard this patch.
web/diag/adminQuery.php 1 patch
Spacing   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 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
-require_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
22
+require_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
23 23
 $languageInstance = new \core\common\Language();
24 24
 $languageInstance->setTextDomain("diagnostics");
25 25
 
@@ -56,119 +56,119 @@  discard block
 block discarded – undo
56 56
     $select = "<div id='sp_abuse_problem'>
57 57
 <select style='margin-left: 0px;' id='select_sp_problem'>";
58 58
     foreach ($sp_problem as $pname => $pdesc) {
59
-        $select = $select . "<option value='$pname'>$pdesc</option>\n";
59
+        $select = $select."<option value='$pname'>$pdesc</option>\n";
60 60
     }
61
-    $select = $select . "</select></div>";
61
+    $select = $select."</select></div>";
62 62
     $res = "
63 63
 <table id='sp_questions'>
64 64
     <tr>
65
-        <td>" . _("Select your problem") . "</td>
65
+        <td>" . _("Select your problem")."</td>
66 66
         <td>$select</td>
67 67
     </tr>
68 68
     <tr>
69
-        <td>" . _("What is the realm of the IdP in question?") . "</td>
69
+        <td>"._("What is the realm of the IdP in question?")."</td>
70 70
         <td>
71 71
                 <input type='text' name='admin_realm' id='admin_realm' value=''>
72 72
                 <button id='realm_in_db_admin' accesskey='R' type='button'>" .
73
-            _("Check if this value is registered") .
73
+            _("Check if this value is registered").
74 74
             "</button>
75 75
         </td>
76 76
     </tr>
77 77
     <tr class='hidden_row'>
78
-        <td>" . _("What is the authentication timestamp of the user session in question?") . "</td>
78
+        <td>" . _("What is the authentication timestamp of the user session in question?")."</td>
79 79
         <td><input type='text' id='timestamp' name='timestamp'>
80 80
             <div id='datepicker'></div>
81 81
         </td>
82 82
     </tr>
83 83
     <tr class='hidden_row'>
84
-        <td>" . _("What is the MAC address of the user session in question?") . "</td>
84
+        <td>" . _("What is the MAC address of the user session in question?")."</td>
85 85
         <td><input type='text' id='mac' name='mac'></td>
86 86
     </tr>
87 87
     <tr class='hidden_row'>
88
-        <td>" . _("Additional comments") . "</td>
88
+        <td>" . _("Additional comments")."</td>
89 89
         <td><textarea id='freetext' name='freetext' cols='60' rows='5'></textarea></td>
90 90
     </tr>
91 91
     <tr class='hidden_row'>
92
-        <td>" . _("Please specify an email address on which the IdP can contact you") . "</td>
92
+        <td>" . _("Please specify an email address on which the IdP can contact you")."</td>
93 93
         <td><input type='text' id='email' name='email'></td>
94 94
     </tr>
95 95
     <tr class='hidden_row' id='send_query_to_idp'>
96
-        <td>" . _("Now you can send your query") . "</td>
97
-        <td><button type='submit' id='submit_idp_query' name='go'>" . _("Send") . "</button></td>
96
+        <td>" . _("Now you can send your query")."</td>
97
+        <td><button type='submit' id='submit_idp_query' name='go'>" . _("Send")."</button></td>
98 98
     </tr>
99 99
  </table>";
100
-    $res = $res . $javascript;
100
+    $res = $res.$javascript;
101 101
 }
102 102
 if ($queryType == 'idp') {
103 103
     $select = "<div id='idp_reported_problem' style='display:;'>
104 104
 <select style='margin-left:0px;' id='select_idp_problem'>";
105 105
     foreach ($idp_problem as $pname => $pdesc) {
106
-        $select = $select . "<option value='$pname'>$pdesc</option>\n";
106
+        $select = $select."<option value='$pname'>$pdesc</option>\n";
107 107
     }
108
-    $select = $select . "</select></div>";
108
+    $select = $select."</select></div>";
109 109
     $res = "
110 110
 <table id='idp_questions'>
111 111
     <tr>
112
-        <td>" . _("Select your problem") . "</td>
112
+        <td>" . _("Select your problem")."</td>
113 113
         <td>$select</td>
114 114
     </tr>
115 115
     <tr>
116
-        <td>" . _("Identify the SP by one of following means") . "</td>
116
+        <td>"._("Identify the SP by one of following means")."</td>
117 117
         <td></td>
118 118
     </tr>
119 119
     <tr id='by_opname'>
120
-        <td>" . _("SP Operator-Name attribute") . "</td>
120
+        <td>" . _("SP Operator-Name attribute")."</td>
121 121
         <td><input type='text' id='opname' name='opname' value=''></td>
122 122
     </tr>
123 123
     <tr id='spmanually'>
124
-        <td>" . _("Select the SP manually:") . "</td>
124
+        <td>" . _("Select the SP manually:")."</td>
125 125
         <td>
126 126
             <div id='select_asp_country'><a href='' id='asp_countries_list'>
127
-            <span id='opnameselect'>" . _("click to select country and organisation") . "</a></span>
127
+            <span id='opnameselect'>" . _("click to select country and organisation")."</a></span>
128 128
             </div>
129 129
             <div id='select_asp_area'></div>
130 130
         </td>
131 131
     </tr>
132 132
     <tr id='asp_desc' style='display: none;'>
133
-        <td>" . _("or") . ' ' . _("at least describe the SP location") . "</td>
133
+        <td>" . _("or").' '._("at least describe the SP location")."</td>
134 134
         <td><input type='text' id='asp_location' name='asp_location' value=''></td>
135 135
     </tr>
136 136
     <tr>
137
-        <td>" . _("What is the outer ID of the user session in question?") . "</td>
137
+        <td>" . _("What is the outer ID of the user session in question?")."</td>
138 138
         <td><input type='text' id='outer_id' name='outer_id' value=''></td>
139 139
     </tr>
140 140
     <tr>
141
-        <td>" . _("What is the authentication timestamp of the user session in question?") . "</td>
141
+        <td>" . _("What is the authentication timestamp of the user session in question?")."</td>
142 142
         <td>
143 143
             <input type='text' id='timestamp' name='timestamp'>
144 144
             <div id='datepicker'></div>
145 145
         </td>
146 146
     </tr>
147 147
     <tr>
148
-        <td>" . _("What is the MAC address of the user session in question?") . "</td>
148
+        <td>" . _("What is the MAC address of the user session in question?")."</td>
149 149
         <td><input type='text' id='mac' name='mac'></td>
150 150
     </tr>
151 151
     <tr>
152
-        <td>" . _("Additional comments about the problem") . "</td>
152
+        <td>" . _("Additional comments about the problem")."</td>
153 153
         <td><textarea id='freetext' name='freetext' cols='60' rows='5'></textarea></td>
154 154
     </tr>
155 155
     <tr>
156
-        <td>" . _("Do you have any contact details by which the user wishes to be contacted by the SP?") . "</td>
156
+        <td>" . _("Do you have any contact details by which the user wishes to be contacted by the SP?")."</td>
157 157
         <td><textarea id='c_details' name='c_details' cols='60' rows='5'></textarea></td>
158 158
     </tr>
159 159
     <tr>
160
-        <td>" . _("Please specify an email address on which the SP can contact you") . "</td>
160
+        <td>" . _("Please specify an email address on which the SP can contact you")."</td>
161 161
         <td><input type='text' id='email' name='email'></td>
162 162
     </tr>
163 163
     <tr class='hidden_row' id='send_query_to_sp'>
164
-        <td>" . _("Now you can send your query") . "</td>
165
-        <td><button type='submit' id='submit_sp_query' name='go'>" . _("Send") . "</button></td>
164
+        <td>" . _("Now you can send your query")."</td>
165
+        <td><button type='submit' id='submit_sp_query' name='go'>" . _("Send")."</button></td>
166 166
     </tr>
167 167
 </table>";
168
-    $res = $res . $javascript;
168
+    $res = $res.$javascript;
169 169
 }
170 170
 if ($queryType == 'idp_send' || $queryType == 'sp_send') {
171
-    include_once dirname(dirname(dirname(__FILE__))) . "/config/_config.php";
171
+    include_once dirname(dirname(dirname(__FILE__)))."/config/_config.php";
172 172
     $cat = new \core\CAT();
173 173
     $returnArray = array();
174 174
     if (count((array) $o) > 0) {
Please login to merge, or discard this patch.
web/404.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
  * @package UserGUI
27 27
  */
28 28
 error_reporting(E_ALL | E_STRICT);
29
-require dirname(dirname(__FILE__)) . "/config/_config.php";
29
+require dirname(dirname(__FILE__))."/config/_config.php";
30 30
 $langObject = new \core\common\Language();
31 31
 $langObject->setTextDomain("web_user");
32 32
 
@@ -38,25 +38,25 @@  discard block
 block discarded – undo
38 38
 <body style='background: #fff url(resources/images/bg_grey_tile.png) repeat-x;'>
39 39
     <div id="heading">
40 40
         <?php
41
-        print '<img src="'. dirname($_SERVER['SCRIPT_NAME']) .'/resources/images/consortium_logo.png" alt="Consortium Logo" style="float:right; padding-right:20px; padding-top:20px"/>';
42
-        print '<div id="motd">' . ( isset(CONFIG['APPEARANCE']['MOTD']) ? CONFIG['APPEARANCE']['MOTD'] : '&nbsp' ) . '</div>';
43
-        print '<h1 style="padding-bottom:0px; height:1em;">' . sprintf(_("Welcome to %s"), CONFIG['APPEARANCE']['productname']) . '</h1>
44
-<h2 style="padding-bottom:0px; height:0px; vertical-align:bottom;">' . CONFIG['APPEARANCE']['productname_long'] . '</h2>';
41
+        print '<img src="'.dirname($_SERVER['SCRIPT_NAME']).'/resources/images/consortium_logo.png" alt="Consortium Logo" style="float:right; padding-right:20px; padding-top:20px"/>';
42
+        print '<div id="motd">'.(isset(CONFIG['APPEARANCE']['MOTD']) ? CONFIG['APPEARANCE']['MOTD'] : '&nbsp').'</div>';
43
+        print '<h1 style="padding-bottom:0px; height:1em;">'.sprintf(_("Welcome to %s"), CONFIG['APPEARANCE']['productname']).'</h1>
44
+<h2 style="padding-bottom:0px; height:0px; vertical-align:bottom;">' . CONFIG['APPEARANCE']['productname_long'].'</h2>';
45 45
         echo '<table id="lang_select"><tr><td>';
46 46
         echo _("View this page in");
47 47
         ?>
48 48
         <?php
49 49
         foreach (CONFIG['LANGUAGES'] as $lang => $value) {
50
-            echo "<a href='javascript:changeLang(\"$lang\")'>" . $value['display'] . "</a> ";
50
+            echo "<a href='javascript:changeLang(\"$lang\")'>".$value['display']."</a> ";
51 51
         }
52
-        echo '</td><td style="text-align:right;padding-right:20px"><a href="' . dirname($_SERVER['SCRIPT_NAME']) . '?lang=' . $langObject->getLang() . '">' . _("Start page") . '</a></td></tr></table>';
52
+        echo '</td><td style="text-align:right;padding-right:20px"><a href="'.dirname($_SERVER['SCRIPT_NAME']).'?lang='.$langObject->getLang().'">'._("Start page").'</a></td></tr></table>';
53 53
         ?>
54 54
     </div> <!-- id="heading" -->
55 55
     <div id="main_body" style='padding:20px;'>
56
-        <h1><?php echo _("This is not the CAT you are looking for.");?></h1>
57
-        <p><?php echo _("Whatever you expected to see at this URL - it's not here. The only thing here is the number");?></p>
56
+        <h1><?php echo _("This is not the CAT you are looking for."); ?></h1>
57
+        <p><?php echo _("Whatever you expected to see at this URL - it's not here. The only thing here is the number"); ?></p>
58 58
         <h2>404</h2>
59
-        <p><?php echo sprintf(_("staring at you. Your mistake? Our error? Who knows! Maybe you should go back to the <a href='%s'>Start Page</a>."), dirname($_SERVER['SCRIPT_NAME']) . '?lang=' . $langObject->getLang());?></p>
59
+        <p><?php echo sprintf(_("staring at you. Your mistake? Our error? Who knows! Maybe you should go back to the <a href='%s'>Start Page</a>."), dirname($_SERVER['SCRIPT_NAME']).'?lang='.$langObject->getLang()); ?></p>
60 60
     </div> <!-- id="main_body" -->
61 61
 
62 62
         <?php echo $deco->footer();
Please login to merge, or discard this patch.
web/resources/css/cat.css.php 1 patch
Spacing   +17 added lines, -17 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 dirname(dirname(dirname(dirname(__FILE__)))) . "/config/_config.php";
22
+require dirname(dirname(dirname(dirname(__FILE__))))."/config/_config.php";
23 23
 $colour1 = CONFIG['APPEARANCE']['colour1'];
24 24
 $colour2 = CONFIG['APPEARANCE']['colour2'];
25 25
 header('Content-type: text/css; charset=utf-8');
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 }
30 30
 
31 31
 body {
32
-    background: <?php echo $colour1;?>;
32
+    background: <?php echo $colour1; ?>;
33 33
     color: #000000;
34 34
     font-family:Verdana, Arial, Helvetica, sans-serif;
35 35
     font-size:11px;
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 }
45 45
 
46 46
 button {
47
-    background: <?php echo $colour2;?>; 
47
+    background: <?php echo $colour2; ?>; 
48 48
     color: #FFFFFF; 
49 49
     min-height: 23px;
50 50
     border-left-style: outset; 
@@ -395,7 +395,7 @@  discard block
 block discarded – undo
395 395
 
396 396
 span.edu_cat {
397 397
     font-weight: bold;
398
-    color: <?php echo $colour2;?>;
398
+    color: <?php echo $colour2; ?>;
399 399
 }
400 400
 
401 401
 span.tooltip {
@@ -491,7 +491,7 @@  discard block
 block discarded – undo
491 491
     border-bottom-style:solid;
492 492
     border-top-width:5px; 
493 493
     border-bottom-width:5px; 
494
-    border-color: <?php echo $colour1;?>; 
494
+    border-color: <?php echo $colour1; ?>; 
495 495
     padding-left:30px;
496 496
 }
497 497
 
@@ -561,7 +561,7 @@  discard block
 block discarded – undo
561 561
 }
562 562
 
563 563
 #close_button {
564
-    background: <?php echo $colour2;?>; 
564
+    background: <?php echo $colour2; ?>; 
565 565
     color: #FFFFFF; 
566 566
     height: 23px;
567 567
     border-left-style: inset; 
@@ -636,7 +636,7 @@  discard block
 block discarded – undo
636 636
     width: 30em; 
637 637
     padding-left: 10px; 
638 638
     padding-right: 0px; 
639
-    background: <?php echo $colour2;?>; 
639
+    background: <?php echo $colour2; ?>; 
640 640
     color: white; 
641 641
     box-shadow: 10px 10px 5px #888888;
642 642
 }
@@ -683,7 +683,7 @@  discard block
 block discarded – undo
683 683
     padding-top: 4px; 
684 684
     padding-bottom: 12px; 
685 685
     padding-left: 30px; 
686
-    background: <?php echo $colour1;?>; 
686
+    background: <?php echo $colour1; ?>; 
687 687
     text-align: left; 
688 688
     text-shadow: 10px 10px 5px #888888;
689 689
 }
@@ -693,9 +693,9 @@  discard block
 block discarded – undo
693 693
     border-bottom-style:solid;
694 694
     border-top-width:5px;
695 695
     border-bottom-width:5px;
696
-    border-color: <?php echo $colour1;?>;
696
+    border-color: <?php echo $colour1; ?>;
697 697
     padding-left:30px;
698
-    color: <?php echo $colour2;?>;
698
+    color: <?php echo $colour2; ?>;
699 699
 }
700 700
 
701 701
 #heading h1 { 
@@ -714,7 +714,7 @@  discard block
 block discarded – undo
714 714
     text-align: justify;
715 715
     border-bottom-style:solid;
716 716
     border-bottom-width:5px;
717
-    border-color: <?php echo $colour1;?>;
717
+    border-color: <?php echo $colour1; ?>;
718 718
     font-size: 11px;
719 719
     font-weight: normal;
720 720
 }
@@ -761,7 +761,7 @@  discard block
 block discarded – undo
761 761
 
762 762
 #faq {
763 763
     padding: 20px;
764
-    color: <?php echo $colour2;?>;
764
+    color: <?php echo $colour2; ?>;
765 765
     background: #ffffff;
766 766
 }
767 767
 
@@ -813,7 +813,7 @@  discard block
 block discarded – undo
813 813
 
814 814
 #line1 {
815 815
     top:145px;
816
-    color: <?php echo $colour2;?>;
816
+    color: <?php echo $colour2; ?>;
817 817
     font-size:20px;
818 818
 }
819 819
 
@@ -860,7 +860,7 @@  discard block
 block discarded – undo
860 860
 
861 861
 #front_page_leftmenu {
862 862
     border-right:solid; 
863
-    border-color: <?php echo $colour1;?>; 
863
+    border-color: <?php echo $colour1; ?>; 
864 864
     border-width:5px; 
865 865
     min-height:400px; 
866 866
     padding-left: 10px; 
@@ -885,11 +885,11 @@  discard block
 block discarded – undo
885 885
 }
886 886
 
887 887
 a:link {
888
-    color:<?php echo $colour2;?>;
888
+    color:<?php echo $colour2; ?>;
889 889
 }
890 890
 
891 891
 a:visited {
892
-    color:<?php echo $colour2;?>;
892
+    color:<?php echo $colour2; ?>;
893 893
 }
894 894
 
895 895
 a:hover {
@@ -897,7 +897,7 @@  discard block
 block discarded – undo
897 897
 }
898 898
 
899 899
 a:active {
900
-    color:<?php echo $colour2;?>;
900
+    color:<?php echo $colour2; ?>;
901 901
 }
902 902
 
903 903
 .comment {
Please login to merge, or discard this patch.
web/lib/user/TextTemplates.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 const DOWNLOAD_REDIRECT_CONTINUE = 1054;
49 49
 const SB_GO_AWAY = 1060;
50 50
 const SB_FRONTPAGE_BIGDOWNLOADBUTTON = 1061;
51
-const SB_FRONTPAGE_ROLLER_CUSTOMBUILT= 1062;
51
+const SB_FRONTPAGE_ROLLER_CUSTOMBUILT = 1062;
52 52
 
53 53
 /**
54 54
  * some of the texts we write are consortium-specific.
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         $this->templates[WELCOME_ABOARD_BACKTODOWNLOADS] = _("Back to downloads");
87 87
         $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']);
88 88
         $this->templates[HEADING_TOPLEVEL_GREET] = sprintf(_("Welcome to %s"), CONFIG['APPEARANCE']['productname']);
89
-        $this->templates[HEADING_TOPLEVEL_PURPOSE] = sprintf(_("Connect your device to %s"),CONFIG_CONFASSISTANT['CONSORTIUM']['display_name']);
89
+        $this->templates[HEADING_TOPLEVEL_PURPOSE] = sprintf(_("Connect your device to %s"), CONFIG_CONFASSISTANT['CONSORTIUM']['display_name']);
90 90
         $this->templates[FRONTPAGE_ROLLER_EASY] = sprintf(_("%s installation made easy:"), CONFIG_CONFASSISTANT['CONSORTIUM']['display_name']);
91 91
         $this->templates[FRONTPAGE_ROLLER_CUSTOMBUILT] = _("Custom built for your organisation");
92 92
         $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/lib/user/Skinjob.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -95,17 +95,17 @@
 block discarded – undo
95 95
         }
96 96
 
97 97
         // does the file exist in the current skin's directory? Has precedence
98
-        if ($submodule !== '' && file_exists(__DIR__ . "/../../skins/" . $this->skin . "/" . $submodule . $path . $filename)) {
99
-            $extrapath = "/skins/" . $this->skin . "/" . $submodule;
98
+        if ($submodule !== '' && file_exists(__DIR__."/../../skins/".$this->skin."/".$submodule.$path.$filename)) {
99
+            $extrapath = "/skins/".$this->skin."/".$submodule;
100 100
         }
101
-        elseif (file_exists(__DIR__ . "/../../skins/" . $this->skin . $path . $filename)) {
102
-            $extrapath = "/skins/" . $this->skin;
103
-        } elseif (file_exists(__DIR__ . "/../../" . $path . $filename)) {
101
+        elseif (file_exists(__DIR__."/../../skins/".$this->skin.$path.$filename)) {
102
+            $extrapath = "/skins/".$this->skin;
103
+        } elseif (file_exists(__DIR__."/../../".$path.$filename)) {
104 104
             $extrapath = "";
105 105
         } else {
106 106
             return FALSE;
107 107
         }       
108
-        return htmlspecialchars(\core\CAT::getRootUrlPath() . $extrapath . $path . $filename, ENT_QUOTES);
108
+        return htmlspecialchars(\core\CAT::getRootUrlPath().$extrapath.$path.$filename, ENT_QUOTES);
109 109
     }
110 110
 
111 111
 }
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -97,8 +97,7 @@
 block discarded – undo
97 97
         // does the file exist in the current skin's directory? Has precedence
98 98
         if ($submodule !== '' && file_exists(__DIR__ . "/../../skins/" . $this->skin . "/" . $submodule . $path . $filename)) {
99 99
             $extrapath = "/skins/" . $this->skin . "/" . $submodule;
100
-        }
101
-        elseif (file_exists(__DIR__ . "/../../skins/" . $this->skin . $path . $filename)) {
100
+        } elseif (file_exists(__DIR__ . "/../../skins/" . $this->skin . $path . $filename)) {
102 101
             $extrapath = "/skins/" . $this->skin;
103 102
         } elseif (file_exists(__DIR__ . "/../../" . $path . $filename)) {
104 103
             $extrapath = "";
Please login to merge, or discard this patch.
web/lib/user/Gui.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -53,10 +53,10 @@
 block discarded – undo
53 53
         $ourlocale = $this->langObject->getLang();
54 54
         header("Content-Type:text/html;charset=utf-8");
55 55
         echo "<!DOCTYPE html>
56
-          <html xmlns='http://www.w3.org/1999/xhtml' lang='" . $ourlocale . "'>
57
-          <head lang='" . $ourlocale . "'>
56
+          <html xmlns='http://www.w3.org/1999/xhtml' lang='" . $ourlocale."'>
57
+          <head lang='" . $ourlocale."'>
58 58
           <meta http-equiv='Content-Type' content='text/html; charset=UTF-8'>";
59
-        echo "<title>" . htmlspecialchars($pagetitle) . "</title>";
59
+        echo "<title>".htmlspecialchars($pagetitle)."</title>";
60 60
         echo '<script type="text/javascript">ie_version = 0;</script>
61 61
 <!--[if IE]>
62 62
 <script type="text/javascript">ie_version=1;</script>
Please login to merge, or discard this patch.
web/lib/admin/PageDecoration.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -49,13 +49,13 @@  discard block
 block discarded – undo
49 49
         $retval = "<div class='sidebar'><p>";
50 50
 
51 51
         if ($advancedControls) {
52
-            $retval .= "<strong>" . _("You are:") . "</strong> ".$_SESSION['name']
52
+            $retval .= "<strong>"._("You are:")."</strong> ".$_SESSION['name']
53 53
             ."<br/>
54 54
               <br/>
55
-              <a href='" . \core\CAT::getRootUrlPath() . "/admin/overview_user.php'>" . _("Go to your Profile page") . "</a> 
56
-              <a href='" . \core\CAT::getRootUrlPath() . "/admin/inc/logout.php'>" . _("Logout") . "</a> ";
55
+              <a href='" . \core\CAT::getRootUrlPath()."/admin/overview_user.php'>"._("Go to your Profile page")."</a> 
56
+              <a href='" . \core\CAT::getRootUrlPath()."/admin/inc/logout.php'>"._("Logout")."</a> ";
57 57
         }
58
-        $retval .= "<a href='" . \core\CAT::getRootUrlPath() . "/'>" . _("Start page") . "</a>
58
+        $retval .= "<a href='".\core\CAT::getRootUrlPath()."/'>"._("Start page")."</a>
59 59
             </p>
60 60
         </div> <!-- sidebar -->";
61 61
         \core\common\Entity::outOfThePotatoes();
@@ -79,22 +79,22 @@  discard block
 block discarded – undo
79 79
                     <h1>$cap1</h1>
80 80
                 </div><!--header_captions-->
81 81
                 <div id='langselection' style='padding-top:20px; padding-left:10px;'>
82
-                    <form action='$place' method='GET' accept-charset='UTF-8'>" . _("View this page in") . "&nbsp;
82
+                    <form action='$place' method='GET' accept-charset='UTF-8'>"._("View this page in")."&nbsp;
83 83
                         <select id='lang' name='lang' onchange='this.form.submit()'>";
84 84
 
85 85
         foreach (CONFIG['LANGUAGES'] as $lang => $value) {
86
-            $retval .= "<option value='$lang' " . (strtoupper($language) == strtoupper($lang) ? "selected" : "" ) . " >" . $value['display'] . "</option> ";
86
+            $retval .= "<option value='$lang' ".(strtoupper($language) == strtoupper($lang) ? "selected" : "")." >".$value['display']."</option> ";
87 87
         }
88 88
         $retval .= "</select>";
89 89
 
90 90
         foreach ($_GET as $var => $value) {
91 91
             if ($var != "lang" && $value != "") {
92
-                $retval .= "<input type='hidden' name='" . htmlspecialchars($var) . "' value='" . htmlspecialchars($value) . "'>";
92
+                $retval .= "<input type='hidden' name='".htmlspecialchars($var)."' value='".htmlspecialchars($value)."'>";
93 93
             }
94 94
         }
95 95
         $retval .= "</form>
96 96
                 </div><!--langselection-->";
97
-        $logoUrl = \core\CAT::getRootUrlPath() . "/resources/images/consortium_logo.png";
97
+        $logoUrl = \core\CAT::getRootUrlPath()."/resources/images/consortium_logo.png";
98 98
         $retval .= "<div class='consortium_logo'>
99 99
                     <img id='test_locate' src='$logoUrl' alt='Consortium Logo'>
100 100
                 </div> <!-- consortium_logo -->
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 
188 188
         if (isset(CONFIG['APPEARANCE']['MOTD']) && CONFIG['APPEARANCE']['MOTD'] != "") {
189 189
             $retval .= "<div id='header_MOTD' style='display:inline-block; padding-left:20px;vertical-align:top;'>
190
-              <p class='MOTD'>" . CONFIG['APPEARANCE']['MOTD'] . "</p>
190
+              <p class='MOTD'>" . CONFIG['APPEARANCE']['MOTD']."</p>
191 191
               </div><!--header_MOTD-->";
192 192
         }
193 193
         $retval .= $this->sidebar($advancedControls);
@@ -213,9 +213,9 @@  discard block
 block discarded – undo
213 213
           <html xmlns='http://www.w3.org/1999/xhtml' lang='$ourlocale'>
214 214
           <head lang='$ourlocale'>
215 215
           <meta http-equiv='Content-Type' content='text/html; charset=UTF-8'>";
216
-        $cssUrl = \core\CAT::getRootUrlPath() . "/resources/css/cat.css.php";
216
+        $cssUrl = \core\CAT::getRootUrlPath()."/resources/css/cat.css.php";
217 217
         $retval .= "<link rel='stylesheet' type='text/css' href='$cssUrl' />";
218
-        $retval .= "<title>" . htmlspecialchars($pagetitle) . "</title>";
218
+        $retval .= "<title>".htmlspecialchars($pagetitle)."</title>";
219 219
         return $retval;
220 220
     }
221 221
 
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
             // we may need to jump up one dir if we are either in admin/ or accountstatus/
230 230
             // (accountstatus courtesy of my good mood. It's userspace not admin space so
231 231
             // it shouldn't be using this function any more.)
232
-            $logoBase = \core\CAT::getRootUrlPath() . "/resources/images";
232
+            $logoBase = \core\CAT::getRootUrlPath()."/resources/images";
233 233
             return "<span id='logos' style='position:fixed; left:50%;'><img src='$logoBase/dante.png' alt='DANTE' style='height:23px;width:47px'/>
234 234
               <img src='$logoBase/eu.png' alt='EU' style='height:23px;width:27px;border-width:0px;'/></span>
235 235
               <span id='eu_text' style='text-align:right;'><a href='http://ec.europa.eu/dgs/connect/index_en.htm' style='text-decoration:none; vertical-align:top;'>European Commission Communications Networks, Content and Technology</a></span>";
@@ -251,9 +251,9 @@  discard block
 block discarded – undo
251 251
             <table style='width:100%'>
252 252
                 <tr>
253 253
                     <td style='padding-left:20px; padding-right:20px; text-align:left; vertical-align:top;'>
254
-                        " . $cat->CAT_COPYRIGHT . "</td>";
254
+                        " . $cat->CAT_COPYRIGHT."</td>";
255 255
         if (!empty(CONFIG['APPEARANCE']['privacy_notice_url'])) {
256
-            $retval .= "<td><a href='".CONFIG['APPEARANCE']['privacy_notice_url']."'>" . sprintf(_("%s Privacy Notice"),CONFIG_CONFASSISTANT['CONSORTIUM']['display_name']) . "</a></td>";
256
+            $retval .= "<td><a href='".CONFIG['APPEARANCE']['privacy_notice_url']."'>".sprintf(_("%s Privacy Notice"), CONFIG_CONFASSISTANT['CONSORTIUM']['display_name'])."</a></td>";
257 257
         }
258 258
         $retval .= "            <td style='padding-left:80px; padding-right:20px; text-align:right; vertical-align:top;'>";
259 259
 
Please login to merge, or discard this patch.