Passed
Pull Request — master (#2414)
by Rico
06:57
created
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/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   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -169,7 +169,7 @@
 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/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/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.