Passed
Push — scrutinizer-patch-1 ( c7d376 )
by
unknown
13:36 queued 05:35
created
app/Http/Controllers/AdminLocationController.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -342,9 +342,11 @@
 block discarded – undo
342 342
             'fqpn',
343 343
         ];
344 344
 
345
-        if ($serverfile !== '') {  // first choice is file on server
345
+        if ($serverfile !== '') {
346
+// first choice is file on server
346 347
             $filename = WT_DATA_DIR . 'places/' . $serverfile;
347
-        } elseif ($_FILES['localfile']['error'] === UPLOAD_ERR_OK) { // 2nd choice is local file
348
+        } elseif ($_FILES['localfile']['error'] === UPLOAD_ERR_OK) {
349
+// 2nd choice is local file
348 350
             $filename = $_FILES['localfile']['tmp_name'];
349 351
         }
350 352
 
Please login to merge, or discard this patch.
app/Http/Controllers/PendingChangesController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
     {
99 99
         $url       = $request->get('url', '');
100 100
         $xref      = $request->get('xref', '');
101
-        $change_id = (int)$request->get('change_id');
101
+        $change_id = (int) $request->get('change_id');
102 102
 
103 103
         $changes = Database::prepare(
104 104
             "SELECT change_id, xref, old_gedcom, new_gedcom" .
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
     {
205 205
         $url       = $request->get('url', '');
206 206
         $xref      = $request->get('xref', '');
207
-        $change_id = (int)$request->get('change_id');
207
+        $change_id = (int) $request->get('change_id');
208 208
 
209 209
         // Reject a change, and subsequent changes to the same record
210 210
         Database::prepare(
Please login to merge, or discard this patch.
app/Date/JulianDate.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
         }
75 75
 
76 76
         if (preg_match('/^(\d+) B\.C\.$/', $year, $match)) {
77
-            return - (int) $match[1];
77
+            return -(int) $match[1];
78 78
         }
79 79
 
80 80
         return (int) $year;
Please login to merge, or discard this patch.
app/Date.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
         // Ambiguous dates - don't override calendar escape
163 163
         if ($cal == '') {
164 164
             if (preg_match('/^(JAN|FEB|MAR|APR|MAY|JUN|JUL|AUG|SEP|OCT|NOV|DEC)$/', $m)) {
165
-                $cal =  GregorianDate::ESCAPE;
165
+                $cal = GregorianDate::ESCAPE;
166 166
             } else {
167 167
                 if (preg_match('/^[345]\d\d\d$/', $y)) {
168 168
                     // Year 3000-5999
@@ -460,7 +460,7 @@  discard block
 block discarded – undo
460 460
     public function addYears(int $years, string $qualifier = ''): Date
461 461
     {
462 462
         $tmp               = clone $this;
463
-        $tmp->date1->year  += $years;
463
+        $tmp->date1->year += $years;
464 464
         $tmp->date1->month = 0;
465 465
         $tmp->date1->day   = 0;
466 466
         $tmp->date1->setJdFromYmd();
Please login to merge, or discard this patch.
app/Http/Controllers/ListController.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -216,7 +216,8 @@  discard block
 block discarded – undo
216 216
                     <li class="wt-initials-list-item d-flex">
217 217
                         <?php if ($count > 0) : ?>
218 218
                             <a href="<?= e(route($route, ['alpha' => $letter, 'ged' => $tree->name()])) ?>" class="wt-initial<?= $letter === $alpha ? ' active' : ''?> '" title="<?= I18N::number($count) ?>"><?= $this->surnameInitial((string) $letter) ?></a>
219
-                        <?php else : ?>
219
+                        <?php else {
220
+    : ?>
220 221
                             <span class="wt-initial text-muted"><?= $this->surnameInitial((string) $letter) ?></span>
221 222
 
222 223
                         <?php endif ?>
@@ -270,6 +271,7 @@  discard block
 block discarded – undo
270 271
 
271 272
             if ($show === 'indi' || $show === 'surn') {
272 273
                 $surns = $this->surnames($tree, $surname, $alpha, $show_marnm === 'yes', $families);
274
+}
273 275
                 if ($show === 'surn') {
274 276
                     // Show the surname list
275 277
                     switch ($tree->getPreference('SURNAME_LIST_STYLE')) {
Please login to merge, or discard this patch.
app/Functions/Functions.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
                                 return $new_path;
321 321
                             }
322 322
 
323
-                            $paths[]                     = $new_path;
323
+                            $paths[] = $new_path;
324 324
                             $visited[$spouse->xref()] = true;
325 325
                         }
326 326
                     }
@@ -334,7 +334,7 @@  discard block
 block discarded – undo
334 334
                                 return $new_path;
335 335
                             }
336 336
 
337
-                            $paths[]                    = $new_path;
337
+                            $paths[] = $new_path;
338 338
                             $visited[$child->xref()] = true;
339 339
                         }
340 340
                     }
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
                                 return $new_path;
354 354
                             }
355 355
 
356
-                            $paths[]                     = $new_path;
356
+                            $paths[] = $new_path;
357 357
                             $visited[$spouse->xref()] = true;
358 358
                         }
359 359
                     }
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
                                 return $new_path;
368 368
                             }
369 369
 
370
-                            $paths[]                    = $new_path;
370
+                            $paths[] = $new_path;
371 371
                             $visited[$child->xref()] = true;
372 372
                         }
373 373
                     }
Please login to merge, or discard this patch.
app/Module/OnThisDayModule.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
 
122 122
         // If we are only showing living individuals, then we don't need to search for DEAT events.
123 123
         if ($filter) {
124
-            $event_array  = array_diff($event_array, Gedcom::DEATH_EVENTS);
124
+            $event_array = array_diff($event_array, Gedcom::DEATH_EVENTS);
125 125
         }
126 126
 
127 127
         $events_filter = implode('|', $event_array);
Please login to merge, or discard this patch.
app/Census/CensusColumnNationality.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@
 block discarded – undo
52 52
         }
53 53
 
54 54
         // Did we emigrate or naturalise?
55
-        foreach ($individual->facts(['IMMI' ,'EMIG', 'NATU'], true) as $fact) {
55
+        foreach ($individual->facts(['IMMI', 'EMIG', 'NATU'], true) as $fact) {
56 56
             if (Date::compare($fact->date(), $this->date()) <= 0) {
57 57
                 $place = $fact->place()->getGedcomName();
58 58
             }
Please login to merge, or discard this patch.
app/Module/UpcomingAnniversariesModule.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
 
122 122
         // If we are only showing living individuals, then we don't need to search for DEAT events.
123 123
         if ($filter) {
124
-            $event_array  = array_diff($event_array, Gedcom::DEATH_EVENTS);
124
+            $event_array = array_diff($event_array, Gedcom::DEATH_EVENTS);
125 125
         }
126 126
 
127 127
         $events_filter = implode('|', $event_array);
Please login to merge, or discard this patch.