Completed
Pull Request — master (#1866)
by Rico
10:27 queued 52s
created
app/Http/Controllers/AdminTreesController.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -407,9 +407,9 @@  discard block
 block discarded – undo
407 407
     public function exportClient(Request $request, Tree $tree): Response
408 408
     {
409 409
         // Validate user parameters
410
-        $convert          = (bool)$request->get('convert');
411
-        $zip              = (bool)$request->get('zip');
412
-        $media            = (bool)$request->get('media');
410
+        $convert          = (bool) $request->get('convert');
411
+        $zip              = (bool) $request->get('zip');
412
+        $media            = (bool) $request->get('media');
413 413
         $media_path       = $request->get('media-path');
414 414
         $privatize_export = $request->get('privatize_export');
415 415
 
@@ -539,8 +539,8 @@  discard block
 block discarded – undo
539 539
     public function importAction(Request $request, Tree $tree): RedirectResponse
540 540
     {
541 541
         $source             = $request->get('source');
542
-        $keep_media         = (bool)$request->get('keep_media');
543
-        $WORD_WRAPPED_NOTES = (bool)$request->get('WORD_WRAPPED_NOTES');
542
+        $keep_media         = (bool) $request->get('keep_media');
543
+        $WORD_WRAPPED_NOTES = (bool) $request->get('WORD_WRAPPED_NOTES');
544 544
         $GEDCOM_MEDIA_PATH  = $request->get('GEDCOM_MEDIA_PATH');
545 545
 
546 546
         // Save these choices as defaults
@@ -603,7 +603,7 @@  discard block
 block discarded – undo
603 603
      */
604 604
     public function index(Tree $tree = null): Response
605 605
     {
606
-        $multiple_tree_threshold = (int)Site::getPreference('MULTIPLE_TREE_THRESHOLD', self::MULTIPLE_TREE_THRESHOLD);
606
+        $multiple_tree_threshold = (int) Site::getPreference('MULTIPLE_TREE_THRESHOLD', self::MULTIPLE_TREE_THRESHOLD);
607 607
         $gedcom_files            = $this->gedcomFiles(WT_DATA_DIR);
608 608
 
609 609
         $all_trees = Tree::getAll();
@@ -1034,16 +1034,16 @@  discard block
 block discarded – undo
1034 1034
     public function preferencesUpdate(Request $request, Tree $tree): RedirectResponse
1035 1035
     {
1036 1036
         // Coming soon
1037
-        if ((bool)$request->get('all_trees')) {
1037
+        if ((bool) $request->get('all_trees')) {
1038 1038
             FlashMessages::addMessage(I18N::translate('The preferences for all family trees have been updated.'), 'success');
1039 1039
         }
1040
-        if ((bool)$request->get('new_trees')) {
1040
+        if ((bool) $request->get('new_trees')) {
1041 1041
             FlashMessages::addMessage(I18N::translate('The preferences for new family trees have been updated.'), 'success');
1042 1042
         }
1043 1043
 
1044 1044
         $tree->setPreference('ADVANCED_NAME_FACTS', implode(',', $request->get('ADVANCED_NAME_FACTS', [])));
1045 1045
         $tree->setPreference('ADVANCED_PLAC_FACTS', implode(',', $request->get('ADVANCED_PLAC_FACTS', [])));
1046
-        $tree->setPreference('ALLOW_THEME_DROPDOWN', (string)(bool)$request->get('ALLOW_THEME_DROPDOWN'));
1046
+        $tree->setPreference('ALLOW_THEME_DROPDOWN', (string) (bool) $request->get('ALLOW_THEME_DROPDOWN'));
1047 1047
         // For backwards compatibility with webtrees 1.x we store the two calendar formats in one variable
1048 1048
         // e.g. "gregorian_and_jewish"
1049 1049
         $tree->setPreference('CALENDAR_FORMAT', implode('_and_', array_unique([
@@ -1052,57 +1052,57 @@  discard block
 block discarded – undo
1052 1052
         ])));
1053 1053
         $tree->setPreference('CHART_BOX_TAGS', implode(',', $request->get('CHART_BOX_TAGS', [])));
1054 1054
         $tree->setPreference('CONTACT_USER_ID', $request->get('CONTACT_USER_ID'));
1055
-        $tree->setPreference('DEFAULT_PEDIGREE_GENERATIONS', (string)(int)$request->get('DEFAULT_PEDIGREE_GENERATIONS'));
1056
-        $tree->setPreference('EXPAND_NOTES', (string)(bool)$request->get('EXPAND_NOTES'));
1057
-        $tree->setPreference('EXPAND_SOURCES', (string)(bool)$request->get('EXPAND_SOURCES'));
1055
+        $tree->setPreference('DEFAULT_PEDIGREE_GENERATIONS', (string) (int) $request->get('DEFAULT_PEDIGREE_GENERATIONS'));
1056
+        $tree->setPreference('EXPAND_NOTES', (string) (bool) $request->get('EXPAND_NOTES'));
1057
+        $tree->setPreference('EXPAND_SOURCES', (string) (bool) $request->get('EXPAND_SOURCES'));
1058 1058
         $tree->setPreference('FAM_FACTS_ADD', implode(',', $request->get('FAM_FACTS_ADD', [])));
1059 1059
         $tree->setPreference('FAM_FACTS_QUICK', implode(',', $request->get('FAM_FACTS_QUICK', [])));
1060 1060
         $tree->setPreference('FAM_FACTS_UNIQUE', implode(',', $request->get('FAM_FACTS_UNIQUE', [])));
1061
-        $tree->setPreference('FULL_SOURCES', (string)(bool)$request->get('FULL_SOURCES'));
1061
+        $tree->setPreference('FULL_SOURCES', (string) (bool) $request->get('FULL_SOURCES'));
1062 1062
         $tree->setPreference('FORMAT_TEXT', $request->get('FORMAT_TEXT'));
1063
-        $tree->setPreference('GENERATE_UIDS', (string)(bool)$request->get('GENERATE_UIDS'));
1063
+        $tree->setPreference('GENERATE_UIDS', (string) (bool) $request->get('GENERATE_UIDS'));
1064 1064
         $tree->setPreference('GEONAMES_ACCOUNT', $request->get('GEONAMES_ACCOUNT'));
1065
-        $tree->setPreference('HIDE_GEDCOM_ERRORS', (string)(bool)$request->get('HIDE_GEDCOM_ERRORS'));
1065
+        $tree->setPreference('HIDE_GEDCOM_ERRORS', (string) (bool) $request->get('HIDE_GEDCOM_ERRORS'));
1066 1066
         $tree->setPreference('INDI_FACTS_ADD', implode(',', $request->get('INDI_FACTS_ADD', [])));
1067 1067
         $tree->setPreference('INDI_FACTS_QUICK', implode(',', $request->get('INDI_FACTS_QUICK', [])));
1068 1068
         $tree->setPreference('INDI_FACTS_UNIQUE', implode(',', $request->get('INDI_FACTS_UNIQUE', [])));
1069 1069
         $tree->setPreference('LANGUAGE', $request->get('LANGUAGE'));
1070
-        $tree->setPreference('MAX_DESCENDANCY_GENERATIONS', (string)(int)$request->get('MAX_DESCENDANCY_GENERATIONS'));
1071
-        $tree->setPreference('MAX_PEDIGREE_GENERATIONS', (string)(int)$request->get('MAX_PEDIGREE_GENERATIONS'));
1070
+        $tree->setPreference('MAX_DESCENDANCY_GENERATIONS', (string) (int) $request->get('MAX_DESCENDANCY_GENERATIONS'));
1071
+        $tree->setPreference('MAX_PEDIGREE_GENERATIONS', (string) (int) $request->get('MAX_PEDIGREE_GENERATIONS'));
1072 1072
         $tree->setPreference('MEDIA_UPLOAD', $request->get('MEDIA_UPLOAD'));
1073 1073
         $tree->setPreference('META_DESCRIPTION', $request->get('META_DESCRIPTION'));
1074 1074
         $tree->setPreference('META_TITLE', $request->get('META_TITLE'));
1075
-        $tree->setPreference('NO_UPDATE_CHAN', (string)(bool)$request->get('NO_UPDATE_CHAN'));
1076
-        $tree->setPreference('PEDIGREE_LAYOUT', (string)(bool)$request->get('PEDIGREE_LAYOUT'));
1075
+        $tree->setPreference('NO_UPDATE_CHAN', (string) (bool) $request->get('NO_UPDATE_CHAN'));
1076
+        $tree->setPreference('PEDIGREE_LAYOUT', (string) (bool) $request->get('PEDIGREE_LAYOUT'));
1077 1077
         $tree->setPreference('PEDIGREE_ROOT_ID', $request->get('PEDIGREE_ROOT_ID', WT_REGEX_XREF));
1078
-        $tree->setPreference('PEDIGREE_SHOW_GENDER', (string)(bool)$request->get('PEDIGREE_SHOW_GENDER'));
1078
+        $tree->setPreference('PEDIGREE_SHOW_GENDER', (string) (bool) $request->get('PEDIGREE_SHOW_GENDER'));
1079 1079
         $tree->setPreference('PREFER_LEVEL2_SOURCES', $request->get('PREFER_LEVEL2_SOURCES'));
1080 1080
         $tree->setPreference('QUICK_REQUIRED_FACTS', implode(',', $request->get('QUICK_REQUIRED_FACTS', [])));
1081 1081
         $tree->setPreference('QUICK_REQUIRED_FAMFACTS', implode(',', $request->get('QUICK_REQUIRED_FAMFACTS', [])));
1082 1082
         $tree->setPreference('REPO_FACTS_ADD', implode(',', $request->get('REPO_FACTS_ADD', [])));
1083 1083
         $tree->setPreference('REPO_FACTS_QUICK', implode(',', $request->get('REPO_FACTS_QUICK', [])));
1084 1084
         $tree->setPreference('REPO_FACTS_UNIQUE', implode(',', $request->get('REPO_FACTS_UNIQUE', [])));
1085
-        $tree->setPreference('SHOW_COUNTER', (string)(bool)$request->get('SHOW_COUNTER'));
1086
-        $tree->setPreference('SHOW_EST_LIST_DATES', (string)(bool)$request->get('SHOW_EST_LIST_DATES'));
1087
-        $tree->setPreference('SHOW_FACT_ICONS', (string)(bool)$request->get('SHOW_FACT_ICONS'));
1088
-        $tree->setPreference('SHOW_GEDCOM_RECORD', (string)(bool)$request->get('SHOW_GEDCOM_RECORD'));
1089
-        $tree->setPreference('SHOW_HIGHLIGHT_IMAGES', (string)(bool)$request->get('SHOW_HIGHLIGHT_IMAGES'));
1090
-        $tree->setPreference('SHOW_LAST_CHANGE', (string)(bool)$request->get('SHOW_LAST_CHANGE'));
1091
-        $tree->setPreference('SHOW_LDS_AT_GLANCE', (string)(bool)$request->get('SHOW_LDS_AT_GLANCE'));
1085
+        $tree->setPreference('SHOW_COUNTER', (string) (bool) $request->get('SHOW_COUNTER'));
1086
+        $tree->setPreference('SHOW_EST_LIST_DATES', (string) (bool) $request->get('SHOW_EST_LIST_DATES'));
1087
+        $tree->setPreference('SHOW_FACT_ICONS', (string) (bool) $request->get('SHOW_FACT_ICONS'));
1088
+        $tree->setPreference('SHOW_GEDCOM_RECORD', (string) (bool) $request->get('SHOW_GEDCOM_RECORD'));
1089
+        $tree->setPreference('SHOW_HIGHLIGHT_IMAGES', (string) (bool) $request->get('SHOW_HIGHLIGHT_IMAGES'));
1090
+        $tree->setPreference('SHOW_LAST_CHANGE', (string) (bool) $request->get('SHOW_LAST_CHANGE'));
1091
+        $tree->setPreference('SHOW_LDS_AT_GLANCE', (string) (bool) $request->get('SHOW_LDS_AT_GLANCE'));
1092 1092
         $tree->setPreference('SHOW_MEDIA_DOWNLOAD', $request->get('SHOW_MEDIA_DOWNLOAD'));
1093 1093
         $tree->setPreference('SHOW_NO_WATERMARK', $request->get('SHOW_NO_WATERMARK'));
1094
-        $tree->setPreference('SHOW_PARENTS_AGE', (string)(bool)$request->get('SHOW_PARENTS_AGE'));
1094
+        $tree->setPreference('SHOW_PARENTS_AGE', (string) (bool) $request->get('SHOW_PARENTS_AGE'));
1095 1095
         $tree->setPreference('SHOW_PEDIGREE_PLACES', $request->get('SHOW_PEDIGREE_PLACES'));
1096
-        $tree->setPreference('SHOW_PEDIGREE_PLACES_SUFFIX', (string)(bool)$request->get('SHOW_PEDIGREE_PLACES_SUFFIX'));
1096
+        $tree->setPreference('SHOW_PEDIGREE_PLACES_SUFFIX', (string) (bool) $request->get('SHOW_PEDIGREE_PLACES_SUFFIX'));
1097 1097
         $tree->setPreference('SHOW_RELATIVES_EVENTS', implode(',', $request->get('SHOW_RELATIVES_EVENTS', [])));
1098 1098
         $tree->setPreference('SOUR_FACTS_ADD', implode(',', $request->get('SOUR_FACTS_ADD', [])));
1099 1099
         $tree->setPreference('SOUR_FACTS_QUICK', implode(',', $request->get('SOUR_FACTS_QUICK', [])));
1100 1100
         $tree->setPreference('SOUR_FACTS_UNIQUE', implode(',', $request->get('SOUR_FACTS_UNIQUE', [])));
1101
-        $tree->setPreference('SUBLIST_TRIGGER_I', (string)(int)$request->get('SUBLIST_TRIGGER_I', 200));
1101
+        $tree->setPreference('SUBLIST_TRIGGER_I', (string) (int) $request->get('SUBLIST_TRIGGER_I', 200));
1102 1102
         $tree->setPreference('SURNAME_LIST_STYLE', $request->get('SURNAME_LIST_STYLE'));
1103 1103
         $tree->setPreference('SURNAME_TRADITION', $request->get('SURNAME_TRADITION'));
1104 1104
         $tree->setPreference('THEME_DIR', $request->get('THEME_DIR'));
1105
-        $tree->setPreference('USE_SILHOUETTE', (string)(bool)$request->get('USE_SILHOUETTE'));
1105
+        $tree->setPreference('USE_SILHOUETTE', (string) (bool) $request->get('USE_SILHOUETTE'));
1106 1106
         $tree->setPreference('WEBMASTER_USER_ID', $request->get('WEBMASTER_USER_ID'));
1107 1107
         $tree->setPreference('WEBTREES_EMAIL', $request->get('WEBTREES_EMAIL'));
1108 1108
         $tree->setPreference('title', $request->get('title'));
@@ -1607,7 +1607,7 @@  discard block
 block discarded – undo
1607 1607
 
1608 1608
         foreach ($gedcom_files as $gedcom_file) {
1609 1609
             // Only import files that have changed
1610
-            $filemtime = (string)filemtime(WT_DATA_DIR . $gedcom_file);
1610
+            $filemtime = (string) filemtime(WT_DATA_DIR . $gedcom_file);
1611 1611
 
1612 1612
             $tree = Tree::findByName($gedcom_file) ?? Tree::create($gedcom_file, $gedcom_file);
1613 1613
 
@@ -1640,7 +1640,7 @@  discard block
 block discarded – undo
1640 1640
      */
1641 1641
     public function unconnected(Request $request, Tree $tree, User $user): Response
1642 1642
     {
1643
-        $associates = (bool)$request->get('associates');
1643
+        $associates = (bool) $request->get('associates');
1644 1644
 
1645 1645
         if ($associates) {
1646 1646
             $sql = "SELECT l_from, l_to FROM `##link` WHERE l_file = :tree_id AND l_type IN ('FAMS', 'FAMC', 'ASSO', '_ASSO')";
Please login to merge, or discard this patch.
app/Http/Controllers/AdminSiteController.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -263,35 +263,35 @@  discard block
 block discarded – undo
263 263
 
264 264
         $args = [];
265 265
         if ($search) {
266
-            $sql            .= " AND log_message LIKE CONCAT('%', :search, '%')";
266
+            $sql .= " AND log_message LIKE CONCAT('%', :search, '%')";
267 267
             $args['search'] = $search;
268 268
         }
269 269
         if ($from) {
270
-            $sql          .= " AND log_time >= :from";
270
+            $sql .= " AND log_time >= :from";
271 271
             $args['from'] = $from;
272 272
         }
273 273
         if ($to) {
274
-            $sql        .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
274
+            $sql .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
275 275
             $args['to'] = $to;
276 276
         }
277 277
         if ($type) {
278
-            $sql          .= " AND log_type = :type";
278
+            $sql .= " AND log_type = :type";
279 279
             $args['type'] = $type;
280 280
         }
281 281
         if ($text) {
282
-            $sql          .= " AND log_message LIKE CONCAT('%', :text, '%')";
282
+            $sql .= " AND log_message LIKE CONCAT('%', :text, '%')";
283 283
             $args['text'] = $text;
284 284
         }
285 285
         if ($ip) {
286
-            $sql        .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
286
+            $sql .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
287 287
             $args['ip'] = $ip;
288 288
         }
289 289
         if ($username) {
290
-            $sql          .= " AND user_name LIKE CONCAT('%', :user, '%')";
290
+            $sql .= " AND user_name LIKE CONCAT('%', :user, '%')";
291 291
             $args['user'] = $username;
292 292
         }
293 293
         if ($gedc) {
294
-            $sql          .= " AND gedcom_name = :gedc";
294
+            $sql .= " AND gedcom_name = :gedc";
295 295
             $args['gedc'] = $gedc;
296 296
         }
297 297
 
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
         }
318 318
 
319 319
         if ($length) {
320
-            $sql            .= " LIMIT :limit OFFSET :offset";
320
+            $sql .= " LIMIT :limit OFFSET :offset";
321 321
             $args['limit']  = $length;
322 322
             $args['offset'] = $start;
323 323
         }
@@ -375,35 +375,35 @@  discard block
 block discarded – undo
375 375
 
376 376
         $args = [];
377 377
         if ($search) {
378
-            $sql            .= " AND log_message LIKE CONCAT('%', :search, '%')";
378
+            $sql .= " AND log_message LIKE CONCAT('%', :search, '%')";
379 379
             $args['search'] = $search;
380 380
         }
381 381
         if ($from) {
382
-            $sql          .= " AND log_time >= :from";
382
+            $sql .= " AND log_time >= :from";
383 383
             $args['from'] = $from;
384 384
         }
385 385
         if ($to) {
386
-            $sql        .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
386
+            $sql .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
387 387
             $args['to'] = $to;
388 388
         }
389 389
         if ($type) {
390
-            $sql          .= " AND log_type = :type";
390
+            $sql .= " AND log_type = :type";
391 391
             $args['type'] = $type;
392 392
         }
393 393
         if ($text) {
394
-            $sql          .= " AND log_message LIKE CONCAT('%', :text, '%')";
394
+            $sql .= " AND log_message LIKE CONCAT('%', :text, '%')";
395 395
             $args['text'] = $text;
396 396
         }
397 397
         if ($ip) {
398
-            $sql        .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
398
+            $sql .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
399 399
             $args['ip'] = $ip;
400 400
         }
401 401
         if ($username) {
402
-            $sql          .= " AND user_name LIKE CONCAT('%', :user, '%')";
402
+            $sql .= " AND user_name LIKE CONCAT('%', :user, '%')";
403 403
             $args['user'] = $username;
404 404
         }
405 405
         if ($gedc) {
406
-            $sql          .= " AND gedcom_name = :gedc";
406
+            $sql .= " AND gedcom_name = :gedc";
407 407
             $args['gedc'] = $gedc;
408 408
         }
409 409
 
@@ -436,31 +436,31 @@  discard block
 block discarded – undo
436 436
 
437 437
         $args = [];
438 438
         if ($from) {
439
-            $sql          .= " AND log_time >= :from";
439
+            $sql .= " AND log_time >= :from";
440 440
             $args['from'] = $from;
441 441
         }
442 442
         if ($to) {
443
-            $sql        .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
443
+            $sql .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
444 444
             $args['to'] = $to;
445 445
         }
446 446
         if ($type) {
447
-            $sql          .= " AND log_type = :type";
447
+            $sql .= " AND log_type = :type";
448 448
             $args['type'] = $type;
449 449
         }
450 450
         if ($text) {
451
-            $sql          .= " AND log_message LIKE CONCAT('%', :text, '%')";
451
+            $sql .= " AND log_message LIKE CONCAT('%', :text, '%')";
452 452
             $args['text'] = $text;
453 453
         }
454 454
         if ($ip) {
455
-            $sql        .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
455
+            $sql .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
456 456
             $args['ip'] = $ip;
457 457
         }
458 458
         if ($username) {
459
-            $sql          .= " AND user_name LIKE CONCAT('%', :user, '%')";
459
+            $sql .= " AND user_name LIKE CONCAT('%', :user, '%')";
460 460
             $args['user'] = $username;
461 461
         }
462 462
         if ($gedc) {
463
-            $sql          .= " AND gedcom_name = :gedc";
463
+            $sql .= " AND gedcom_name = :gedc";
464 464
             $args['gedc'] = $gedc;
465 465
         }
466 466
 
Please login to merge, or discard this patch.
app/Http/Controllers/AdminUsersController.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -240,7 +240,7 @@
 block discarded – undo
240 240
             // Link to send email to other users.
241 241
             if ($user_id != $user->getUserId()) {
242 242
                 $datum[4] = '<a href="' . e(route('message', ['to'  => $datum[2],
243
-                                                              'url' => route('admin-users'),
243
+                                                                'url' => route('admin-users'),
244 244
                     ])) . '">' . $datum[4] . '</a>';
245 245
             }
246 246
 
Please login to merge, or discard this patch.
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
      */
50 50
     public function cleanup(Request $request): Response
51 51
     {
52
-        $months = (int)$request->get('months', 6);
52
+        $months = (int) $request->get('months', 6);
53 53
 
54 54
         $inactive_threshold   = time() - $months * 30 * self::SECONDS_PER_DAY;
55 55
         $unverified_threshold = time() - 7 * self::SECONDS_PER_DAY;
@@ -58,9 +58,9 @@  discard block
 block discarded – undo
58 58
 
59 59
         $inactive_users = array_filter($users, function (User $user) use ($inactive_threshold): bool {
60 60
             if ($user->getPreference('sessiontime') === '0') {
61
-                $datelogin = (int)$user->getPreference('reg_timestamp');
61
+                $datelogin = (int) $user->getPreference('reg_timestamp');
62 62
             } else {
63
-                $datelogin = (int)$user->getPreference('sessiontime');
63
+                $datelogin = (int) $user->getPreference('sessiontime');
64 64
             }
65 65
 
66 66
             return $datelogin < $inactive_threshold && $user->getPreference('verified');
@@ -68,9 +68,9 @@  discard block
 block discarded – undo
68 68
 
69 69
         $unverified_users = array_filter($users, function (User $user) use ($unverified_threshold): bool {
70 70
             if ($user->getPreference('sessiontime') === '0') {
71
-                $datelogin = (int)$user->getPreference('reg_timestamp');
71
+                $datelogin = (int) $user->getPreference('reg_timestamp');
72 72
             } else {
73
-                $datelogin = (int)$user->getPreference('sessiontime');
73
+                $datelogin = (int) $user->getPreference('sessiontime');
74 74
             }
75 75
 
76 76
             return $datelogin < $unverified_threshold && !$user->getPreference('verified');
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     public function cleanupAction(Request $request): RedirectResponse
98 98
     {
99 99
         foreach (User::all() as $user) {
100
-            if ((bool)$request->get('del_' . $user->getUserId())) {
100
+            if ((bool) $request->get('del_' . $user->getUserId())) {
101 101
                 Log::addAuthenticationLog('Deleted user: ' . $user->getUserName());
102 102
                 $user->delete();
103 103
 
@@ -144,10 +144,10 @@  discard block
 block discarded – undo
144 144
     public function data(Request $request, User $user): JsonResponse
145 145
     {
146 146
         $search = $request->get('search')['value'];
147
-        $start  = (int)$request->get('start');
148
-        $length = (int)$request->get('length');
147
+        $start  = (int) $request->get('start');
148
+        $length = (int) $request->get('length');
149 149
         $order  = $request->get('order', []);
150
-        $draw   = (int)$request->get('draw');
150
+        $draw   = (int) $request->get('draw');
151 151
 
152 152
         $sql_select =
153 153
             "SELECT SQL_CALC_FOUND_ROWS u.user_id, user_name, real_name, email, us1.setting_value AS language, us2.setting_value AS registered_at, us3.setting_value AS active_at, us4.setting_value AS verified, us5.setting_value AS verified_by_admin" .
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
         $args = [];
163 163
 
164 164
         if ($search) {
165
-            $sql_select       .= " AND (user_name LIKE CONCAT('%', :search_1, '%') OR real_name LIKE CONCAT('%', :search_2, '%') OR email LIKE CONCAT('%', :search_3, '%'))";
165
+            $sql_select .= " AND (user_name LIKE CONCAT('%', :search_1, '%') OR real_name LIKE CONCAT('%', :search_2, '%') OR email LIKE CONCAT('%', :search_3, '%'))";
166 166
             $args['search_1'] = $search;
167 167
             $args['search_2'] = $search;
168 168
             $args['search_3'] = $search;
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
 
192 192
         if ($length) {
193 193
             $user->setPreference('admin_users_page_size', $length);
194
-            $sql_select     .= " LIMIT :limit OFFSET :offset";
194
+            $sql_select .= " LIMIT :limit OFFSET :offset";
195 195
             $args['limit']  = $length;
196 196
             $args['offset'] = $start;
197 197
         }
@@ -199,8 +199,8 @@  discard block
 block discarded – undo
199 199
         $rows = Database::prepare($sql_select)->execute($args)->fetchAll();
200 200
 
201 201
         // Total filtered/unfiltered rows
202
-        $recordsFiltered = (int)Database::prepare("SELECT FOUND_ROWS()")->fetchOne();
203
-        $recordsTotal    = (int)Database::prepare("SELECT COUNT(*) FROM `##user` WHERE user_id > 0")->fetchOne();
202
+        $recordsFiltered = (int) Database::prepare("SELECT FOUND_ROWS()")->fetchOne();
203
+        $recordsTotal    = (int) Database::prepare("SELECT COUNT(*) FROM `##user` WHERE user_id > 0")->fetchOne();
204 204
 
205 205
         $installed_languages = [];
206 206
         foreach (I18N::installedLocales() as $installed_locale) {
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
      */
287 287
     public function edit(Request $request): Response
288 288
     {
289
-        $user_id = (int)$request->get('user_id');
289
+        $user_id = (int) $request->get('user_id');
290 290
         $user    = User::find($user_id);
291 291
 
292 292
         if ($user === null) {
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
      */
370 370
     public function update(Request $request, User $user): RedirectResponse
371 371
     {
372
-        $user_id        = (int)$request->get('user_id');
372
+        $user_id        = (int) $request->get('user_id');
373 373
         $username       = $request->get('username');
374 374
         $real_name      = $request->get('real_name');
375 375
         $email          = $request->get('email');
@@ -379,11 +379,11 @@  discard block
 block discarded – undo
379 379
         $timezone       = $request->get('timezone');
380 380
         $contact_method = $request->get('contact_method');
381 381
         $comment        = $request->get('comment');
382
-        $auto_accept    = (bool)$request->get('auto_accept');
383
-        $canadmin       = (bool)$request->get('canadmin');
384
-        $visible_online = (bool)$request->get('visible_online');
385
-        $verified       = (bool)$request->get('verified');
386
-        $approved       = (bool)$request->get('approved');
382
+        $auto_accept    = (bool) $request->get('auto_accept');
383
+        $canadmin       = (bool) $request->get('canadmin');
384
+        $visible_online = (bool) $request->get('visible_online');
385
+        $verified       = (bool) $request->get('verified');
386
+        $approved       = (bool) $request->get('approved');
387 387
 
388 388
         $edit_user = User::find($user_id);
389 389
 
@@ -400,10 +400,10 @@  discard block
 block discarded – undo
400 400
             ->setPreference('TIMEZONE', $timezone)
401 401
             ->setPreference('contactmethod', $contact_method)
402 402
             ->setPreference('comment', $comment)
403
-            ->setPreference('auto_accept', (string)$auto_accept)
404
-            ->setPreference('visibleonline', (string)$visible_online)
405
-            ->setPreference('verified', (string)$verified)
406
-            ->setPreference('verified_by_admin', (string)$approved);
403
+            ->setPreference('auto_accept', (string) $auto_accept)
404
+            ->setPreference('visibleonline', (string) $visible_online)
405
+            ->setPreference('verified', (string) $verified)
406
+            ->setPreference('verified_by_admin', (string) $approved);
407 407
 
408 408
         if ($pass1 !== '') {
409 409
             $edit_user->setPassword($pass1);
@@ -415,7 +415,7 @@  discard block
 block discarded – undo
415 415
         }
416 416
 
417 417
         foreach (Tree::getAll() as $tree) {
418
-            $path_length = (int)$request->get('RELATIONSHIP_PATH_LENGTH' . $tree->getTreeId());
418
+            $path_length = (int) $request->get('RELATIONSHIP_PATH_LENGTH' . $tree->getTreeId());
419 419
             $gedcom_id   = $request->get('gedcomid' . $tree->getTreeId(), '');
420 420
             $can_edit    = $request->get('canedit' . $tree->getTreeId(), '');
421 421
 
@@ -426,7 +426,7 @@  discard block
 block discarded – undo
426 426
 
427 427
             $tree->setUserPreference($edit_user, 'gedcomid', $gedcom_id);
428 428
             $tree->setUserPreference($edit_user, 'canedit', $can_edit);
429
-            $tree->setUserPreference($edit_user, 'RELATIONSHIP_PATH_LENGTH', (string)$path_length);
429
+            $tree->setUserPreference($edit_user, 'RELATIONSHIP_PATH_LENGTH', (string) $path_length);
430 430
         }
431 431
 
432 432
         $url = route('admin-users');
Please login to merge, or discard this patch.
app/Http/Controllers/TimelineChartController.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -60,11 +60,11 @@  discard block
 block discarded – undo
60 60
     {
61 61
         $this->checkModuleIsActive($tree, 'timeline_chart');
62 62
 
63
-        $scale = (int)$request->get('scale', self::SCALE_DEFAULT);
63
+        $scale = (int) $request->get('scale', self::SCALE_DEFAULT);
64 64
         $scale = min($scale, self::SCALE_MAX);
65 65
         $scale = max($scale, self::SCALE_MIN);
66 66
 
67
-        $xrefs = (array)$request->get('xrefs', []);
67
+        $xrefs = (array) $request->get('xrefs', []);
68 68
         $xrefs = array_unique($xrefs);
69 69
         $xrefs = array_filter($xrefs, function (string $xref) use ($tree): bool {
70 70
             $individual = Individual::getInstance($xref, $tree);
@@ -100,14 +100,14 @@  discard block
 block discarded – undo
100 100
 
101 101
         $zoom_in_url = route('timeline', [
102 102
             'ged'   => $tree->getName(),
103
-            'scale' => min(self::SCALE_MAX, $scale + (int)($scale * 0.2 + 1)),
103
+            'scale' => min(self::SCALE_MAX, $scale + (int) ($scale * 0.2 + 1)),
104 104
             'xrefs' => $xrefs,
105 105
         ]);
106 106
 
107 107
         $zoom_out_url = route('timeline', [
108 108
             'route' => 'timeline',
109 109
             'ged'   => $tree->getName(),
110
-            'scale' => max(self::SCALE_MIN, $scale - (int)($scale * 0.2 + 1)),
110
+            'scale' => max(self::SCALE_MIN, $scale - (int) ($scale * 0.2 + 1)),
111 111
             'xrefs' => $xrefs,
112 112
         ]);
113 113
 
@@ -132,11 +132,11 @@  discard block
 block discarded – undo
132 132
     {
133 133
         $this->checkModuleIsActive($tree, 'timeline_chart');
134 134
 
135
-        $scale = (int)$request->get('scale', self::SCALE_DEFAULT);
135
+        $scale = (int) $request->get('scale', self::SCALE_DEFAULT);
136 136
         $scale = min($scale, self::SCALE_MAX);
137 137
         $scale = max($scale, self::SCALE_MIN);
138 138
 
139
-        $xrefs = (array)$request->get('xrefs', []);
139
+        $xrefs = (array) $request->get('xrefs', []);
140 140
         $xrefs = array_unique($xrefs);
141 141
 
142 142
         /** @var Individual[] $individuals */
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
             return $individual !== null && $individual->canShow();
149 149
         });
150 150
 
151
-        $baseyear    = (int)date('Y');
151
+        $baseyear    = (int) date('Y');
152 152
         $topyear     = 0;
153 153
         $indifacts   = [];
154 154
         $birthyears  = [];
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
                         $topyear  = max($topyear, $date->y);
184 184
 
185 185
                         if (!$individual->isDead()) {
186
-                            $topyear = max($topyear, (int)date('Y'));
186
+                            $topyear = max($topyear, (int) date('Y'));
187 187
                         }
188 188
 
189 189
                         // do not add the same fact twice (prevents marriages from being added multiple times)
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
         }
197 197
 
198 198
         if ($scale === 0) {
199
-            $scale = (int)(($topyear - $baseyear) / 20 * count($indifacts) / 4);
199
+            $scale = (int) (($topyear - $baseyear) / 20 * count($indifacts) / 4);
200 200
             if ($scale < 6) {
201 201
                 $scale = 6;
202 202
             }
Please login to merge, or discard this patch.
app/Services/UpgradeService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@
 block discarded – undo
73 73
     {
74 74
         $latest_version = $this->fetchLatestVersion();
75 75
 
76
-        list(, , $url) = explode('|', $latest_version . '||');
76
+        list(,, $url) = explode('|', $latest_version . '||');
77 77
 
78 78
         return $url;
79 79
     }
Please login to merge, or discard this patch.
app/Functions/FunctionsPrintLists.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
                 } else {
54 54
                     $size = 75.0 + 125.0 * ($count - $minimum) / ($maximum - $minimum);
55 55
                 }
56
-                $url  = route($route, [
56
+                $url = route($route, [
57 57
                     'surname' => $surn,
58 58
                     'ged'     => $tree->getName(),
59 59
                 ]);
@@ -130,13 +130,13 @@  discard block
 block discarded – undo
130 130
                 }
131 131
                 $newcol = ceil($count / $col);
132 132
                 $html2  = '<table class="list_table"><tr>';
133
-                $html2  .= '<td class="list_value" style="padding: 14px;">';
133
+                $html2 .= '<td class="list_value" style="padding: 14px;">';
134 134
 
135 135
                 foreach ($html as $surns) {
136 136
                     $html2 .= $surns . '<br>';
137 137
                     $i++;
138 138
                     if ($i == $newcol && $i < $count) {
139
-                        $html2  .= '</td><td class="list_value" style="padding: 14px;">';
139
+                        $html2 .= '</td><td class="list_value" style="padding: 14px;">';
140 140
                         $newcol = $i + ceil($count / $col);
141 141
                     }
142 142
                 }
Please login to merge, or discard this patch.
app/Http/Controllers/Auth/ForgotPasswordController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 
70 70
             Log::addAuthenticationLog('Password request was sent to user: ' . $user->getUserName());
71 71
 
72
-            $sender = new User((object)[
72
+            $sender = new User((object) [
73 73
                 'user_id'   => null,
74 74
                 'user_name' => '',
75 75
                 'real_name' => $tree->getTitle(),
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
         $max       = strlen($passchars) - 1;
105 105
 
106 106
         for ($i = 0; $i < 8; $i++) {
107
-            $index    = random_int(0, $max);
107
+            $index = random_int(0, $max);
108 108
             $password .= $passchars{$index};
109 109
         }
110 110
 
Please login to merge, or discard this patch.
app/Http/Controllers/SetupController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     {
48 48
         define('WT_LOCALE', I18N::init('en-US'));
49 49
 
50
-        $step     = (int)$request->get('step', '1');
50
+        $step     = (int) $request->get('step', '1');
51 51
         $errors   = $this->serverErrors();
52 52
         $warnings = $this->serverWarnings();
53 53
         $data     = $this->extractParameters($request);
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
      */
365 365
     private function maxExecutionTime(): int
366 366
     {
367
-        return (int)ini_get('max_execution_time');
367
+        return (int) ini_get('max_execution_time');
368 368
     }
369 369
 
370 370
     /**
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
                 $memory_limit = $memory_limit / 1024 / 1024;
398 398
         }
399 399
 
400
-        return (int)$memory_limit;
400
+        return (int) $memory_limit;
401 401
     }
402 402
 
403 403
     /**
@@ -454,11 +454,11 @@  discard block
 block discarded – undo
454 454
             /* I18N: a program feature */
455 455
             'simplexml' => I18N::translate('reporting'),
456 456
         ];
457
-        $settings   = [
457
+        $settings = [
458 458
             /* I18N: a program feature */
459 459
             'file_uploads' => I18N::translate('file upload capability'),
460 460
         ];
461
-        $warnings   = [];
461
+        $warnings = [];
462 462
 
463 463
         foreach ($extensions as $extension => $features) {
464 464
             if (!extension_loaded($extension)) {
Please login to merge, or discard this patch.
app/Http/Controllers/HourglassChartController.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -55,15 +55,15 @@  discard block
 block discarded – undo
55 55
 
56 56
         $this->checkIndividualAccess($individual);
57 57
 
58
-        $maximum_generations = (int)$tree->getPreference('MAX_DESCENDANCY_GENERATIONS', self::DEFAULT_MAXIMUM_GENERATIONS);
59
-        $default_generations = (int)$tree->getPreference('DEFAULT_PEDIGREE_GENERATIONS', self::DEFAULT_GENERATIONS);
58
+        $maximum_generations = (int) $tree->getPreference('MAX_DESCENDANCY_GENERATIONS', self::DEFAULT_MAXIMUM_GENERATIONS);
59
+        $default_generations = (int) $tree->getPreference('DEFAULT_PEDIGREE_GENERATIONS', self::DEFAULT_GENERATIONS);
60 60
 
61
-        $generations = (int)$request->get('generations', $default_generations);
61
+        $generations = (int) $request->get('generations', $default_generations);
62 62
 
63 63
         $generations = min($generations, $maximum_generations);
64 64
         $generations = max($generations, self::MINIMUM_GENERATIONS);
65 65
 
66
-        $show_spouse = (bool)$request->get('show_spouse');
66
+        $show_spouse = (bool) $request->get('show_spouse');
67 67
 
68 68
         /* I18N: %s is an individual’s name */
69 69
         $title = I18N::translate('Hourglass chart of %s', $individual->getFullName());
@@ -95,14 +95,14 @@  discard block
 block discarded – undo
95 95
 
96 96
         $this->checkIndividualAccess($individual);
97 97
 
98
-        $maximum_generations = (int)$tree->getPreference('MAX_DESCENDANCY_GENERATIONS', self::DEFAULT_MAXIMUM_GENERATIONS);
99
-        $default_generations = (int)$tree->getPreference('DEFAULT_PEDIGREE_GENERATIONS', self::DEFAULT_GENERATIONS);
98
+        $maximum_generations = (int) $tree->getPreference('MAX_DESCENDANCY_GENERATIONS', self::DEFAULT_MAXIMUM_GENERATIONS);
99
+        $default_generations = (int) $tree->getPreference('DEFAULT_PEDIGREE_GENERATIONS', self::DEFAULT_GENERATIONS);
100 100
 
101
-        $generations = (int)$request->get('generations', $default_generations);
101
+        $generations = (int) $request->get('generations', $default_generations);
102 102
         $generations = min($generations, $maximum_generations);
103 103
         $generations = max($generations, self::MINIMUM_GENERATIONS);
104 104
 
105
-        $show_spouse = (bool)$request->get('show_spouse');
105
+        $show_spouse = (bool) $request->get('show_spouse');
106 106
 
107 107
         ob_start();
108 108
         $this->printDescendency($individual, 1, $generations, $show_spouse, true);
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
         return new Response(view('hourglass-chart', [
116 116
             'descendants' => $descendants,
117 117
             'ancestors'   => $ancestors,
118
-            'bhalfheight' => (int)(Theme::theme()->parameter('chart-box-y') / 2),
118
+            'bhalfheight' => (int) (Theme::theme()->parameter('chart-box-y') / 2),
119 119
         ]));
120 120
     }
121 121
 
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 
135 135
         $this->checkIndividualAccess($individual);
136 136
 
137
-        $show_spouse = (bool)$request->get('show_spouse');
137
+        $show_spouse = (bool) $request->get('show_spouse');
138 138
 
139 139
         ob_start();
140 140
         $this->printPersonPedigree($individual, 0, 1, $show_spouse);
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
         $xref       = $request->get('xref');
157 157
         $individual = Individual::getInstance($xref, $tree);
158 158
 
159
-        $show_spouse = (bool)$request->get('show_spouse');
159
+        $show_spouse = (bool) $request->get('show_spouse');
160 160
 
161 161
         ob_start();
162 162
         $this->printDescendency($individual, 1, 2, $show_spouse, false);
@@ -309,7 +309,7 @@  discard block
 block discarded – undo
309 309
                     echo '<a href="' . e(route('hourglass', [
310 310
                             'xref'        => $spouse->getXref(),
311 311
                             'generations' => $generations,
312
-                            'show_spouse' => (int)$show_spouse,
312
+                            'show_spouse' => (int) $show_spouse,
313 313
                             'ged'         => $spouse->getTree()->getName(),
314 314
                         ])) . '" class="name1">' . $spouse->getFullName() . '</a>';
315 315
                 }
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
                     echo '<a href="' . e(route('hourglass', [
318 318
                             'xref'        => $child->getXref(),
319 319
                             'generations' => $generations,
320
-                            'show_spouse' => (int)$show_spouse,
320
+                            'show_spouse' => (int) $show_spouse,
321 321
                             'ged'         => $child->getTree()->getName(),
322 322
                         ])) . '" class="name1">' . $child->getFullName() . '</a>';
323 323
                 }
@@ -332,7 +332,7 @@  discard block
 block discarded – undo
332 332
                         echo '<a href="' . e(route('hourglass', [
333 333
                                 'xref'        => $husb->getXref(),
334 334
                                 'generations' => $generations,
335
-                                'show_spouse' => (int)$show_spouse,
335
+                                'show_spouse' => (int) $show_spouse,
336 336
                                 'ged'         => $husb->getTree()->getName(),
337 337
                             ])) . '" class="name1">' . $husb->getFullName() . '</a>';
338 338
                     }
@@ -341,7 +341,7 @@  discard block
 block discarded – undo
341 341
                         echo '<a href="' . e(route('hourglass', [
342 342
                                 'xref'        => $wife->getXref(),
343 343
                                 'generations' => $generations,
344
-                                'show_spouse' => (int)$show_spouse,
344
+                                'show_spouse' => (int) $show_spouse,
345 345
                                 'ged'         => $wife->getTree()->getName(),
346 346
                             ])) . '" class="name1">' . $wife->getFullName() . '</a>';
347 347
                     }
@@ -360,7 +360,7 @@  discard block
 block discarded – undo
360 360
                         echo '<a href="' . e(route('hourglass', [
361 361
                                 'xref'        => $child->getXref(),
362 362
                                 'generations' => $generations,
363
-                                'show_spouse' => (int)$show_spouse,
363
+                                'show_spouse' => (int) $show_spouse,
364 364
                                 'ged'         => $child->getTree()->getName(),
365 365
                             ])) . '" class="name1">' . $child->getFullName() . '</a>';
366 366
                     }
@@ -423,7 +423,7 @@  discard block
 block discarded – undo
423 423
                         'href'         => '#',
424 424
                         'data-route'   => 'hourglass-add-asc',
425 425
                         'data-xref'    => $ARID,
426
-                        'data-spouses' => (int)$show_spouse,
426
+                        'data-spouses' => (int) $show_spouse,
427 427
                         'data-tree'    => $family->getHusband()->getTree()->getName(),
428 428
                     ]);
429 429
                 }
@@ -460,7 +460,7 @@  discard block
 block discarded – undo
460 460
                         'href'         => '#',
461 461
                         'data-route'   => 'hourglass-add-asc',
462 462
                         'data-xref'    => $ARID,
463
-                        'data-spouses' => (int)$show_spouse,
463
+                        'data-spouses' => (int) $show_spouse,
464 464
                         'data-tree'    => $family->getWife()->getTree()->getName(),
465 465
                     ]);
466 466
                 }
Please login to merge, or discard this patch.