Passed
Push — analysis-XV3nbg ( 6a7be2 )
by Greg
09:07
created
app/Module.php 1 patch
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -158,7 +158,7 @@
 block discarded – undo
158 158
                     $modules[$module->getName()] = $module;
159 159
                 } else {
160 160
                     // The module has been deleted or is broken? Disable it.
161
-                    Log::addConfigurationLog("Module {$module_name} is missing or broken - disabling it. ", null);
161
+                    Log::addConfigurationLog("module {$module_name} is missing or broken - disabling it. ", null);
162 162
                     DB::table('module')
163 163
                         ->where('module_name', '=', $module_name)
164 164
                         ->update(['status' => 'disabled']);
Please login to merge, or discard this patch.
app/Theme/MinimalTheme.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
      * Where are our CSS, JS and other assets?
29 29
      */
30 30
     protected const THEME_DIR  = 'minimal';
31
-    public const ASSET_DIR  = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
31
+    public const ASSET_DIR = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
32 32
     protected const STYLESHEET = self::ASSET_DIR . 'style.css';
33 33
 
34 34
     /**
Please login to merge, or discard this patch.
app/Theme/XeneaTheme.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
      * Where are our CSS, JS and other assets?
29 29
      */
30 30
     protected const THEME_DIR  = 'xenea';
31
-    public const ASSET_DIR  = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
31
+    public const ASSET_DIR = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
32 32
     protected const STYLESHEET = self::ASSET_DIR . 'style.css';
33 33
 
34 34
     /**
Please login to merge, or discard this patch.
app/Theme/ColorsTheme.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
      * Where are our CSS, JS and other assets?
33 33
      */
34 34
     protected const THEME_DIR  = 'colors';
35
-    public const ASSET_DIR  = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
35
+    public const ASSET_DIR = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
36 36
     protected const STYLESHEET = self::ASSET_DIR . 'style.css';
37 37
 
38 38
     /** @var string[] A list of color palettes */
Please login to merge, or discard this patch.
app/Theme/FabTheme.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
      * Where are our CSS, JS and other assets?
30 30
      */
31 31
     protected const THEME_DIR  = 'fab';
32
-    public const ASSET_DIR  = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
32
+    public const ASSET_DIR = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
33 33
     protected const STYLESHEET = self::ASSET_DIR . 'style.css';
34 34
 
35 35
     /**
Please login to merge, or discard this patch.
app/Theme/CloudsTheme.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
      * Where are our CSS, JS and other assets?
31 31
      */
32 32
     protected const THEME_DIR  = 'clouds';
33
-    public const ASSET_DIR  = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
33
+    public const ASSET_DIR = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
34 34
     protected const STYLESHEET = self::ASSET_DIR . 'style.css';
35 35
 
36 36
     /**
Please login to merge, or discard this patch.
app/Theme/AbstractTheme.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
      * Where are our CSS, JS and other assets?
48 48
      */
49 49
     protected const THEME_DIR  = '_common';
50
-    public const ASSET_DIR  = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
50
+    public const ASSET_DIR = 'themes/' . self::THEME_DIR . '/css-2.0.0/';
51 51
     protected const STYLESHEET = self::ASSET_DIR . 'style.css';
52 52
 
53 53
     protected const PERSON_BOX_CLASSES = [
Please login to merge, or discard this patch.
app/Http/Controllers/ListController.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
                     'ged'      => $tree->name(),
146 146
                     'show_all' => 'yes',
147 147
                 ];
148
-                $show    = $request->get('show', 'surn');
148
+                $show = $request->get('show', 'surn');
149 149
             }
150 150
         } elseif ($surname) {
151 151
             $alpha    = $this->localization_service->initialLetter($surname); // so we can highlight the initial letter
@@ -520,7 +520,7 @@  discard block
 block discarded – undo
520 520
     private function allMedia(Tree $tree, string $folder, string $subfolders, string $sort, string $filter, string $form_type): array
521 521
     {
522 522
         // All files in the folder, plus external files
523
-        $sql  =
523
+        $sql =
524 524
             "SELECT m_id AS xref, m_gedcom AS gedcom" .
525 525
             " FROM `##media`" .
526 526
             " JOIN `##media_file` USING (m_id, m_file)" .
@@ -539,11 +539,11 @@  discard block
 block discarded – undo
539 539
         // Include / exclude subfolders (but always include external)
540 540
         switch ($subfolders) {
541 541
             case 'include':
542
-                $sql    .= " AND (multimedia_file_refn LIKE CONCAT(?, '%') $sql_external)";
542
+                $sql .= " AND (multimedia_file_refn LIKE CONCAT(?, '%') $sql_external)";
543 543
                 $args[] = Database::escapeLike($folder);
544 544
                 break;
545 545
             case 'exclude':
546
-                $sql    .= " AND (multimedia_file_refn LIKE CONCAT(?, '%') AND multimedia_file_refn NOT LIKE CONCAT(?, '%/%') $sql_external)";
546
+                $sql .= " AND (multimedia_file_refn LIKE CONCAT(?, '%') AND multimedia_file_refn NOT LIKE CONCAT(?, '%/%') $sql_external)";
547 547
                 $args[] = Database::escapeLike($folder);
548 548
                 $args[] = Database::escapeLike($folder);
549 549
                 break;
@@ -551,13 +551,13 @@  discard block
 block discarded – undo
551 551
 
552 552
         // Apply search terms
553 553
         if ($filter) {
554
-            $sql    .= " AND (SUBSTRING_INDEX(multimedia_file_refn, '/', -1) LIKE CONCAT('%', ?, '%') OR descriptive_title LIKE CONCAT('%', ?, '%'))";
554
+            $sql .= " AND (SUBSTRING_INDEX(multimedia_file_refn, '/', -1) LIKE CONCAT('%', ?, '%') OR descriptive_title LIKE CONCAT('%', ?, '%'))";
555 555
             $args[] = Database::escapeLike($filter);
556 556
             $args[] = Database::escapeLike($filter);
557 557
         }
558 558
 
559 559
         if ($form_type) {
560
-            $sql    .= " AND source_media_type = ?";
560
+            $sql .= " AND source_media_type = ?";
561 561
             $args[] = $form_type;
562 562
         }
563 563
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -219,7 +219,8 @@  discard block
 block discarded – undo
219 219
                     <li class="wt-initials-list-item d-flex">
220 220
                         <?php if ($count > 0) : ?>
221 221
                             <a href="<?= e(route($route, ['alpha' => $letter, 'ged' => $tree->name()])) ?>" class="wt-initial<?= $letter === $alpha ? ' active' : '' ?> '" title="<?= I18N::number($count) ?>"><?= $this->surnameInitial((string) $letter) ?></a>
222
-                        <?php else : ?>
222
+                        <?php else {
223
+    : ?>
223 224
                             <span class="wt-initial text-muted"><?= $this->surnameInitial((string) $letter) ?></span>
224 225
 
225 226
                         <?php endif ?>
@@ -273,6 +274,7 @@  discard block
 block discarded – undo
273 274
 
274 275
             if ($show === 'indi' || $show === 'surn') {
275 276
                 $surns = $this->individual_list_service->surnames($surname, $alpha, $show_marnm === 'yes', $families, WT_LOCALE, I18N::collation());
277
+}
276 278
                 if ($show === 'surn') {
277 279
                     // Show the surname list
278 280
                     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
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
 
71 71
         $top10 = DB::table('hit_counter')
72 72
             ->where('gedcom_id', '=', $tree->id())
73
-            ->whereIn('page_name', ['individual.php','family.php','source.php','repo.php','note.php','mediaviewer.php'])
73
+            ->whereIn('page_name', ['individual.php', 'family.php', 'source.php', 'repo.php', 'note.php', 'mediaviewer.php'])
74 74
             ->orderByDesc('page_count')
75 75
             ->limit((int) $num)
76 76
             ->pluck('page_count', 'page_parameter');
Please login to merge, or discard this patch.