Passed
Push — develop ( 679c96...94f658 )
by Greg
06:17
created
app/Http/Controllers/EditIndividualController.php 2 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -37,12 +37,12 @@
 block discarded – undo
37 37
 class EditIndividualController extends AbstractEditController
38 38
 {
39 39
     /**
40
-      * Add a child to an existing individual (creating a one-parent family).
41
-      *
42
-      * @param ServerRequestInterface $request
43
-      *
44
-      * @return ResponseInterface
45
-      */
40
+     * Add a child to an existing individual (creating a one-parent family).
41
+     *
42
+     * @param ServerRequestInterface $request
43
+     *
44
+     * @return ResponseInterface
45
+     */
46 46
     public function addChild(ServerRequestInterface $request): ResponseInterface
47 47
     {
48 48
         $tree = $request->getAttribute('tree');
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -547,7 +547,7 @@  discard block
 block discarded – undo
547 547
         $tree = $request->getAttribute('tree');
548 548
         assert($tree instanceof Tree);
549 549
 
550
-        $xref  = $request->getQueryParams()['xref'];
550
+        $xref = $request->getQueryParams()['xref'];
551 551
 
552 552
         $individual = Factory::individual()->make($xref, $tree);
553 553
         $individual = Auth::checkIndividualAccess($individual, true);
@@ -559,7 +559,7 @@  discard block
 block discarded – undo
559 559
         $family = Factory::family()->make($famid, $tree);
560 560
         $family = Auth::checkFamilyAccess($family, true);
561 561
 
562
-        $PEDI  = $params['PEDI'];
562
+        $PEDI = $params['PEDI'];
563 563
 
564 564
         // Replace any existing child->family link (we may be changing the PEDI);
565 565
         $fact_id = '';
@@ -631,7 +631,7 @@  discard block
 block discarded – undo
631 631
         $tree = $request->getAttribute('tree');
632 632
         assert($tree instanceof Tree);
633 633
 
634
-        $xref   = $request->getQueryParams()['xref'];
634
+        $xref = $request->getQueryParams()['xref'];
635 635
 
636 636
         $individual = Factory::individual()->make($xref, $tree);
637 637
         $individual = Auth::checkIndividualAccess($individual, true);
Please login to merge, or discard this patch.
app/Source.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
 {
33 33
     public const RECORD_TYPE = 'SOUR';
34 34
 
35
-    protected const ROUTE_NAME  = SourcePage::class;
35
+    protected const ROUTE_NAME = SourcePage::class;
36 36
 
37 37
     /**
38 38
      * A closure which will create a record from a database row.
Please login to merge, or discard this patch.
app/Http/Controllers/Admin/ImportThumbnailsController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@
 block discarded – undo
174 174
                 // Turn each filename into a row for the table
175 175
                 $original = $this->findOriginalFileFromThumbnail($thumbnail);
176 176
 
177
-                $original_url  = route('unused-media-thumbnail', [
177
+                $original_url = route('unused-media-thumbnail', [
178 178
                     'path' => $original,
179 179
                     'w'    => 100,
180 180
                     'h'    => 100,
Please login to merge, or discard this patch.
app/Http/Controllers/ListController.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -172,7 +172,7 @@
 block discarded – undo
172 172
                     'tree'      => $tree->name(),
173 173
                     'show_all' => 'yes',
174 174
                 ];
175
-                $show    = $request->getQueryParams()['show'] ?? 'surn';
175
+                $show = $request->getQueryParams()['show'] ?? 'surn';
176 176
             }
177 177
         } elseif ($surname !== '') {
178 178
             $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
@@ -246,7 +246,8 @@  discard block
 block discarded – undo
246 246
                     <li class="wt-initials-list-item d-flex">
247 247
                         <?php if ($count > 0) : ?>
248 248
                             <a href="<?= e(route('module', ['module' => $module, 'action' => $action, 'alpha' => $letter, 'tree' => $tree->name()])) ?>" class="wt-initial px-1<?= $letter === $alpha ? ' active' : '' ?> '" title="<?= I18N::number($count) ?>"><?= $this->surnameInitial((string) $letter) ?></a>
249
-                        <?php else : ?>
249
+                        <?php else {
250
+    : ?>
250 251
                             <span class="wt-initial px-1 text-muted"><?= $this->surnameInitial((string) $letter) ?></span>
251 252
 
252 253
                         <?php endif ?>
@@ -300,6 +301,7 @@  discard block
 block discarded – undo
300 301
 
301 302
             if ($show === 'indi' || $show === 'surn') {
302 303
                 $surns = $this->individual_list_service->surnames($surname, $alpha, $show_marnm === 'yes', $families, I18N::locale());
304
+}
303 305
                 if ($show === 'surn') {
304 306
                     // Show the surname list
305 307
                     switch ($tree->getPreference('SURNAME_LIST_STYLE')) {
Please login to merge, or discard this patch.
app/Module/TopPageViewsModule.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
 
76 76
         $query = DB::table('hit_counter')
77 77
             ->where('gedcom_id', '=', $tree->id())
78
-            ->whereIn('page_name', ['individual.php','family.php','source.php','repo.php','note.php','mediaviewer.php'])
78
+            ->whereIn('page_name', ['individual.php', 'family.php', 'source.php', 'repo.php', 'note.php', 'mediaviewer.php'])
79 79
             ->orderByDesc('page_count');
80 80
 
81 81
         $results = [];
Please login to merge, or discard this patch.
app/Services/MediaFileService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@
 block discarded – undo
80 80
         $sizePostMax = $this->parseIniFileSize(ini_get('post_max_size'));
81 81
         $sizeUploadMax = $this->parseIniFileSize(ini_get('upload_max_filesize'));
82 82
 
83
-        $bytes =  min($sizePostMax, $sizeUploadMax);
83
+        $bytes = min($sizePostMax, $sizeUploadMax);
84 84
         $kb    = intdiv($bytes + 1023, 1024);
85 85
 
86 86
         return I18N::translate('%s KB', I18N::number($kb));
Please login to merge, or discard this patch.
app/Webtrees.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@
 block discarded – undo
66 66
     // The system files are always in this location.
67 67
     // It is also the default location of user data, such as media and GEDCOM files.
68 68
     // The user files could be anywhere supported by Flysystem.
69
-    public const DATA_DIR  = self::ROOT_DIR . 'data/';
69
+    public const DATA_DIR = self::ROOT_DIR . 'data/';
70 70
 
71 71
     // Location of the file containing the database connection details.
72 72
     public const CONFIG_FILE = self::DATA_DIR . 'config.ini.php';
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
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
         $data_filesystem = $request->getAttribute('filesystem.data');
150 150
         assert($data_filesystem instanceof FilesystemInterface);
151 151
 
152
-        $files  = $request->getQueryParams()['files']; // local|external|unused
152
+        $files = $request->getQueryParams()['files']; // local|external|unused
153 153
 
154 154
         // Files within this folder
155 155
         $media_folder = $request->getQueryParams()['media_folder'];
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
                     $create_form = '';
265 265
                     foreach ($media_trees as $media_tree => $media_directory) {
266 266
                         if (Str::startsWith($row[0], $media_directory)) {
267
-                            $tmp         = substr($row[0], strlen($media_directory));
267
+                            $tmp = substr($row[0], strlen($media_directory));
268 268
                             $create_form .=
269 269
                                 '<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>';
270 270
                         }
@@ -355,8 +355,8 @@  discard block
 block discarded – undo
355 355
             try {
356 356
                 // This will work for local filesystems.  For remote filesystems, we will
357 357
                 // need to copy the file locally to work out the image size.
358
-                $imgsize = getimagesize(Webtrees::DATA_DIR .  $file);
359
-                $html    .= '<dt>' . I18N::translate('Image dimensions') . '</dt>';
358
+                $imgsize = getimagesize(Webtrees::DATA_DIR . $file);
359
+                $html .= '<dt>' . I18N::translate('Image dimensions') . '</dt>';
360 360
                 /* I18N: image dimensions, width × height */
361 361
                 $html .= '<dd>' . I18N::translate('%1$s × %2$s pixels', I18N::number($imgsize['0']), I18N::number($imgsize['1'])) . '</dd>';
362 362
             } catch (Throwable $ex) {
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
@@ -52,7 +52,7 @@
 block discarded – undo
52 52
      */
53 53
     public function find($user_id): ?User
54 54
     {
55
-        return app('cache.array')->remember('user-' . $user_id, static function () use ($user_id): ?User {
55
+        return app('cache.array')->remember('user-' . $user_id, static function () use ($user_id) : ?User {
56 56
             return DB::table('user')
57 57
                 ->where('user_id', '=', $user_id)
58 58
                 ->get()
Please login to merge, or discard this patch.