Passed
Push — develop ( cda1d6...9904e8 )
by Greg
06:21
created
app/Module/ListsMenuModule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
     public function getMenu(Tree $tree): ?Menu
88 88
     {
89 89
         $submenus = $this->module_service->findByComponent(ModuleListInterface::class, $tree, Auth::user())
90
-            ->map(function (ModuleListInterface $module) use ($tree): ?Menu {
90
+            ->map(function (ModuleListInterface $module) use ($tree) : ?Menu {
91 91
                 return $module->listMenu($tree);
92 92
             })
93 93
             ->filter()
Please login to merge, or discard this patch.
app/Statistics.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2616,7 +2616,7 @@
 block discarded – undo
2616 2616
         /** @var ModuleBlockInterface|null $module */
2617 2617
         $module = $this->module_service
2618 2618
             ->findByComponent(ModuleBlockInterface::class, $this->tree, Auth::user())
2619
-            ->filter(function (ModuleInterface $module) use ($block): bool {
2619
+            ->filter(function (ModuleInterface $module) use ($block) : bool {
2620 2620
                 return $module->name() === $block && $module->name() !== 'html';
2621 2621
             })
2622 2622
             ->first();
Please login to merge, or discard this patch.
app/Module/HourglassChartModule.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -291,7 +291,7 @@  discard block
 block discarded – undo
291 291
             if ($kcount == 0) {
292 292
                 echo "</td><td style='width:", app(ModuleThemeInterface::class)->parameter('chart-box-x'), "px'>";
293 293
             } else {
294
-                echo '<a href="#" title="' . I18N::translate('Children') . '" data-route="Descendants" data-xref="' .  e($pid) . '" data-spouses="' .  e($show_spouse) . '" data-tree="' .  e($individual->tree()->name()) . '">' . view('icons/arrow-left') . '</a>';
294
+                echo '<a href="#" title="' . I18N::translate('Children') . '" data-route="Descendants" data-xref="' . e($pid) . '" data-spouses="' . e($show_spouse) . '" data-tree="' . e($individual->tree()->name()) . '">' . view('icons/arrow-left') . '</a>';
295 295
 
296 296
                 //-- move the arrow up to line up with the correct box
297 297
                 if ($show_spouse) {
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
                 }
373 373
 
374 374
                 // filter out root person from children array so only siblings remain
375
-                $siblings       = $family->children()->filter(function (Individual $x) use ($individual): bool {
375
+                $siblings = $family->children()->filter(function (Individual $x) use ($individual): bool {
376 376
                     return $x !== $individual;
377 377
                 });
378 378
 
@@ -445,7 +445,7 @@  discard block
 block discarded – undo
445 445
                 echo '<td id="td_' . e($ARID) . '">';
446 446
 
447 447
                 if ($generation == $generations - 1 && $family->husband()->childFamilies()) {
448
-                    echo '<a href="#" title="' . I18N::translate('Parents') . '" data-route="Ancestors" data-xref="' .  e($ARID) . '" data-spouses="' .  e($show_spouse) . '" data-tree="' .  e($family->husband()->tree()->name()) . '">' . view('icons/arrow-right') . '</a>';
448
+                    echo '<a href="#" title="' . I18N::translate('Parents') . '" data-route="Ancestors" data-xref="' . e($ARID) . '" data-spouses="' . e($show_spouse) . '" data-tree="' . e($family->husband()->tree()->name()) . '">' . view('icons/arrow-right') . '</a>';
449 449
                 }
450 450
 
451 451
                 $this->printPersonPedigree($family->husband(), $generation + 1, $generations, $show_spouse);
@@ -476,7 +476,7 @@  discard block
 block discarded – undo
476 476
                 echo '<td id="td_' . e($ARID) . '">';
477 477
 
478 478
                 if ($generation == $generations - 1 && $family->wife()->childFamilies()) {
479
-                    echo '<a href="#" title="' . I18N::translate('Parents') . '" data-route="Ancestors" data-xref="' .  e($ARID) . '" data-spouses="' .  e($show_spouse) . '" data-tree="' .  e($family->wife()->tree()->name()) . '">' . view('icons/arrow-right') . '</a>';
479
+                    echo '<a href="#" title="' . I18N::translate('Parents') . '" data-route="Ancestors" data-xref="' . e($ARID) . '" data-spouses="' . e($show_spouse) . '" data-tree="' . e($family->wife()->tree()->name()) . '">' . view('icons/arrow-right') . '</a>';
480 480
                 }
481 481
 
482 482
                 $this->printPersonPedigree($family->wife(), $generation + 1, $generations, $show_spouse);
Please login to merge, or discard this patch.
app/Services/ModuleService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -555,7 +555,7 @@
 block discarded – undo
555 555
     public function findByName(string $module_name, bool $include_disabled = false): ?ModuleInterface
556 556
     {
557 557
         return $this->all($include_disabled)
558
-            ->filter(function (ModuleInterface $module) use ($module_name): bool {
558
+            ->filter(function (ModuleInterface $module) use ($module_name) : bool {
559 559
                 return $module->name() === $module_name;
560 560
             })
561 561
             ->first();
Please login to merge, or discard this patch.
app/View.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -252,7 +252,7 @@
 block discarded – undo
252 252
         [$namespace, $view_name] = explode(self::NAMESPACE_SEPARATOR, $view_name, 2);
253 253
 
254 254
         if ((self::$namespaces[$namespace] ?? null) === null) {
255
-            throw new RuntimeException('Namespace "' . e($namespace) .  '" not found.');
255
+            throw new RuntimeException('Namespace "' . e($namespace) . '" not found.');
256 256
         }
257 257
 
258 258
         $view_file = self::$namespaces[$namespace] . $view_name . self::TEMPLATE_EXTENSION;
Please login to merge, or discard this patch.
app/Module/PedigreeChartModule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -177,7 +177,7 @@
 block discarded – undo
177 177
 
178 178
         // Father’s ancestors link to the father’s pedigree
179 179
         // Mother’s ancestors link to the mother’s pedigree..
180
-        $links = $ancestors->map(function (?Individual $individual, $sosa) use ($ancestors, $orientation, $generations): string {
180
+        $links = $ancestors->map(function (?Individual $individual, $sosa) use ($ancestors, $orientation, $generations) : string {
181 181
             if ($individual instanceof Individual && $sosa >= 2 ** $generations / 2 && $individual->childFamilies()->isNotEmpty()) {
182 182
                 // The last row/column, and there are more generations.
183 183
                 if ($sosa >= 2 ** $generations * 3 / 4) {
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
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
      */
42 42
     public function find($user_id): ?User
43 43
     {
44
-        return app('cache.array')->rememberForever(__CLASS__ . $user_id, function () use ($user_id): ?User {
44
+        return app('cache.array')->rememberForever(__CLASS__ . $user_id, function () use ($user_id) : ?User {
45 45
             return DB::table('user')
46 46
                 ->where('user_id', '=', $user_id)
47 47
                 ->get()
Please login to merge, or discard this patch.
app/Module/InteractiveTree/TreeView.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@
 block discarded – undo
75 75
     public function getPersons(Tree $tree, string $request): string
76 76
     {
77 77
         $json_requests = explode(';', $request);
78
-        $r    = [];
78
+        $r = [];
79 79
         foreach ($json_requests as $json_request) {
80 80
             $firstLetter = substr($json_request, 0, 1);
81 81
             $json_request = substr($json_request, 1);
Please login to merge, or discard this patch.
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.