sunnysideup / silverstripe-simple-template-caching

src/Extensions/SimpleTemplateCachingSiteConfigExtension.php

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
     public function updateCMSFields(FieldList $fields)
     {
         $name = '';
-        if(class_exists($this->getOwner()->ClassNameLastEdited)) {
+        if (class_exists($this->getOwner()->ClassNameLastEdited)) {
             $name = Injector::inst()->get($this->getOwner()->ClassNameLastEdited)->i18n_singular_name();
         }
         $fields->addFieldsToTab(
@@ -43,16 +43,16 @@ 
                 DatetimeField::create('CacheKeyLastEdited', 'Content Last Edited')
                     ->setRightTitle('The frontend template cache will be invalidated every time this value changes. It changes every time anything is changed in the database.'),
                 ReadonlyField::create('ClassNameLastEdited', 'Last class updated')
-                    ->setRightTitle('Last object updated. The name of this object is: '.$name),
+                    ->setRightTitle('Last object updated. The name of this object is: ' . $name),
             ]
         );
-        if($this->getOwner()->RecordCacheUpdates) {
+        if ($this->getOwner()->RecordCacheUpdates) {
             $fields->addFieldsToTab(
                 'Root.Caching',
                 [
                     GridField::create(
                         'ObjectsUpdated',
-                        'Last '.self::MAX_OBJECTS_UPDATED.' objects updated',
+                        'Last ' . self::MAX_OBJECTS_UPDATED . ' objects updated',
                         ObjectsUpdated::get()->limit(self::MAX_OBJECTS_UPDATED),
                         GridFieldConfig_RecordViewer::create()
                     )
@@ -64,30 +64,30 @@ 
     public static function site_cache_key(): string
     {
         $obj = SiteConfig::current_site_config();
-        if($obj->HasCaching) {
-            return (string) 'ts_'.strtotime($obj->CacheKeyLastEdited);
+        if ($obj->HasCaching) {
+            return (string) 'ts_' . strtotime($obj->CacheKeyLastEdited);
         } else {
-            return  (string) 'ts_'.time();
+            return  (string) 'ts_' . time();
         }
     }
 
     public static function update_cache_key(?string $className = '')
     {
         $obj = SiteConfig::current_site_config();
-        if($obj->HasCaching) {
+        if ($obj->HasCaching) {
             DB::query('
                 UPDATE "SiteConfig"
                 SET
                     "CacheKeyLastEdited" = \'' . DBDatetime::now()->Rfc2822() . '\',
-                    "ClassNameLastEdited" = \'' . addslashes($className). '\'
-                WHERE ID = '.$obj->ID.'
+                    "ClassNameLastEdited" = \'' . addslashes($className) . '\'
+                WHERE ID = '.$obj->ID . '
             ;');
         }
-        if($obj->RecordCacheUpdates) {
+        if ($obj->RecordCacheUpdates) {
             $recordId = Injector::inst()
-                ->create(ObjectsUpdated::class, ['ClassNameLastEdited' => $className])
+                ->create(ObjectsUpdated::class, [ 'ClassNameLastEdited' => $className ])
                 ->write();
-            DB::query('DELETE FROM ObjectsUpdated WHERE ID < '.(Int) ($recordId - self::MAX_OBJECTS_UPDATED));
+            DB::query('DELETE FROM ObjectsUpdated WHERE ID < ' . (Int) ($recordId - self::MAX_OBJECTS_UPDATED));
         }
     }
 }

src/Extensions/DataObjectExtension.php

Spacing +4 added lines, -4 removed lines

@@ -20,9 +20,9 @@ 
         // NB.
         // if the dataobject has the versioned extension then the cache should be invalidated onAfterPublish
         // hasStages function is part of the Versioned class so safe to check here
-        if (! $owner->hasExtension(Versioned::class)) {
+        if (!$owner->hasExtension(Versioned::class)) {
             $this->doUpdateCache($className);
-        } elseif (! $owner->hasStages()) {
+        } elseif (!$owner->hasStages()) {
             $this->doUpdateCache($className);
         }
     }
@@ -89,11 +89,11 @@ 
     private function canUpdateCache($className): bool
     {
         // we want to always avoid this to avoid a loop.
-        if($className === ObjectsUpdated::class) {
+        if ($className === ObjectsUpdated::class) {
             return false;
         }
         $excludedClasses = (array) Config::inst()->get(self::class, 'excluded_classes_for_caching');
 
-        return ! in_array($className, $excludedClasses, true);
+        return !in_array($className, $excludedClasses, true);
     }
 }