Passed
Push — analysis-z4G20e ( fbc4ca )
by Greg
09: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.
app/Http/Controllers/AutocompleteController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@
 block discarded – undo
103 103
         $pages = [];
104 104
 
105 105
         // Escape the query for MySQL and PHP, converting spaces to wildcards.
106
-        $like_query  = strtr($query, [
106
+        $like_query = strtr($query, [
107 107
             '_' => '\\_',
108 108
             '%' => '\\%',
109 109
             ' ' => '%',
Please login to merge, or discard this patch.
app/Services/SearchService.php 1 patch
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -185,15 +185,15 @@
 block discarded – undo
185 185
     }
186 186
 
187 187
     /**
188
-    * Search for sources by name.
189
-    *
190
-    * @param Tree   $tree
191
-    * @param string $search
192
-    * @param int    $offset
193
-    * @param int    $limit
194
-    *
195
-    * @return Collection|Source[]
196
-    */
188
+     * Search for sources by name.
189
+     *
190
+     * @param Tree   $tree
191
+     * @param string $search
192
+     * @param int    $offset
193
+     * @param int    $limit
194
+     *
195
+     * @return Collection|Source[]
196
+     */
197 197
     public function searchSourcesByName(Tree $tree, string $search, int $offset = 0, int $limit = PHP_INT_MAX): Collection
198 198
     {
199 199
         $query = DB::table('sources')
Please login to merge, or discard this patch.