Passed
Push — develop ( 4b87ca...72589b )
by Greg
10:52 queued 04:37
created
app/Functions/FunctionsPrintLists.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -57,8 +57,8 @@  discard block
 block discarded – undo
57 57
                     $size = 75.0 + 125.0 * ($count - $minimum) / ($maximum - $minimum);
58 58
                 }
59 59
                 
60
-                $tag = ($module instanceof ModuleListInterface)?'a':'span';
61
-                $html .= '<'.$tag.' style="font-size:' . $size . '%"';
60
+                $tag = ($module instanceof ModuleListInterface) ? 'a' : 'span';
61
+                $html .= '<' . $tag . ' style="font-size:' . $size . '%"';
62 62
                 if ($module instanceof ModuleListInterface) {
63 63
                     $url = $module->listUrl($tree, ['surname' => $surn]);
64 64
                     $html .= ' href="' . e($url) . '"';
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
                 } else {
70 70
                     $html .= $spfxsurn;
71 71
                 }
72
-                $html .= '</'.$tag.'> ';
72
+                $html .= '</' . $tag . '> ';
73 73
             }
74 74
         }
75 75
 
@@ -104,12 +104,12 @@  discard block
 block discarded – undo
104 104
 
105 105
             // If all the surnames are just case variants, then merge them into one
106 106
             // Comment out this block if you want SMITH listed separately from Smith
107
-            $tag = ($module instanceof ModuleListInterface)?'a':'span';
108
-            $subhtml = '<'.$tag;
107
+            $tag = ($module instanceof ModuleListInterface) ? 'a' : 'span';
108
+            $subhtml = '<' . $tag;
109 109
             if ($url !== null) {
110 110
                 $subhtml .= ' href="' . e($url) . '"';
111 111
             }
112
-            $subhtml .= ' dir="auto">' . e(implode(I18N::$list_separator, array_keys($surns))) . '</'.$tag.'>';
112
+            $subhtml .= ' dir="auto">' . e(implode(I18N::$list_separator, array_keys($surns))) . '</' . $tag . '>';
113 113
             
114 114
             if ($totals) {
115 115
                 $subtotal = 0;
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
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
         // Ambiguous dates - don't override calendar escape
157 157
         if ($cal === '') {
158 158
             if (preg_match('/^(JAN|FEB|MAR|APR|MAY|JUN|JUL|AUG|SEP|OCT|NOV|DEC)$/', $m)) {
159
-                $cal =  GregorianDate::ESCAPE;
159
+                $cal = GregorianDate::ESCAPE;
160 160
             } elseif (preg_match('/^[345]\d\d\d$/', $y)) {
161 161
                 // Year 3000-5999
162 162
                 $cal = JewishDate::ESCAPE;
@@ -451,7 +451,7 @@  discard block
 block discarded – undo
451 451
     public function addYears(int $years, string $qualifier = ''): Date
452 452
     {
453 453
         $tmp               = clone $this;
454
-        $tmp->date1->year  += $years;
454
+        $tmp->date1->year += $years;
455 455
         $tmp->date1->month = 0;
456 456
         $tmp->date1->day   = 0;
457 457
         $tmp->date1->setJdFromYmd();
Please login to merge, or discard this patch.
app/Http/Controllers/CalendarController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -550,7 +550,7 @@
 block discarded – undo
550 550
         $html = '';
551 551
 
552 552
         foreach ($list as $id => $facts) {
553
-            $tmp  = GedcomRecord::getInstance($id, $tree);
553
+            $tmp = GedcomRecord::getInstance($id, $tree);
554 554
             $html .= $tag1 . '<a href="' . e($tmp->url()) . '">' . $tmp->fullName() . '</a> ';
555 555
             $html .= '<div class="indent">' . $facts . '</div>' . $tag2;
556 556
         }
Please login to merge, or discard this patch.
app/Module/FanChartModule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -344,7 +344,7 @@
 block discarded – undo
344 344
                     $html  .= '<div id="' . $person->xref() . '" class="fan_chart_menu">';
345 345
                     $html  .= '<div class="person_box"><div class="details1">';
346 346
                     $html .= '<div class="charts">';
347
-                    $html  .= '<a href="' . e($person->url()) . '" class="dropdown-item">' . $name. '</a>';
347
+                    $html  .= '<a href="' . e($person->url()) . '" class="dropdown-item">' . $name . '</a>';
348 348
                     foreach ($theme->individualBoxMenu($person) as $menu) {
349 349
                         $html .= '<a href="' . e($menu->getLink()) . '" class="dropdown-item p-1 ' . e($menu->getClass()) . '">' . $menu->getLabel() . '</a>';
350 350
                     }
Please login to merge, or discard this patch.
app/Functions/FunctionsEdit.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -575,7 +575,7 @@
 block discarded – undo
575 575
             $html .= view('components/select', ['id' => $id, 'name' => $name, 'selected' => $value, 'options' => GedcomCodeName::getValues()]);
576 576
         } elseif ($fact === 'TYPE' && $level === '3') {
577 577
             //-- Build the selector for the Media 'TYPE' Fact
578
-            $html          .= '<select name="text[]"><option selected value="" ></option>';
578
+            $html .= '<select name="text[]"><option selected value="" ></option>';
579 579
             $selectedValue = strtolower($value);
580 580
             if (!array_key_exists($selectedValue, GedcomTag::getFileFormTypes())) {
581 581
                 $html .= '<option selected value="' . e($value) . '" >' . e($value) . '</option>';
Please login to merge, or discard this patch.
app/Services/UserService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
      */
43 43
     public function find($user_id): ?User
44 44
     {
45
-        return app('cache.array')->rememberForever(__CLASS__ . $user_id, static function () use ($user_id): ?User {
45
+        return app('cache.array')->rememberForever(__CLASS__ . $user_id, static function () use ($user_id) : ?User {
46 46
             return DB::table('user')
47 47
                 ->where('user_id', '=', $user_id)
48 48
                 ->get()
Please login to merge, or discard this patch.
app/Module/RelationshipsChartModule.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -321,14 +321,14 @@  discard block
 block discarded – undo
321 321
                         case 'sis':
322 322
                         case 'sib':
323 323
                             $table[$x + 1][$y] = '<div style="background:url(' . e(asset('css/images/hline.png')) . ') repeat-x center;  width: 94px; text-align: center"><div class="hline-text" style="height: 32px;">' . Functions::getRelationshipNameFromPath($relationships[$n], Individual::getInstance($path[$n - 1], $tree), Individual::getInstance($path[$n + 1], $tree)) . '</div><div style="height: 32px;">' . view('icons/arrow-right') . '</div></div>';
324
-                            $x                 += 2;
324
+                            $x += 2;
325 325
                             break;
326 326
                         case 'son':
327 327
                         case 'dau':
328 328
                         case 'chi':
329 329
                             if ($n > 2 && preg_match('/fat|mot|par/', $relationships[$n - 2])) {
330 330
                                 $table[$x + 1][$y - 1] = '<div style="background:url(' . $diagonal2 . '); width: 64px; height: 64px; text-align: center;"><div style="height: 32px; text-align: end;">' . Functions::getRelationshipNameFromPath($relationships[$n], Individual::getInstance($path[$n - 1], $tree), Individual::getInstance($path[$n + 1], $tree)) . '</div><div style="height: 32px; text-align: start;">' . view('icons/arrow-down') . '</div></div>';
331
-                                $x                     += 2;
331
+                                $x += 2;
332 332
                             } else {
333 333
                                 $table[$x][$y - 1] = '<div style="background:url(' . e('"' . asset('css/images/vline.png') . '"') . ') repeat-y center; height: 64px; text-align: center;"><div class="vline-text" style="display: inline-block; width:50%; line-height: 64px;">' . Functions::getRelationshipNameFromPath($relationships[$n], Individual::getInstance($path[$n - 1], $tree), Individual::getInstance($path[$n + 1], $tree)) . '</div><div style="display: inline-block; width:50%; line-height: 64px;">' . view('icons/arrow-down') . '</div></div>';
334 334
                             }
@@ -339,7 +339,7 @@  discard block
 block discarded – undo
339 339
                         case 'par':
340 340
                             if ($n > 2 && preg_match('/son|dau|chi/', $relationships[$n - 2])) {
341 341
                                 $table[$x + 1][$y + 1] = '<div style="background:url(' . $diagonal1 . '); background-position: top right; width: 64px; height: 64px; text-align: center;"><div style="height: 32px; text-align: start;">' . Functions::getRelationshipNameFromPath($relationships[$n], Individual::getInstance($path[$n - 1], $tree), Individual::getInstance($path[$n + 1], $tree)) . '</div><div style="height: 32px; text-align: end;">' . view('icons/arrow-down') . '</div></div>';
342
-                                $x                     += 2;
342
+                                $x += 2;
343 343
                             } else {
344 344
                                 $table[$x][$y + 1] = '<div style="background:url(' . e('"' . asset('css/images/vline.png') . '"') . ') repeat-y center; height: 64px; text-align:center; "><div class="vline-text" style="display: inline-block; width: 50%; line-height: 64px;">' . Functions::getRelationshipNameFromPath($relationships[$n], Individual::getInstance($path[$n - 1], $tree), Individual::getInstance($path[$n + 1], $tree)) . '</div><div style="display: inline-block; width: 50%; line-height: 32px">' . view('icons/arrow-up') . '</div></div>';
345 345
                             }
@@ -550,17 +550,17 @@  discard block
 block discarded – undo
550 550
      */
551 551
     private function oldStyleRelationshipPath(Tree $tree, array $path): array
552 552
     {
553
-        $spouse_codes  = [
553
+        $spouse_codes = [
554 554
             'M' => 'hus',
555 555
             'F' => 'wif',
556 556
             'U' => 'spo',
557 557
         ];
558
-        $parent_codes  = [
558
+        $parent_codes = [
559 559
             'M' => 'fat',
560 560
             'F' => 'mot',
561 561
             'U' => 'par',
562 562
         ];
563
-        $child_codes   = [
563
+        $child_codes = [
564 564
             'M' => 'son',
565 565
             'F' => 'dau',
566 566
             'U' => 'chi',
Please login to merge, or discard this patch.
app/Http/Controllers/Admin/MediaController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
                     $create_form = '';
336 336
                     foreach ($media_trees as $media_tree => $media_directory) {
337 337
                         if (Str::startsWith($media_folder . $unused_file, $media_directory)) {
338
-                            $tmp         = substr($media_folder . $unused_file, strlen($media_directory));
338
+                            $tmp = substr($media_folder . $unused_file, strlen($media_directory));
339 339
                             $create_form .=
340 340
                                 '<p><a href="#" data-toggle="modal" data-target="#modal-create-media-from-file" data-file="' . e($tmp) . '" data-tree="' . e($media_tree) . '" onclick="document.getElementById(\'file\').value=this.dataset.file; document.getElementById(\'ged\').value=this.dataset.tree;">' . I18N::translate('Create') . '</a> — ' . e($media_tree) . '<p>';
341 341
                         }
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
 
462 462
             try {
463 463
                 $imgsize = getimagesize($full_path);
464
-                $html    .= '<dt>' . I18N::translate('Image dimensions') . '</dt>';
464
+                $html .= '<dt>' . I18N::translate('Image dimensions') . '</dt>';
465 465
                 /* I18N: image dimensions, width × height */
466 466
                 $html .= '<dd>' . I18N::translate('%1$s × %2$s pixels', I18N::number($imgsize['0']), I18N::number($imgsize['1'])) . '</dd>';
467 467
             } catch (Throwable $ex) {
Please login to merge, or discard this patch.
app/Http/Controllers/HomePageController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -908,7 +908,7 @@
 block discarded – undo
908 908
     private function filterActiveBlocks(Collection $blocks, Collection $active_blocks): Collection
909 909
     {
910 910
         return $blocks->map(static function (string $block_name) use ($active_blocks): ?ModuleBlockInterface {
911
-            return $active_blocks->filter(static function (ModuleInterface $block) use ($block_name): bool {
911
+            return $active_blocks->filter(static function (ModuleInterface $block) use ($block_name) : bool {
912 912
                 return $block->name() === $block_name;
913 913
             })->first();
914 914
         })
Please login to merge, or discard this patch.