Passed
Push — develop ( fea5b5...4b87ca )
by Greg
14:06 queued 07:00
created
app/Http/Controllers/SetupController.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -82,8 +82,7 @@
 block discarded – undo
82 82
         MigrationService $migration_service,
83 83
         ServerCheckService $server_check_service,
84 84
         UserService $user_service
85
-    )
86
-    {
85
+    ) {
87 86
         $this->user_service         = $user_service;
88 87
         $this->migration_service    = $migration_service;
89 88
         $this->server_check_service = $server_check_service;
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/Http/Controllers/CalendarController.php 2 patches
Upper-Lower-Casing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -93,13 +93,13 @@  discard block
 block discarded – undo
93 93
             if (strlen($match[1]) > strlen($match[2])) {
94 94
                 $match[2] = substr($match[1], 0, strlen($match[1]) - strlen($match[2])) . $match[2];
95 95
             }
96
-            $ged_date = new Date("FROM {$cal} {$match[1]} TO {$cal} {$match[2]}");
96
+            $ged_date = new Date("from {$cal} {$match[1]} TO {$cal} {$match[2]}");
97 97
             $view     = 'year';
98 98
         } elseif (preg_match('/^(\d+)(\?+)$/', $year, $match)) {
99 99
             // advanced-year "decade/century wildcard"
100 100
             $y1       = $match[1] . str_replace('?', '0', $match[2]);
101 101
             $y2       = $match[1] . str_replace('?', '9', $match[2]);
102
-            $ged_date = new Date("FROM {$cal} {$y1} TO {$cal} {$y2}");
102
+            $ged_date = new Date("from {$cal} {$y1} TO {$cal} {$y2}");
103 103
             $view     = 'year';
104 104
         } else {
105 105
             if ($year < 0) {
@@ -210,13 +210,13 @@  discard block
 block discarded – undo
210 210
             if (strlen($match[1]) > strlen($match[2])) {
211 211
                 $match[2] = substr($match[1], 0, strlen($match[1]) - strlen($match[2])) . $match[2];
212 212
             }
213
-            $ged_date = new Date("FROM {$cal} {$match[1]} TO {$cal} {$match[2]}");
213
+            $ged_date = new Date("from {$cal} {$match[1]} TO {$cal} {$match[2]}");
214 214
             $view     = 'year';
215 215
         } elseif (preg_match('/^(\d+)(\?+)$/', $year, $match)) {
216 216
             // advanced-year "decade/century wildcard"
217 217
             $y1       = $match[1] . str_replace('?', '0', $match[2]);
218 218
             $y2       = $match[1] . str_replace('?', '9', $match[2]);
219
-            $ged_date = new Date("FROM {$cal} {$y1} TO {$cal} {$y2}");
219
+            $ged_date = new Date("from {$cal} {$y1} TO {$cal} {$y2}");
220 220
             $view     = 'year';
221 221
         } else {
222 222
             if ($year < 0) {
Please login to merge, or discard this 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/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.
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.