Passed
Pull Request — master (#322)
by Mirko
08:41
created
htdocs/lib2/html2text.class.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -534,7 +534,7 @@  discard block
 block discarded – undo
534 534
         if (substr($link, 0, 7) == 'http://' || substr($link, 0, 8) == 'https://' ||
535 535
             substr($link, 0, 7) == 'mailto:'
536 536
         ) {
537
-            $this->_link_count ++;
537
+            $this->_link_count++;
538 538
             $this->_link_list .= "[" . $this->_link_count . "] $link\n";
539 539
             $additional = ' [' . $this->_link_count . ']';
540 540
         } elseif (substr($link, 0, 11) == 'javascript:') {
@@ -542,7 +542,7 @@  discard block
 block discarded – undo
542 542
             $additional = '';
543 543
             // what about href="#anchor" ?
544 544
         } else {
545
-            $this->_link_count ++;
545
+            $this->_link_count++;
546 546
             $this->_link_list .= "[" . $this->_link_count . "] " . $this->url;
547 547
             if (substr($link, 0, 1) != '/') {
548 548
                 $this->_link_list .= '/';
Please login to merge, or discard this patch.
htdocs/lib2/util.inc.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -229,7 +229,7 @@
 block discarded – undo
229 229
 
230 230
     while ($nCurPos < mb_strlen($str)) {
231 231
         $nNextSep = mb_strlen($str);
232
-        for ($nSepPos = 0; $nSepPos < mb_strlen($sep); $nSepPos ++) {
232
+        for ($nSepPos = 0; $nSepPos < mb_strlen($sep); $nSepPos++) {
233 233
             $nThisPos = mb_strpos($str, mb_substr($sep, $nSepPos, 1), $nCurPos);
234 234
             if ($nThisPos !== false) {
235 235
                 if ($nNextSep > $nThisPos) {
Please login to merge, or discard this patch.
htdocs/map2.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
                        WHERE option_id=6 AND user_id='&1'", true, $login->userid))
68 68
 ) {
69 69
     $fullscreen = true;
70
-    $tpl->popup = true;        // disables page header and -frame
70
+    $tpl->popup = true; // disables page header and -frame
71 71
     $tpl->popupmargin = false;
72 72
 } else {
73 73
     $fullscreen = false;
@@ -456,7 +456,7 @@  discard block
 block discarded – undo
456 456
     $caches_found = 0;
457 457
     while ($r = sql_fetch_assoc($rs)) {
458 458
         echo '<cache name="' . xmlentities($r['name']) . '" wpoc="' . xmlentities($r['wp_oc']) . '" />' . "\n";
459
-        ++ $caches_found;
459
+        ++$caches_found;
460 460
     }
461 461
     sql_free_result($rs);
462 462
 
@@ -496,7 +496,7 @@  discard block
 block discarded – undo
496 496
         "SELECT `slave_id`
497 497
         FROM `map2_result`
498 498
         WHERE `result_id`='&1' AND DATE_ADD(`date_created`, INTERVAL '&2' SECOND)>NOW()",
499
-        - 2,
499
+        -2,
500 500
         $nResultId,
501 501
         $opt['map']['maxcacheage']
502 502
     );
Please login to merge, or discard this patch.
htdocs/log.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@
 block discarded – undo
121 121
     // get logtext editormode (from form or from userprofile)
122 122
     // 1 = text; 2 = HTML; 3 = tinyMCE
123 123
     if (isset($_POST['descMode'])) {
124
-        $descMode = $_POST['descMode'] + 0;  // Ocprop: 2
124
+        $descMode = $_POST['descMode'] + 0; // Ocprop: 2
125 125
         if (($descMode < 1) || ($descMode > 3)) {
126 126
             $descMode = 3;
127 127
         }
Please login to merge, or discard this patch.
htdocs/editcache.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
                 }
140 140
 
141 141
                 //here we read all used information from the form if submitted, otherwise from DB
142
-                $cache_name = trim(isset($_POST['name']) ? $_POST['name'] : $cache_record['name']);  // Ocprop
142
+                $cache_name = trim(isset($_POST['name']) ? $_POST['name'] : $cache_record['name']); // Ocprop
143 143
                 $cache_type = isset($_POST['type']) ? $_POST['type'] : $cache_record['type'];
144 144
                 if (!isset($_POST['size'])) {
145 145
                     if ($cache_type == 4 || $cache_type == 5) {
@@ -153,15 +153,15 @@  discard block
 block discarded – undo
153 153
                 $cache_hidden_day = isset($_POST['hidden_day']) ? $_POST['hidden_day'] : date(
154 154
                     'd',
155 155
                     strtotime($cache_record['date_hidden'])
156
-                );  // Ocprop
156
+                ); // Ocprop
157 157
                 $cache_hidden_month = isset($_POST['hidden_month']) ? $_POST['hidden_month'] : date(
158 158
                     'm',
159 159
                     strtotime($cache_record['date_hidden'])
160
-                );  // Ocprop
160
+                ); // Ocprop
161 161
                 $cache_hidden_year = isset($_POST['hidden_year']) ? $_POST['hidden_year'] : date(
162 162
                     'Y',
163 163
                     strtotime($cache_record['date_hidden'])
164
-                );  // Ocprop
164
+                ); // Ocprop
165 165
 
166 166
                 if (is_null($cache_record['date_activate'])) {
167 167
                     $cache_activate_day = isset($_POST['activate_day']) ? $_POST['activate_day'] : date('d');
@@ -187,11 +187,11 @@  discard block
 block discarded – undo
187 187
                     );
188 188
                 }
189 189
 
190
-                $cache_difficulty = isset($_POST['difficulty']) ? $_POST['difficulty'] : $cache_record['difficulty'];  // Ocprop
191
-                $cache_terrain = isset($_POST['terrain']) ? $_POST['terrain'] : $cache_record['terrain'];  // Ocprop
192
-                $cache_country = isset($_POST['country']) ? $_POST['country'] : $cache_record['country'];  // Ocprop
190
+                $cache_difficulty = isset($_POST['difficulty']) ? $_POST['difficulty'] : $cache_record['difficulty']; // Ocprop
191
+                $cache_terrain = isset($_POST['terrain']) ? $_POST['terrain'] : $cache_record['terrain']; // Ocprop
192
+                $cache_country = isset($_POST['country']) ? $_POST['country'] : $cache_record['country']; // Ocprop
193 193
                 $show_all_countries = isset($_POST['show_all_countries']) ? $_POST['show_all_countries'] : 0;
194
-                $status = isset($_POST['status']) ? $_POST['status'] : $cache_record['status'];  // Ocprop
194
+                $status = isset($_POST['status']) ? $_POST['status'] : $cache_record['status']; // Ocprop
195 195
                 $status_old = $cache_record['status'];
196 196
                 $search_time = isset($_POST['search_time']) ? $_POST['search_time'] : $cache_record['search_time'];
197 197
                 $way_length = isset($_POST['way_length']) ? $_POST['way_length'] : $cache_record['way_length'];
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
 
254 254
                 $log_pw = isset($_POST['log_pw']) ? mb_substr($_POST['log_pw'], 0, 20) : $cache_record['logpw'];
255 255
                 // fix #4356: gc waypoints are frequently copy&pasted with leading spaces
256
-                $wp_gc = isset($_POST['wp_gc']) ? strtoupper(trim($_POST['wp_gc'])) : $cache_record['wp_gc'];  // Ocprop
256
+                $wp_gc = isset($_POST['wp_gc']) ? strtoupper(trim($_POST['wp_gc'])) : $cache_record['wp_gc']; // Ocprop
257 257
                 $showlists = isset($_POST['showlists']) ? 1 : $cache_record['show_cachelists'] + 0;
258 258
                 $protect_old_coords = isset($_POST['protect_old_coords']) ? 1 : $cache_record['protect_old_coords'] + 0;
259 259
 
@@ -265,12 +265,12 @@  discard block
 block discarded – undo
265 265
 
266 266
                 if (isset($_POST['latNS'])) {
267 267
                     //get coords from post-form
268
-                    $coords_latNS = $_POST['latNS'];  // Ocprop
269
-                    $coords_lonEW = $_POST['lonEW'];  // Ocprop
270
-                    $coords_lat_h = $_POST['lat_h'];  // Ocprop
271
-                    $coords_lon_h = $_POST['lon_h'];  // Ocprop
272
-                    $coords_lat_min = $_POST['lat_min'];  // Ocprop
273
-                    $coords_lon_min = $_POST['lon_min'];  // Ocprop
268
+                    $coords_latNS = $_POST['latNS']; // Ocprop
269
+                    $coords_lonEW = $_POST['lonEW']; // Ocprop
270
+                    $coords_lat_h = $_POST['lat_h']; // Ocprop
271
+                    $coords_lon_h = $_POST['lon_h']; // Ocprop
272
+                    $coords_lat_min = $_POST['lat_min']; // Ocprop
273
+                    $coords_lon_min = $_POST['lon_min']; // Ocprop
274 274
                 } else {
275 275
                     //get coords from DB
276 276
                     $coords_lon = $cache_record['longitude'];
@@ -564,7 +564,7 @@  discard block
 block discarded – undo
564 564
                                     $logtype = 13;
565 565
                                     break;
566 566
                                 default:
567
-                                    $logtype = 0;  // ???
567
+                                    $logtype = 0; // ???
568 568
                             }
569 569
                             if ($logtype > 0) {
570 570
                                 sql(
@@ -585,7 +585,7 @@  discard block
 block discarded – undo
585 585
 
586 586
                         // update cache attributes
587 587
                         $attriblist = "999";
588
-                        for ($i = 0; $i < count($cache_attribs); $i ++) {
588
+                        for ($i = 0; $i < count($cache_attribs); $i++) {
589 589
                             if ($cache_attribs[$i] + 0 > 0) {
590 590
                                 sql(
591 591
                                     "INSERT IGNORE INTO `caches_attributes` (`cache_id`, `attrib_id`) VALUES('&1', '&2')",
@@ -719,7 +719,7 @@  discard block
 block discarded – undo
719 719
                         $line = mb_ereg_replace('{name}', escape_javascript($record['name']), $line);
720 720
                         $line = mb_ereg_replace('{color}', $rAttrGroup['color'], $line);
721 721
                         $group_line .= $line;
722
-                        $nLineAttrCount ++;
722
+                        $nLineAttrCount++;
723 723
 
724 724
                         $line = $cache_attrib_js;
725 725
                         $line = mb_ereg_replace('{id}', $record['id'], $line);
@@ -776,7 +776,7 @@  discard block
 block discarded – undo
776 776
 
777 777
                 //difficulty
778 778
                 $difficulty_options = '';
779
-                for ($i = 2; $i <= 10; $i ++) {
779
+                for ($i = 2; $i <= 10; $i++) {
780 780
                     if ($cache_difficulty == $i) {
781 781
                         $difficulty_options .= '<option value="' . $i . '" selected="selected">' . $i / 2 . '</option>';
782 782
                     } else {
@@ -788,7 +788,7 @@  discard block
 block discarded – undo
788 788
 
789 789
                 //build terrain options
790 790
                 $terrain_options = '';
791
-                for ($i = 2; $i <= 10; $i ++) {
791
+                for ($i = 2; $i <= 10; $i++) {
792 792
                     if ($cache_terrain == $i) {
793 793
                         $terrain_options .= '<option value="' . $i . '" selected="selected">' . $i / 2 . '</option>';
794 794
                     } else {
@@ -955,7 +955,7 @@  discard block
 block discarded – undo
955 955
                     $tmp = mb_ereg_replace('{publish_notnow_checked}', ($publish == 'notnow') ? 'checked' : '', $tmp);
956 956
 
957 957
                     $activation_hours = '';
958
-                    for ($i = 0; $i <= 23; $i ++) {
958
+                    for ($i = 0; $i <= 23; $i++) {
959 959
                         if ($cache_activate_hour == $i) {
960 960
                             $activation_hours .= '<option value="' . $i . '" selected="selected">' . $i . '</option>';
961 961
                         } else {
@@ -983,7 +983,7 @@  discard block
 block discarded – undo
983 983
                         $cache_id
984 984
                     );
985 985
 
986
-                    for ($i = 0; $i < mysql_num_rows($rspictures); $i ++) {
986
+                    for ($i = 0; $i < mysql_num_rows($rspictures); $i++) {
987 987
                         $tmpline = ($i == 0 ? $pictureline0 : $pictureline);
988 988
                         $pic_record = sql_fetch_array($rspictures);
989 989
 
@@ -1066,7 +1066,7 @@  discard block
 block discarded – undo
1066 1066
                 tpl_set_var('showlists_checked', $showlists ? 'checked="checked"' : '');
1067 1067
                 tpl_set_var('protectcoords_checked', $protect_old_coords ? 'checked="checked"' : '');
1068 1068
 
1069
-                tpl_set_var('reset', $reset);  // obsolete
1069
+                tpl_set_var('reset', $reset); // obsolete
1070 1070
                 tpl_set_var('submit', $submit);
1071 1071
             } else {
1072 1072
                 //TODO: not the owner
Please login to merge, or discard this patch.
htdocs/activation.php 1 patch
Spacing   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,10 +14,8 @@
 block discarded – undo
14 14
 // We use short param codes 'u' and 'c' to generate short-enough activation
15 15
 // url that will not be wrapped in plain-text emails.
16 16
 
17
-$code = isset($_REQUEST['code']) ? trim($_REQUEST['code']) :
18
-    (isset($_REQUEST['c']) ? trim($_REQUEST['c']) : '');
19
-$email = isset($_REQUEST['email']) ? trim($_REQUEST['email']) :
20
-    (isset($_REQUEST['e']) ? trim($_REQUEST['e']) : '');
17
+$code = isset($_REQUEST['code']) ? trim($_REQUEST['code']) : (isset($_REQUEST['c']) ? trim($_REQUEST['c']) : '');
18
+$email = isset($_REQUEST['email']) ? trim($_REQUEST['email']) : (isset($_REQUEST['e']) ? trim($_REQUEST['e']) : '');
21 19
 
22 20
 $tpl->assign('errorEMail', false);
23 21
 $tpl->assign('errorCode', false);
Please login to merge, or discard this patch.
htdocs/lang/de/ocstyle/rating.inc.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
 
18 18
  ****************************************************************************/
19 19
 
20
-$rating_tpl =    '<tr>
20
+$rating_tpl = '<tr>
21 21
                 <td valign="top">' . t('Recommendations:') . '</td>
22 22
                 <td valign="top">
23 23
                     {rating_msg}
Please login to merge, or discard this patch.
htdocs/lang/de/ocstyle/lib/menu.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
     /* selmenuitem contains the selected (bold) menu item */
286 286
     global $mnu_selmenuitem;
287 287
 
288
-    for ($i = 0, $ret = - 1; ($i < count($menustructure)) && ($ret == - 1); $i ++) {
288
+    for ($i = 0, $ret = - 1; ($i < count($menustructure)) && ($ret == - 1); $i++) {
289 289
         if ($menustructure[$i]['siteid'] == $pageid) {
290 290
             $mnu_selmenuitem = $menustructure[$i];
291 291
 
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
313 313
     global $menu;
314 314
     $c = 0;
315 315
     $max = count($menu);
316
-    for ($i = 0; $i < $max; $i ++) {
316
+    for ($i = 0; $i < $max; $i++) {
317 317
         if ($menu[$i]['visible'] == true) {
318 318
             $sTarget = isset($menu[$i]['target']) ? $menu[$i]['target'] : '';
319 319
             $sItem = '<a href="' . $menu[$i]['filename'] . '" ' . $sTarget . '>' . htmlspecialchars(t($menu[$i]['menustring']), ENT_COMPAT, 'UTF-8') . '</a>';
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
 
329 329
             echo $sItem . "\n";
330 330
 
331
-            $c ++;
331
+            $c++;
332 332
         }
333 333
     }
334 334
 }
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
 
349 349
     if (!$bHasSubmenu) {
350 350
         /* prüfen, ob ein Submenü vorhanden ist */
351
-        for ($i = 0, $bSubmenu = false; ($i < count($menustructure)) && ($bSubmenu == false); $i ++) {
351
+        for ($i = 0, $bSubmenu = false; ($i < count($menustructure)) && ($bSubmenu == false); $i++) {
352 352
             if (isset($menustructure[$i]['submenu'])) {
353 353
                 $bSubmenu = true;
354 354
             }
@@ -365,7 +365,7 @@  discard block
 block discarded – undo
365 365
         }
366 366
     }
367 367
     $maxMenuStructure = count($menustructure);
368
-    for ($i = 0; $i < $maxMenuStructure; $i ++) {
368
+    for ($i = 0; $i < $maxMenuStructure; $i++) {
369 369
         if ($menustructure[$i]['visible'] == true) {
370 370
             if ($menustructure[$i]['siteid'] == $pageid) {
371 371
                 echo '<li class="' . $cssclass . ' group_active"><a href="' . $menustructure[$i]['filename'] . '">' . htmlspecialchars(t($menustructure[$i]['menustring']), ENT_COMPAT, 'UTF-8') . '</a></li>' . "\n";
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
  */
398 398
 function mnu_IsMenuParentOf($parentmenuitems, $menuitemid)
399 399
 {
400
-    for ($i = 0; $i < count($parentmenuitems); $i ++) {
400
+    for ($i = 0; $i < count($parentmenuitems); $i++) {
401 401
         if ($parentmenuitems[$i]['siteid'] == $menuitemid) {
402 402
             return true;
403 403
         }
@@ -439,7 +439,7 @@  discard block
 block discarded – undo
439 439
 function mnu_prv_EchoBreadCrumbSubItem($pageid, $menustructure)
440 440
 {
441 441
     $maxMenuStructure = count($menustructure);
442
-    for ($i = 0; $i < $maxMenuStructure; $i ++) {
442
+    for ($i = 0; $i < $maxMenuStructure; $i++) {
443 443
         if ($menustructure[$i]['siteid'] == $pageid) {
444 444
             echo '&nbsp;&gt;&nbsp;' . htmlspecialchars(t($menustructure[$i]['menustring']), ENT_COMPAT, 'UTF-8');
445 445
 
Please login to merge, or discard this patch.
htdocs/lang/de/expressions.inc.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
 
22 22
 //common vars
23 23
 $dateformat = $opt['locale'][$locale]['format']['date'];
24
-$reset = t('Reset');  // obsolete
24
+$reset = t('Reset'); // obsolete
25 25
 $yes = t('Yes');
26 26
 $no = t('No');
27 27
 
Please login to merge, or discard this patch.