sunnysideup / silverstripe-site-wide-search

src/Admin/SearchAdmin.php

Indentation +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
                 _t(__CLASS__ . '.ReplaceToggle', 'Replace with ... (optional - make a backup first!)'),
                 [
                     (new CheckboxField('ApplyReplace', 'Run replace (please make sure to make a backup first!)', $this->applyReplace))
-                      ->setDescription('Check this to replace the searched value set above with its replacement value. Note that searches ignore uppercase / lowercase, but replace actions will only search and replace values with the same upper / lowercase.'),
+                        ->setDescription('Check this to replace the searched value set above with its replacement value. Note that searches ignore uppercase / lowercase, but replace actions will only search and replace values with the same upper / lowercase.'),
                     (new TextField('ReplaceWith', 'Replace (optional - careful!)', $this->replace ?? ''))
                         ->setAttribute('placeholder', 'e.g. contract - make sure to also tick checkbox below'),
                 ]

Spacing +17 added lines, -17 removed lines

@@ -57,8 +57,8 @@ 
 
     protected function init()
     {
-        if($this->request->param('Action')) {
-            if(empty($this->request->postVars())) {
+        if ($this->request->param('Action')) {
+            if (empty($this->request->postVars())) {
                 $this->redirect('/admin/find');
             }
         }
@@ -111,7 +111,7 @@ 
 
         if (!$this->getRequest()->requestVar('Keywords')) {
             $lastResults = $this->lastSearchResults();
-            if($lastResults) {
+            if ($lastResults) {
                 $resultsTitle = 'Last Results';
             } else {
                 $resultsTitle = 'Last Edited';
@@ -181,9 +181,9 @@ 
         Environment::increaseTimeLimitTo(300);
         Environment::setMemoryLimitMax(-1);
         Environment::increaseMemoryLimitTo(-1);
-        if(empty($this->rawData)) {
+        if (empty($this->rawData)) {
             $lastResults = $this->lastSearchResults();
-            if($lastResults) {
+            if ($lastResults) {
                 return $lastResults;
             }
         }
@@ -208,18 +208,18 @@ 
             ->setWordsAsString($this->keywords)
             ->getLinks()
         ;
-        if($results->count() === 1) {
+        if ($results->count() === 1) {
             $result = $results->first();
-            if($result->HasCMSEditLink) {
+            if ($result->HasCMSEditLink) {
                 // files do not re-redirect nicely...
-                if(!in_array(File::class, ClassInfo::ancestry($result->ClassName), true)) {
+                if (!in_array(File::class, ClassInfo::ancestry($result->ClassName), true)) {
                     $this->redirect($result->CMSEditLink());
                 }
             }
         }
         // Accessing the session
         $session = $this->getRequest()->getSession();
-        if($session) {
+        if ($session) {
             $session->set('QuickSearchLastResults', serialize($results->toArray()));
         }
         return $results;
@@ -250,16 +250,16 @@ 
     {
         // Accessing the session
         $session = $this->getRequest()->getSession();
-        if($this->quickSearchType) {
+        if ($this->quickSearchType) {
             $session->set('QuickSearchType', $this->quickSearchType);
-        } elseif($session) {
+        } elseif ($session) {
             $this->quickSearchType = $session->get('QuickSearchType');
-            if(isset($_GET['flush'])) {
+            if (isset($_GET['flush'])) {
                 $this->quickSearchType = '';
                 $session->set('QuickSearchType', '');
             }
         }
-        if(!$this->quickSearchType) {
+        if (!$this->quickSearchType) {
             $this->quickSearchType = $this->Config()->get('default_quick_search_type');
         }
         return (string) $this->quickSearchType;
@@ -269,15 +269,15 @@ 
     {
         // Accessing the session
         $session = $this->getRequest()->getSession();
-        if($session) {
-            if(isset($_GET['flush'])) {
+        if ($session) {
+            if (isset($_GET['flush'])) {
                 $session->clear('QuickSearchLastResults');
             } else {
                 $data = $session->get('QuickSearchLastResults');
-                if($data) {
+                if ($data) {
                     $array = unserialize($data);
                     $al = ArrayList::create();
-                    foreach($array as $item) {
+                    foreach ($array as $item) {
                         $al->push($item);
                     }
                     return $al;

src/QuickSearches/QuickSearchBaseClass.php

Spacing +3 added lines, -3 removed lines

@@ -52,10 +52,10 @@
             'limited' => 'Limited search',
         ];
         $availableSearchClasses = self::available_quick_searches();
-        if(!empty($availableSearchClasses) > 0) {
-            foreach($availableSearchClasses as $availableSearchClass) {
+        if (!empty($availableSearchClasses) > 0) {
+            foreach ($availableSearchClasses as $availableSearchClass) {
                 $singleton = Injector::inst()->get($availableSearchClass);
-                if($singleton->isEnabled()) {
+                if ($singleton->isEnabled()) {
                     $array[$availableSearchClass] = $singleton->getTitle();
                 }
             }

src/Helpers/FindClassesAndFields.php

Spacing +4 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
 
     public static function inst(string $baseClass)
     {
-        if(self::$singleton === null) {
+        if (self::$singleton === null) {
             self::$singleton = Injector::inst()->get(static::class);
         }
         self::$singleton->setBaseClass($baseClass);
@@ -128,12 +128,12 @@ 
         foreach ($this->cache['AllValidFields'][$className] as $name => $type) {
             if ($this->isValidFieldType($type, $className, $name)) {
                 $array[] = $name;
-            } elseif(in_array($name, $includedFields, true)) {
+            } elseif (in_array($name, $includedFields, true)) {
                 $array[] = $name;
             }
         }
-        if(isset($includedClassFieldCombos[$className])) {
-            foreach($includedClassFieldCombos[$className] as $name) {
+        if (isset($includedClassFieldCombos[$className])) {
+            foreach ($includedClassFieldCombos[$className] as $name) {
                 $array[] = $name;
             }
         }

src/Api/SearchApi.php

Spacing +20 added lines, -20 removed lines

@@ -112,13 +112,13 @@ 
 
     public function setQuickSearchType(string $nameOrType): SearchApi
     {
-        if($nameOrType === 'all') {
+        if ($nameOrType === 'all') {
             $this->isQuickSearch = false;
             $this->quickSearchType = '';
-        } elseif($nameOrType === 'limited') {
+        } elseif ($nameOrType === 'limited') {
             $this->isQuickSearch = true;
             $this->quickSearchType = '';
-        } elseif(class_exists($nameOrType)) {
+        } elseif (class_exists($nameOrType)) {
             $this->quickSearchType = $nameOrType;
             $object = Injector::inst()->get($nameOrType);
             $this->setIncludedClasses($object->getClassesToSearch());
@@ -149,7 +149,7 @@ 
 
     public function setBaseClass(string $class): SearchApi
     {
-        if(class_exists($class)) {
+        if (class_exists($class)) {
             $this->baseClass = $class;
         }
 
@@ -224,7 +224,7 @@ 
 
     public function buildCache(?string $word = ''): SearchApi
     {
-        if($this->cacheHasBeenBuilt !== $word) {
+        if ($this->cacheHasBeenBuilt !== $word) {
             $this->getLinksInner($word);
             $this->cacheHasBeenBuilt = $word;
         }
@@ -255,7 +255,7 @@ 
     public function doReplacement(string $word, string $replace): int
     {
         $count = 0;
-        if($word) {
+        if ($word) {
             $this->buildCache($word);
             $replace = $this->securityCheckInput($replace);
             foreach ($this->objects as $item) {
@@ -263,7 +263,7 @@ 
                 if ($item->canEdit()) {
                     $fields = $this->getAllValidFields($className);
                     foreach ($fields as $field) {
-                        if(!$this->includeFieldTest($className, $field)) {
+                        if (!$this->includeFieldTest($className, $field)) {
                             continue;
                         }
                         $new = str_replace($word, $replace, $item->{$field});
@@ -328,7 +328,7 @@ 
         $this->workOutInclusionsAndExclusions();
 
         // important to do this first
-        if($word) {
+        if ($word) {
             $this->setWordsAsString($word);
         }
         $this->workOutWordsForSearching();
@@ -341,7 +341,7 @@ 
         if (count($this->words)) {
             foreach ($this->getAllDataObjects() as $className) {
 
-                if(!$this->includeClassTest($className)) {
+                if (!$this->includeClassTest($className)) {
                     continue;
                 }
 
@@ -349,7 +349,7 @@ 
                 $fields = $this->getAllValidFields($className);
                 $filterAny = [];
                 foreach ($fields as $field) {
-                    if(!$this->includeFieldTest($className, $field)) {
+                    if (!$this->includeFieldTest($className, $field)) {
                         continue;
                     }
                     $filterAny[$field . ':PartialMatch'] = $this->words;
@@ -363,7 +363,7 @@ 
                         DB::alteration_message(' ... Filter: ' . implode(', ', array_keys($filterAny)));
                     }
                     $defaultList = $this->getDefaultList($className);
-                    if(empty($defaultList)) {
+                    if (empty($defaultList)) {
                         $array[$className] = $className::get();
                     }
                     $array[$className] = $array[$className]->filter(['ClassName' => $className]);
@@ -406,7 +406,7 @@ 
         $array = [];
         $classNames = $this->getAllDataObjects();
         foreach ($classNames as $className) {
-            if($this->includeClassTest($className)) {
+            if ($this->includeClassTest($className)) {
                 $array[$className] = $className::get()
                     ->filter('LastEdited:GreaterThan', date('Y-m-d H:i:s', $threshold))
                     ->sort(['LastEdited' => 'DESC'])
@@ -454,7 +454,7 @@ 
                         ->limit($limit)
                     ;
                     foreach ($items as $item) {
-                        if(isset($fullListCheck[$item->ClassName][$item->ID])) {
+                        if (isset($fullListCheck[$item->ClassName][$item->ID])) {
                             continue;
                         }
                         if ($item->canView()) {
@@ -487,7 +487,7 @@ 
 
         $items = $this->turnArrayIntoObjects($matches);
         foreach ($items as $item) {
-            if($item->canView()) {
+            if ($item->canView()) {
                 $link = $finder->getLink($item, $this->excludedClasses);
                 $cmsEditLink = $item->canEdit() ? $finder->getCMSEditLink($item) : '';
                 $list->push(
@@ -511,7 +511,7 @@ 
         }
         $finder->saveCache();
 
-        if(!empty($this->sortOverride)) {
+        if (!empty($this->sortOverride)) {
             return $list->sort($this->sortOverride);
         } else {
             return $list->sort(['SiteWideSearchSortValue' => 'ASC']);
@@ -664,13 +664,13 @@ 
 
     protected function includeClassTest(string $className): bool
     {
-        if(count($this->includedClassesWithSubClassess) && !in_array($className, $this->includedClassesWithSubClassess, true)) {
+        if (count($this->includedClassesWithSubClassess) && !in_array($className, $this->includedClassesWithSubClassess, true)) {
             if ($this->debug) {
                 DB::alteration_message(' ... Skipping as not included ' . $className);
             }
             return false;
         }
-        if(count($this->excludedClassesWithSubClassess) && in_array($className, $this->excludedClassesWithSubClassess, true)) {
+        if (count($this->excludedClassesWithSubClassess) && in_array($className, $this->excludedClassesWithSubClassess, true)) {
             if ($this->debug) {
                 DB::alteration_message(' ... Skipping as excluded ' . $className);
             }
@@ -685,9 +685,9 @@ 
 
     protected function includeFieldTest(string $className, string $field): bool
     {
-        if(isset($this->includedClassFieldCombos[$className][$field])) {
+        if (isset($this->includedClassFieldCombos[$className][$field])) {
             return true;
-        } elseif(count($this->includedFields)) {
+        } elseif (count($this->includedFields)) {
             return in_array($field, $this->includedFields, true);
         } elseif (count($this->excludedFields)) {
             return in_array($field, $this->includedFields, true) ? false : true;
@@ -699,7 +699,7 @@ 
     protected function includeSubClasses(array $classes): array
     {
         $toAdd = [];
-        foreach($classes as $class) {
+        foreach ($classes as $class) {
             $toAdd = array_merge($toAdd, ClassInfo::subclassesFor($class, false));
         }
         return array_unique(array_merge($classes, $toAdd));

src/Helpers/FindEditableObjects.php

Spacing +4 added lines, -4 removed lines

@@ -246,14 +246,14 @@ 
 
             if (!isset($this->relationTypesCovered[$relType])) {
                 $rels = null;
-                if($dataObject->hasMethod($relationName)) {
+                if ($dataObject->hasMethod($relationName)) {
                     $rels = $dataObject->{$relationName}();
                 } else {
                     user_error('Relation ' . print_r($relationName, 1) . ' does not exist on ' . $dataObject->ClassName . ' Relations are: ' . print_r($this->getRelations($dataObject), 1), E_USER_NOTICE);
                 }
                 if ($rels) {
                     if ($rels instanceof DataList) {
-                        if(!$rels instanceof UnsavedRelationList) {
+                        if (!$rels instanceof UnsavedRelationList) {
                             $rels = $rels->first();
                         }
                     }
@@ -308,8 +308,8 @@ 
      */
     protected function classCanBeIncluded(string $dataObjectClassName): bool
     {
-        if(count($this->excludedClasses)) {
-            if(!class_exists($dataObjectClassName)) {
+        if (count($this->excludedClasses)) {
+            if (!class_exists($dataObjectClassName)) {
                 return false;
             }
             return !in_array($dataObjectClassName, $this->excludedClasses, true);