Passed
Push — 2.1 ( 004439...09d3ca )
by Greg
14:59 queued 08:59
created
app/Module/IndividualFactsTabModule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -126,7 +126,7 @@
 block discarded – undo
126 126
             ->flatten();
127 127
 
128 128
         // Don't show family meta-data tags
129
-        $exclude_facts  = new Collection(['FAM:CHAN', 'FAM:_UID']);
129
+        $exclude_facts = new Collection(['FAM:CHAN', 'FAM:_UID']);
130 130
         // Don't show tags that are shown in tabs or sidebars
131 131
         $exclude_facts = $exclude_facts->merge($sidebar_facts)->merge($tab_facts);
132 132
 
Please login to merge, or discard this patch.
app/Factories/SharedNoteFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@
 block discarded – undo
89 89
      *
90 90
      * @return SharedNote
91 91
      */
92
-    public function new(string $xref, string $gedcom, ?string $pending, Tree $tree): SharedNote
92
+    public function new(string $xref, string $gedcom, ?string $pending, Tree $tree) : SharedNote
93 93
     {
94 94
         return new SharedNote($xref, $gedcom, $pending, $tree);
95 95
     }
Please login to merge, or discard this patch.
app/Contracts/SharedNoteFactoryInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,5 +59,5 @@
 block discarded – undo
59 59
      *
60 60
      * @return SharedNote
61 61
      */
62
-    public function new(string $xref, string $gedcom, ?string $pending, Tree $tree): SharedNote;
62
+    public function new(string $xref, string $gedcom, ?string $pending, Tree $tree) : SharedNote;
63 63
 }
Please login to merge, or discard this patch.
app/Report/ReportParserGenerate.php 1 patch
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1090,7 +1090,7 @@
 block discarded – undo
1090 1090
                     } else {
1091 1091
                         $value = strip_tags(Registry::markdownFactory()->autolink($value, $this->tree), ['br']);
1092 1092
                     }
1093
-                    $value = strtr($value, [MarkdownFactory::BREAK => ' ']);
1093
+                    $value = strtr($value, [MarkdownFactory::break => ' ']);
1094 1094
                 }
1095 1095
 
1096 1096
                 if (!empty($attrs['truncate'])) {
Please login to merge, or discard this patch.
app/Services/LocalizationService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -94,7 +94,7 @@
 block discarded – undo
94 94
         'hu' => ['CS' => 'CS', 'DZS' => 'DZS', 'DZ' => 'DZ', 'GY' => 'GY', 'LY' => 'LY', 'NY' => 'NY', 'SZ' => 'SZ', 'TY' => 'TY', 'ZS' => 'ZS'],
95 95
         'nl' => ['IJ' => 'IJ'],
96 96
         'nn' => ['AA' => 'ƅ'],
97
-        'sk' => ['DZ' => 'DZ', 'DŽ' => 'DŽ','CH' => 'CH'],
97
+        'sk' => ['DZ' => 'DZ', 'DŽ' => 'DŽ', 'CH' => 'CH'],
98 98
     ];
99 99
 
100 100
     /**
Please login to merge, or discard this patch.
app/Report/RightToLeftSupport.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
                         $currentLen        = $endPos + 2;
200 200
                         $directive         = substr($workingText, 0, $currentLen);
201 201
                         $workingText       = substr($workingText, $currentLen);
202
-                        $result            .= self::$waitingText . $directive;
202
+                        $result .= self::$waitingText . $directive;
203 203
                         self::$waitingText = '';
204 204
                         break;
205 205
                     }
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
                             break;
309 309
                         }
310 310
                         self::$waitingText .= $currentLetter;
311
-                        $workingText       = substr($workingText, $currentLen);
311
+                        $workingText = substr($workingText, $currentLen);
312 312
                         if ($openParIndex !== false) {
313 313
                             // Opening parentheses always inherit the following directionality
314 314
                             while (true) {
@@ -318,13 +318,13 @@  discard block
 block discarded – undo
318 318
                                 if (str_starts_with($workingText, ' ')) {
319 319
                                     // Spaces following this left parenthesis inherit the following directionality too
320 320
                                     self::$waitingText .= ' ';
321
-                                    $workingText       = substr($workingText, 1);
321
+                                    $workingText = substr($workingText, 1);
322 322
                                     continue;
323 323
                                 }
324 324
                                 if (str_starts_with($workingText, ' ')) {
325 325
                                     // Spaces following this left parenthesis inherit the following directionality too
326 326
                                     self::$waitingText .= ' ';
327
-                                    $workingText       = substr($workingText, 6);
327
+                                    $workingText = substr($workingText, 6);
328 328
                                     continue;
329 329
                                 }
330 330
                                 break;
@@ -341,7 +341,7 @@  discard block
 block discarded – undo
341 341
                         // Exceptions to this rule will be handled later during final clean-up.
342 342
                         //
343 343
                         if (self::$currentState !== '') {
344
-                            $result            .= self::$waitingText;
344
+                            $result .= self::$waitingText;
345 345
                             self::$waitingText = '';
346 346
                         }
347 347
                         break 2; // double break because we're waiting for more information
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
                 }
471 471
                 if (substr($result . "\n", 0, self::LENGTH_START) !== self::START_LTR && substr($result . "\n", 0, self::LENGTH_START) !== self::START_RTL) {
472 472
                     $leadingText .= substr($result, 0, 1);
473
-                    $result      = substr($result, 1);
473
+                    $result = substr($result, 1);
474 474
                     continue;
475 475
                 }
476 476
                 $result = substr($result, 0, self::LENGTH_START) . $leadingText . substr($result, self::LENGTH_START);
@@ -612,11 +612,11 @@  discard block
 block discarded – undo
612 612
     private static function breakCurrentSpan(string &$result): void
613 613
     {
614 614
         // Interrupt the current span, insert that <br>, and then continue the current span
615
-        $result            .= self::$waitingText;
615
+        $result .= self::$waitingText;
616 616
         self::$waitingText = '';
617 617
 
618 618
         $breakString = '<' . self::$currentState . 'br>';
619
-        $result      .= $breakString;
619
+        $result .= $breakString;
620 620
     }
621 621
 
622 622
     /**
@@ -673,7 +673,7 @@  discard block
 block discarded – undo
673 673
                 break;
674 674
             } // No more numeric strings
675 675
 
676
-            $tempResult    .= substr($textSpan, 0, $posLRE + 3); // Copy everything preceding the numeric string
676
+            $tempResult .= substr($textSpan, 0, $posLRE + 3); // Copy everything preceding the numeric string
677 677
             $numericString = substr($textSpan, $posLRE + 3, $posPDF - $posLRE); // Separate the entire numeric string
678 678
             $textSpan      = substr($textSpan, $posPDF + 3);
679 679
             $posColon      = strpos($numericString, ':');
@@ -1140,7 +1140,7 @@  discard block
 block discarded – undo
1140 1140
 
1141 1141
             // We're done: finish the span
1142 1142
             $textSpan = self::starredName($textSpan, 'RTL'); // Wrap starred name in <u> and </u> tags
1143
-            $result   .= $textSpan . self::END_RTL;
1143
+            $result .= $textSpan . self::END_RTL;
1144 1144
         }
1145 1145
 
1146 1146
         if (self::$currentState !== 'LTR' && self::$currentState !== 'RTL') {
Please login to merge, or discard this patch.
app/Services/UpgradeService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -301,7 +301,7 @@
 block discarded – undo
301 301
     {
302 302
         $latest_version = $this->fetchLatestVersion(false);
303 303
 
304
-        [, , $url] = explode('|', $latest_version . '||');
304
+        [,, $url] = explode('|', $latest_version . '||');
305 305
 
306 306
         return $url;
307 307
     }
Please login to merge, or discard this patch.
app/Statistics/Repository/EventRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -232,7 +232,7 @@
 block discarded – undo
232 232
             ->orderBy('d_type')
233 233
             ->limit(1)
234 234
             ->get()
235
-            ->map(static fn (object $row): object => (object) [
235
+            ->map(static fn (object $row) : object => (object) [
236 236
                 'id'   => $row->id,
237 237
                 'year' => (int) $row->year,
238 238
                 'fact' => $row->fact,
Please login to merge, or discard this patch.
app/Module/IndividualListModule.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -290,7 +290,8 @@  discard block
 block discarded – undo
290 290
                     <li class="wt-initials-list-item d-flex">
291 291
                         <?php if ($count > 0) : ?>
292 292
                             <a href="<?= e($this->listUrl($tree, ['alpha' => $letter, 'show_marnm' => $show_marnm, 'tree' => $tree->name()])) ?>" class="wt-initial px-1<?= $letter === $alpha ? ' active' : '' ?> '" title="<?= I18N::number($count) ?>"><?= $this->displaySurnameInitial((string) $letter) ?></a>
293
-                        <?php else : ?>
293
+                        <?php else {
294
+    : ?>
294 295
                             <span class="wt-initial px-1 text-muted"><?= $this->displaySurnameInitial((string) $letter) ?></span>
295 296
 
296 297
                         <?php endif ?>
@@ -345,6 +346,7 @@  discard block
 block discarded – undo
345 346
                 switch ($alpha) {
346 347
                     case '@':
347 348
                         $filter = static fn(string $x): bool => $x === Individual::NOMEN_NESCIO;
349
+}
348 350
                         break;
349 351
                     case ',':
350 352
                         $filter = static fn(string $x): bool => $x === '';
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
         $surname_initials = $this->surnameInitials($surname_data);
217 217
 
218 218
         // We've requested a surname that doesn't currently exist.
219
-        if ($surname !== ''  && !array_key_exists($surname, $all_surns)) {
219
+        if ($surname !== '' && !array_key_exists($surname, $all_surns)) {
220 220
             $message = I18N::translate('There are no individuals with the surname ā€œ%sā€œ', e($surname));
221 221
             FlashMessages::addMessage($message);
222 222
 
@@ -604,7 +604,7 @@  discard block
 block discarded – undo
604 604
             ->groupBy([$this->binaryColumn('n_givn')]);
605 605
 
606 606
         foreach ($query->get() as $row) {
607
-            $initial            = I18N::strtoupper(I18N::language()->initialLetter($row->n_givn));
607
+            $initial = I18N::strtoupper(I18N::language()->initialLetter($row->n_givn));
608 608
             $initials[$initial] ??= 0;
609 609
             $initials[$initial] += (int) $row->count;
610 610
         }
Please login to merge, or discard this patch.