sunnysideup / silverstripe-site-wide-search

src/Helpers/FindEditableObjects.php

Spacing +2 added lines, -2 removed lines

@@ -303,8 +303,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);

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 +18 added lines, -18 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;
         }
 
@@ -267,14 +267,14 @@ 
         $count = 0;
         // we should have these already.
         $word = $this->securityCheckInput($word);
-        if($word) {
+        if ($word) {
             // important to do this first
             foreach ($this->objects as $item) {
                 $className = $item->ClassName;
                 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});
@@ -351,7 +351,7 @@ 
         if (count($this->words)) {
             foreach ($this->getAllDataObjects() as $className) {
 
-                if(!$this->includeClassTest($className)) {
+                if (!$this->includeClassTest($className)) {
                     continue;
                 }
 
@@ -359,7 +359,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;
@@ -373,7 +373,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]);
@@ -416,7 +416,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'])
@@ -463,7 +463,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()) {
@@ -497,7 +497,7 @@ 
         $items = $this->turnArrayIntoObjects($matches);
         foreach ($items as $item) {
             $link = $finder->getLink($item, $this->excludedClasses);
-            if($item->canView()) {
+            if ($item->canView()) {
                 $cmsEditLink = $item->canEdit() ? $finder->getCMSEditLink($item) : '';
                 $list->push(
                     ArrayData::create(
@@ -519,7 +519,7 @@ 
         }
         $finder->saveCache();
 
-        if(!empty($this->sortOverride)) {
+        if (!empty($this->sortOverride)) {
             return $list->sort($this->sortOverride);
         } else {
             return $list->sort(['SiteWideSearchSortValue' => 'ASC']);
@@ -676,13 +676,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);
             }
@@ -697,9 +697,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;
@@ -711,7 +711,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));