Passed
Push — analysis-zEMnvx ( 3aa707 )
by Greg
09:38 queued 01:39
created
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.