View GitHub Repository silverstripe-simple-template-caching

Schedule Inspection
Subscribe
Passed
Push — master ( ff6b8b...076954 )
by Nicolaas
09:30
created
Status
Category
Toggle Patches
src/Extensions/DataObjectExtension.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -18,9 +18,9 @@  discard block
 
 block discarded – undo
18 18 
         // NB.
19 19 
         // if the dataobject has the versioned extension then the cache should be invalidated onAfterPublish
20 20 
         // hasStages function is part of the Versioned class so safe to check here
21 
-        if (! $owner->hasExtension(Versioned::class)) {
21 
+        if (!$owner->hasExtension(Versioned::class)) {
22 22 
             $this->doUpdateCache($className);
23 
-        } elseif (! $owner->hasStages()) {
23 
+        } elseif (!$owner->hasStages()) {
24 24 
             $this->doUpdateCache($className);
25 25 
         }
26 26 
     }
 
@@ -87,6 +87,6 @@  discard block
 
 block discarded – undo
87 87 
     {
88 88 
         $excludedClasses = Config::inst()->get(self::class, 'excluded_classes');
89 89
90 
-        return ! in_array($className, $excludedClasses, true);
90 
+        return !in_array($className, $excludedClasses, true);
91 91 
     }
92 92 
 }
Please login to merge, or discard this patch.
src/Extensions/PageControllerExtension.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -28,7 +28,7 @@  discard block
 
 block discarded – undo
28 28 
             self::$_can_cache_content_string = '';
29 29 
             if ($this->owner->hasMethod('canCachePage')) {
30 30 
                 self::$_can_cache_content_string = $this->owner->canCachePage() ? '' : 'can-no-cache-' . $this->owner->ID;
31 
-                if (! $this->canCacheCheck()) {
31 
+                if (!$this->canCacheCheck()) {
32 32 
                     return false;
33 33 
                 }
34 34 
             }
 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37 
             $action = $this->owner->request->param('Action');
38 38 
             if ($action) {
39 39 
                 self::$_can_cache_content_string .= $action;
40 
-                if (! $this->canCacheCheck()) {
40 
+                if (!$this->canCacheCheck()) {
41 41 
                     return false;
42 42 
                 }
43 43 
             }
 
@@ -45,7 +45,7 @@  discard block
 
 block discarded – undo
45 45 
             // id
46 46 
             if ($id) {
47 47 
                 self::$_can_cache_content_string .= $id;
48 
-                if (! $this->canCacheCheck()) {
48 
+                if (!$this->canCacheCheck()) {
49 49 
                     return false;
50 50 
                 }
51 51 
             }
 
@@ -58,7 +58,7 @@  discard block
 
 block discarded – undo
58 58 
                     } elseif (is_numeric($item)) {
59 59 
                         self::$_can_cache_content_string .= $item;
60 60 
                     }
61 
-                    if (! $this->canCacheCheck()) {
61 
+                    if (!$this->canCacheCheck()) {
62 62 
                         return false;
63 63 
                     }
64 64 
                 }
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
             $member = Security::getCurrentUser();
69 69 
             if ($member && $member->exists()) {
70 70 
                 self::$_can_cache_content_string .= $member->ID;
71 
-                if (! $this->canCacheCheck()) {
71 
+                if (!$this->canCacheCheck()) {
72 72 
                     return false;
73 73 
                 }
74 74 
             }
Please login to merge, or discard this patch.