Passed
Push — analysis-zE454y ( adf87c )
by Greg
13:48 queued 04:56
created
app/Statistics/Google/ChartMortality.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
                 $tot_l
58 58
             ];
59 59
 
60
-            $data[] =  [
60
+            $data[] = [
61 61
                 I18N::translate('Dead'),
62 62
                 $tot_d
63 63
             ];
Please login to merge, or discard this patch.
app/Statistics/Google/ChartMarriageAge.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -102,9 +102,9 @@
 block discarded – undo
102 102
             ->groupBy(['century', 'sex']);
103 103
 
104 104
         return $male->unionAll($female)
105
-           ->orderBy('century')
106
-           ->get()
107
-           ->all();
105
+            ->orderBy('century')
106
+            ->get()
107
+            ->all();
108 108
     }
109 109
 
110 110
     /**
Please login to merge, or discard this patch.
app/Module/StatisticsChartModule.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -962,7 +962,7 @@  discard block
 block discarded – undo
962 962
 
963 963
         $data = [
964 964
             array_merge(
965
-                [ I18N::translate('Century') ],
965
+                [I18N::translate('Century')],
966 966
                 array_values($z_axis)
967 967
             )
968 968
         ];
@@ -979,7 +979,7 @@  discard block
 block discarded – undo
979 979
 
980 980
         foreach ($intermediate as $key => $values) {
981 981
             $data[] = array_merge(
982
-                [ $x_axis[$key] ],
982
+                [$x_axis[$key]],
983 983
                 $values
984 984
             );
985 985
         }
Please login to merge, or discard this patch.
app/Functions/FunctionsPrint.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
                     if ($note->canShow()) {
146 146
                         $noterec = $note->gedcom();
147 147
                         $nt      = preg_match("/0 @$nmatch[1]@ NOTE (.*)/", $noterec, $n1match);
148
-                        $data    .= self::printNoteRecord($tree, ($nt > 0) ? $n1match[1] : '', 1, $noterec);
148
+                        $data .= self::printNoteRecord($tree, ($nt > 0) ? $n1match[1] : '', 1, $noterec);
149 149
                     }
150 150
                 } else {
151 151
                     $data = '<div class="fact_NOTE"><span class="label">' . I18N::translate('Note') . '</span>: <span class="field error">' . $nmatch[1] . '</span></div>';
@@ -362,20 +362,20 @@  discard block
 block discarded – undo
362 362
                 if (preg_match_all('/\n3 (?:_HEB|ROMN) (.+)/', $placerec, $matches)) {
363 363
                     foreach ($matches[1] as $match) {
364 364
                         $wt_place = new Place($match, $tree);
365
-                        $html     .= ' - ' . $wt_place->fullName();
365
+                        $html .= ' - ' . $wt_place->fullName();
366 366
                     }
367 367
                 }
368 368
                 $map_lati = '';
369 369
                 $cts      = preg_match('/\d LATI (.*)/', $placerec, $match);
370 370
                 if ($cts > 0) {
371 371
                     $map_lati = $match[1];
372
-                    $html     .= '<br><span class="label">' . I18N::translate('Latitude') . ': </span>' . $map_lati;
372
+                    $html .= '<br><span class="label">' . I18N::translate('Latitude') . ': </span>' . $map_lati;
373 373
                 }
374 374
                 $map_long = '';
375 375
                 $cts      = preg_match('/\d LONG (.*)/', $placerec, $match);
376 376
                 if ($cts > 0) {
377 377
                     $map_long = $match[1];
378
-                    $html     .= ' <span class="label">' . I18N::translate('Longitude') . ': </span>' . $map_long;
378
+                    $html .= ' <span class="label">' . I18N::translate('Longitude') . ': </span>' . $map_long;
379 379
                 }
380 380
                 if ($map_lati && $map_long) {
381 381
                     $map_lati = trim(strtr($map_lati, 'NSEW,�', ' - -. ')); // S5,6789 ==> -5.6789
Please login to merge, or discard this patch.
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -322,10 +322,10 @@
 block discarded – undo
322 322
         }
323 323
         // print gedcom ages
324 324
         foreach ([
325
-                     I18N::translate('Age')     => $fact_age,
326
-                     I18N::translate('Husband') => $husb_age,
327
-                     I18N::translate('Wife')    => $wife_age,
328
-                 ] as $label => $age) {
325
+                        I18N::translate('Age')     => $fact_age,
326
+                        I18N::translate('Husband') => $husb_age,
327
+                        I18N::translate('Wife')    => $wife_age,
328
+                    ] as $label => $age) {
329 329
             if ($age !== '') {
330 330
                 $html .= ' <span class="label">' . $label . ':</span> <span class="age">' . FunctionsDate::getAgeAtEvent($age) . '</span>';
331 331
             }
Please login to merge, or discard this patch.
app/Module/DescendancyChartModule.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
         echo '<td>';
260 260
         foreach ($person->childFamilies() as $cfamily) {
261 261
             foreach ($cfamily->spouses() as $parent) {
262
-                echo '<a href="' . e($this->chartUrl($parent, ['generations' => $generations])) . '" title="' .  I18N::translate('Start at parents') . '">' . view('icons/arrow-up') . '<span class="sr-only">' .  I18N::translate('Start at parents') . '</span></a>';
262
+                echo '<a href="' . e($this->chartUrl($parent, ['generations' => $generations])) . '" title="' . I18N::translate('Start at parents') . '">' . view('icons/arrow-up') . '<span class="sr-only">' . I18N::translate('Start at parents') . '</span></a>';
263 263
                 // only show the arrow for one of the parents
264 264
                 break;
265 265
             }
@@ -338,7 +338,7 @@  discard block
 block discarded – undo
338 338
         if ($spouse) {
339 339
             foreach ($spouse->childFamilies() as $cfamily) {
340 340
                 foreach ($cfamily->spouses() as $parent) {
341
-                    echo '<a href="' . e($this->chartUrl($parent, ['generations' => $generations])) . '" title="' .  strip_tags($this->chartTitle($parent)) . '">' . view('icons/arrow-up') . '<span class="sr-only">' .  strip_tags($this->chartTitle($parent)) . '</span></a>';
341
+                    echo '<a href="' . e($this->chartUrl($parent, ['generations' => $generations])) . '" title="' . strip_tags($this->chartTitle($parent)) . '">' . view('icons/arrow-up') . '<span class="sr-only">' . strip_tags($this->chartTitle($parent)) . '</span></a>';
342 342
                     // only show the arrow for one of the parents
343 343
                     break;
344 344
                 }
Please login to merge, or discard this patch.
app/Module/ChartsBlockModule.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
                         'generations' => 3,
111 111
                         'layout'      => PedigreeChartModule::ORIENTATION_RIGHT,
112 112
                     ]);
113
-                    $content   = view('modules/charts/chart', [
113
+                    $content = view('modules/charts/chart', [
114 114
                         'block_id'  => $block_id,
115 115
                         'chart_url' => $chart_url,
116 116
                     ]);
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
                         'generations' => 2,
126 126
                         'chart_style' => DescendancyChartModule::CHART_STYLE_LIST,
127 127
                     ]);
128
-                    $content   = view('modules/charts/chart', [
128
+                    $content = view('modules/charts/chart', [
129 129
                         'block_id'  => $block_id,
130 130
                         'chart_url' => $chart_url,
131 131
                     ]);
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
                         'ajax'        => true,
140 140
                         'generations' => 2,
141 141
                     ]);
142
-                    $content   = view('modules/charts/chart', [
142
+                    $content = view('modules/charts/chart', [
143 143
                         'block_id'  => $block_id,
144 144
                         'chart_url' => $chart_url,
145 145
                     ]);
Please login to merge, or discard this patch.
app/Statistics/Repository/PlaceRepository.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
      */
50 50
     public function __construct(Tree $tree)
51 51
     {
52
-        $this->tree          = $tree;
52
+        $this->tree = $tree;
53 53
         $this->country_service = new CountryService();
54 54
     }
55 55
 
Please login to merge, or discard this patch.
app/Http/Controllers/ListController.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -233,7 +233,8 @@  discard block
 block discarded – undo
233 233
                     <li class="wt-initials-list-item d-flex">
234 234
                         <?php if ($count > 0) : ?>
235 235
                             <a href="<?= e(route('module', ['module' => $module, 'action' => $action, 'alpha' => $letter, 'ged' => $tree->name()])) ?>" class="wt-initial px-1<?= $letter === $alpha ? ' active' : '' ?> '" title="<?= I18N::number($count) ?>"><?= $this->surnameInitial((string) $letter) ?></a>
236
-                        <?php else : ?>
236
+                        <?php else {
237
+    : ?>
237 238
                             <span class="wt-initial px-1 text-muted"><?= $this->surnameInitial((string) $letter) ?></span>
238 239
 
239 240
                         <?php endif ?>
@@ -287,6 +288,7 @@  discard block
 block discarded – undo
287 288
 
288 289
             if ($show === 'indi' || $show === 'surn') {
289 290
                 $surns = $this->individual_list_service->surnames($surname, $alpha, $show_marnm === 'yes', $families, WT_LOCALE, I18N::collation());
291
+}
290 292
                 if ($show === 'surn') {
291 293
                     // Show the surname list
292 294
                     switch ($tree->getPreference('SURNAME_LIST_STYLE')) {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -134,10 +134,10 @@  discard block
 block discarded – undo
134 134
         switch ($show_marnm) {
135 135
             case 'no':
136 136
             case 'yes':
137
-                $user->setPreference($families?'family-list-marnm':'individual-list-marnm', $show_marnm);
137
+                $user->setPreference($families ? 'family-list-marnm' : 'individual-list-marnm', $show_marnm);
138 138
                 break;
139 139
             default:
140
-                $show_marnm = $user->getPreference($families?'family-list-marnm':'individual-list-marnm');
140
+                $show_marnm = $user->getPreference($families ? 'family-list-marnm' : 'individual-list-marnm');
141 141
         }
142 142
 
143 143
         // Make sure selections are consistent.
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
                     'ged'      => $tree->name(),
170 170
                     'show_all' => 'yes',
171 171
                 ];
172
-                $show    = $request->getQueryParams()['show'] ?? 'surn';
172
+                $show = $request->getQueryParams()['show'] ?? 'surn';
173 173
             }
174 174
         } elseif ($surname) {
175 175
             $alpha    = $this->localization_service->initialLetter($surname); // so we can highlight the initial letter
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.