Passed
Pull Request — master (#3417)
by Jonathan
05:27
created
app/Http/Controllers/Admin/LocationController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -315,14 +315,14 @@
 block discarded – undo
315 315
 
316 316
         if ($place_id === 0) {
317 317
             $breadcrumbs[]   = I18N::translate('Add');
318
-            $title           .= ' — ' . I18N::translate('Add');
318
+            $title .= ' — ' . I18N::translate('Add');
319 319
             $latitude        = '';
320 320
             $longitude       = '';
321 321
             $map_bounds      = $parent->boundingRectangle();
322 322
             $marker_position = [$parent->latitude(), $parent->longitude()];
323 323
         } else {
324 324
             $breadcrumbs[]   = I18N::translate('Edit');
325
-            $title           .= ' — ' . I18N::translate('Edit');
325
+            $title .= ' — ' . I18N::translate('Edit');
326 326
             $latitude        = $location->latitude();
327 327
             $longitude       = $location->longitude();
328 328
             $map_bounds      = $location->boundingRectangle();
Please login to merge, or discard this patch.
app/Functions/FunctionsPrintFacts.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -583,7 +583,7 @@  discard block
 block discarded – undo
583 583
         for ($j = 0; $j < $ct; $j++) {
584 584
             if (!str_contains($match[$j][1], '@')) {
585 585
                 $source = e($match[$j][1] . preg_replace('/\n\d CONT ?/', "\n", $match[$j][2]));
586
-                $data   .= '<div class="fact_SOUR"><span class="label">' . I18N::translate('Source') . ':</span> <span class="field" dir="auto">' . Filter::formatText($source, $tree) . '</span></div>';
586
+                $data .= '<div class="fact_SOUR"><span class="label">' . I18N::translate('Source') . ':</span> <span class="field" dir="auto">' . Filter::formatText($source, $tree) . '</span></div>';
587 587
             }
588 588
         }
589 589
         // Find source for each fact
@@ -601,7 +601,7 @@  discard block
 block discarded – undo
601 601
                     }
602 602
                     $srec     = substr($factrec, $spos1, $spos2 - $spos1);
603 603
                     $lt       = preg_match_all("/$nlevel \w+/", $srec, $matches);
604
-                    $data     .= '<div class="fact_SOUR">';
604
+                    $data .= '<div class="fact_SOUR">';
605 605
                     $id       = 'collapse-' . Uuid::uuid4()->toString();
606 606
                     $expanded = (bool) $tree->getPreference('EXPAND_SOURCES');
607 607
                     if ($lt > 0) {
Please login to merge, or discard this patch.
app/Http/Controllers/Admin/MediaController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
         $data_filesystem = $request->getAttribute('filesystem.data');
153 153
         assert($data_filesystem instanceof FilesystemInterface);
154 154
 
155
-        $files  = $request->getQueryParams()['files']; // local|external|unused
155
+        $files = $request->getQueryParams()['files']; // local|external|unused
156 156
 
157 157
         // Files within this folder
158 158
         $media_folder = $request->getQueryParams()['media_folder'];
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
                     $create_form = '';
267 267
                     foreach ($media_trees as $media_tree => $media_directory) {
268 268
                         if (str_starts_with($row[0], $media_directory)) {
269
-                            $tmp         = substr($row[0], strlen($media_directory));
269
+                            $tmp = substr($row[0], strlen($media_directory));
270 270
                             $create_form .=
271 271
                                 '<p><a href="#" data-toggle="modal" data-target="#modal-create-media-from-file" data-file="' . e($tmp) . '" data-url="' . e(route('create-media-from-file', ['tree' => $media_tree])) . '" onclick="document.getElementById(\'modal-create-media-from-file-form\').action=this.dataset.url; document.getElementById(\'file\').value=this.dataset.file;">' . I18N::translate('Create') . '</a> — ' . e($media_tree) . '<p>';
272 272
                         }
@@ -360,8 +360,8 @@  discard block
 block discarded – undo
360 360
             try {
361 361
                 // This will work for local filesystems.  For remote filesystems, we will
362 362
                 // need to copy the file locally to work out the image size.
363
-                $imgsize = getimagesize(Webtrees::DATA_DIR .  $file);
364
-                $html    .= '<dt>' . I18N::translate('Image dimensions') . '</dt>';
363
+                $imgsize = getimagesize(Webtrees::DATA_DIR . $file);
364
+                $html .= '<dt>' . I18N::translate('Image dimensions') . '</dt>';
365 365
                 /* I18N: image dimensions, width × height */
366 366
                 $html .= '<dd>' . I18N::translate('%1$s × %2$s pixels', I18N::number($imgsize['0']), I18N::number($imgsize['1'])) . '</dd>';
367 367
             } catch (Throwable $ex) {
Please login to merge, or discard this patch.
app/Module/BranchesListModule.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
 {
65 65
     use ModuleListTrait;
66 66
 
67
-    protected const ROUTE_URL  = '/tree/{tree}/branches{/surname}';
67
+    protected const ROUTE_URL = '/tree/{tree}/branches{/surname}';
68 68
 
69 69
     /** @var ModuleService */
70 70
     protected $module_service;
Please login to merge, or discard this patch.
app/Module/IndividualListModule.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
 {
60 60
     use ModuleListTrait;
61 61
 
62
-    protected const ROUTE_URL  = '/tree/{tree}/individual-list';
62
+    protected const ROUTE_URL = '/tree/{tree}/individual-list';
63 63
 
64 64
     /** @var LocalizationService */
65 65
     private $localization_service;
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
         $alpha = $params['alpha'] ?? '';
200 200
 
201 201
         // All individuals with this surname
202
-        $surname = $params['surname'] ??  '';
202
+        $surname = $params['surname'] ?? '';
203 203
 
204 204
         // All individuals
205 205
         $show_all = $params['show_all'] ?? 'no';
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
                     'tree'     => $tree->name(),
253 253
                     'show_all' => 'yes',
254 254
                 ];
255
-                $show    = $params['show'] ?? 'surn';
255
+                $show = $params['show'] ?? 'surn';
256 256
             }
257 257
         } elseif ($surname !== '') {
258 258
             $alpha    = $this->localization_service->initialLetter($surname, I18N::locale()); // so we can highlight the initial letter
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -326,7 +326,8 @@  discard block
 block discarded – undo
326 326
                     <li class="wt-initials-list-item d-flex">
327 327
                         <?php if ($count > 0) : ?>
328 328
                             <a href="<?= e($this->listUrl($tree, ['alpha' => $letter, 'tree' => $tree->name()])) ?>" class="wt-initial px-1<?= $letter === $alpha ? ' active' : '' ?> '" title="<?= I18N::number($count) ?>"><?= $this->surnameInitial((string) $letter) ?></a>
329
-                        <?php else : ?>
329
+                        <?php else {
330
+    : ?>
330 331
                             <span class="wt-initial px-1 text-muted"><?= $this->surnameInitial((string) $letter) ?></span>
331 332
 
332 333
                         <?php endif ?>
@@ -380,6 +381,7 @@  discard block
 block discarded – undo
380 381
 
381 382
             if ($show === 'indi' || $show === 'surn') {
382 383
                 $surns = $this->surnames($tree, $surname, $alpha, $show_marnm === 'yes', $families, I18N::locale());
384
+}
383 385
                 if ($show === 'surn') {
384 386
                     // Show the surname list
385 387
                     switch ($tree->getPreference('SURNAME_LIST_STYLE')) {
Please login to merge, or discard this patch.