View GitHub Repository silverstripe-simple-template-caching

Schedule Inspection
Subscribe
Passed
Push — master ( ba9102...299015 )
by Nicolaas
11:04
created
Status
Category
Toggle Patches
src/Extensions/PageControllerExtension.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -54,38 +54,38 @@  discard block
 
 block discarded – undo
54 54 
             //action
55 55 
             $action = $owner->request->param('Action');
56 56 
             if ($action) {
57 
-                self::$_can_cache_content_string .= 'UA'.$action;
57 
+                self::$_can_cache_content_string .= 'UA' . $action;
58 58 
             }
59 59
60 60 
             // id
61 61 
             $id = $owner->request->param('ID');
62 62 
             if ($id) {
63 
-                self::$_can_cache_content_string .= 'UI'.$id;
63 
+                self::$_can_cache_content_string .= 'UI' . $id;
64 64 
             }
65 65
66 66 
             // otherid
67 67 
             $otherId = $owner->request->param('OtherID');
68 68 
             if ($otherId) {
69 
-                self::$_can_cache_content_string .= 'UI'.$otherId;
69 
+                self::$_can_cache_content_string .= 'UI' . $otherId;
70 70 
             }
71 71
72 72 
             //request vars
73 73 
             $requestVars = $owner->request->requestVars();
74 74 
             if ($requestVars) {
75 75 
                 foreach ($owner->request->requestVars() as $key => $item) {
76 
-                    self::$_can_cache_content_string .= serialize($key.$item);
76 
+                    self::$_can_cache_content_string .= serialize($key . $item);
77 77 
                 }
78 78 
             }
79 79
80 80 
             //member
81 81 
             $member = Security::getCurrentUser();
82 82 
             if ($member && $member->exists()) {
83 
-                if(Config::inst()->get(self::class, 'unique_cache_for_each_member')) {
84 
-                    self::$_can_cache_content_string .= 'UM'.$member->ID;
85 
-                } elseif(Config::inst()->get(self::class, 'unique_cache_for_each_member_group_combo')) {
83 
+                if (Config::inst()->get(self::class, 'unique_cache_for_each_member')) {
84 
+                    self::$_can_cache_content_string .= 'UM' . $member->ID;
85 
+                } elseif (Config::inst()->get(self::class, 'unique_cache_for_each_member_group_combo')) {
86 86 
                     $groupIds = $member->Groups()->columnUnique();
87 87 
                     sort($groupIds, SORT_NUMERIC);
88 
-                    self::$_can_cache_content_string .= 'UG'.implode(',', $groupIds);
88 
+                    self::$_can_cache_content_string .= 'UG' . implode(',', $groupIds);
89 89 
                 }
90 90 
             }
91 91 
             // crucial
 
@@ -107,7 +107,7 @@  discard block
 
 block discarded – undo
107 107
108 108 
     public function HasCacheKeyContent(): bool
109 109 
     {
110 
-        if($this->getOwner()->NeverCachePublicly) {
110 
+        if ($this->getOwner()->NeverCachePublicly) {
111 111 
             return false;
112 112 
         }
113 113 
         return $this->HasCacheKeys();
 
@@ -136,7 +136,7 @@  discard block
 
 block discarded – undo
136 136 
     public function CacheKeyContent(?bool $ignoreHasCacheKeys = false): string
137 137 
     {
138 138 
         $owner = $this->getOwner();
139 
-        if($owner->NeverCachePublicly) {
139 
+        if ($owner->NeverCachePublicly) {
140 140 
             return $this->getRandomKey();
141 141 
         }
142 142 
         $cacheKey = $this->CacheKeyGenerator('C');
 
@@ -157,7 +157,7 @@  discard block
 
 block discarded – undo
157 157 
                 $string .= '_ID_' . $owner->ID;
158 158 
             }
159 159 
         } else {
160 
-            $string = 'NOT_CACHED__ID_' .  $this->getRandomKey();
160 
+            $string = 'NOT_CACHED__ID_' . $this->getRandomKey();
161 161 
         }
162 162
163 163 
         return $string;
Please login to merge, or discard this patch.