@@ -37,12 +37,12 @@ |
||
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'); |
@@ -547,7 +547,7 @@ discard block |
||
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 |
||
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 |
||
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); |
@@ -32,7 +32,7 @@ |
||
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. |
@@ -174,7 +174,7 @@ |
||
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, |
@@ -172,7 +172,7 @@ |
||
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 |
@@ -246,7 +246,8 @@ discard block |
||
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 |
||
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')) { |
@@ -75,7 +75,7 @@ |
||
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 = []; |
@@ -80,7 +80,7 @@ |
||
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)); |
@@ -66,7 +66,7 @@ |
||
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'; |
@@ -149,7 +149,7 @@ discard block |
||
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 |
||
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 |
||
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) { |
@@ -52,7 +52,7 @@ |
||
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() |