Fix #1141 referencing DI element when not initialized

getgrav · Jun 21, 2017 · 09f4d2b · 09f4d2b
1 parent 5bf721c
commit 09f4d2b
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/admin.php b/admin.php
@@ -523,18 +523,17 @@ public function onTwigSiteVariables()
      */
     public function onShutdown()
     {
-        // Just so we know that we're in this debug mode
-        if ($this->config->get('plugins.admin.popularity.enabled')) {
-
-            // Only track non-admin
-            if (!$this->active) {
+        if ($this->active) {
+            //only activate when Admin is active
+            if ($this->shouldLoadAdditionalFilesInBackground()) {
+                $this->loadAdditionalFilesInBackground();
+            }
+        } else {
+            //if popularity is enabled, track non-admin hits
+            if ($this->config->get('plugins.admin.popularity.enabled')) {
                 $this->popularity->trackHit();
             }
         }
-
-        if ($this->grav['admin']->shouldLoadAdditionalFilesInBackground()) {
-            $this->grav['admin']->loadAdditionalFilesInBackground();
-        }
     }
 
     /**