Merge pull request #18 from helsingborg-stad/fix/do-not-check-config-before-hooks

fix: do not check config too early

Author: sebastianthulin

source/php/Asset/AssetInterface.php

@@ -10,6 +10,7 @@
 
 interface AssetInterface
 {
+  public function shouldEnqueue(): bool;
   public function getHook(): string;
   public function getHandle(): string;
   public function getFilename(): string;

source/php/Asset/AssetRegistry.php

@@ -104,6 +104,10 @@ public function register(): void
 
     public function enqueue(): void
     {
+        if($this->isEnabled() === false) {
+            return;
+        }
+
         $filename = $this->getFilename();
 
         if ($this->getType($filename) === 'css') {

source/php/Asset/ContextDetection.php

@@ -6,6 +6,11 @@
 
 class ContextDetection extends AssetRegistry implements AssetInterface
 {
+  public function shouldEnqueue(): bool
+  {
+    return $this->config->isContextCheckEnabled();
+  }
+
   public function getHook(): string
   {
     return 'wp_enqueue_scripts';

source/php/Asset/EditorHighlight.php

@@ -6,6 +6,11 @@
 
 class EditorHighlight extends AssetRegistry implements AssetInterface
 {
+  public function shouldEnqueue(): bool
+  {
+    return true;
+  }
+
   public function getHook(): string
   {
     return 'mce_external_plugins';

source/php/Asset/FrontendStyles.php

@@ -6,6 +6,11 @@
 
 class FrontendStyles extends AssetRegistry implements AssetInterface
 {
+  public function shouldEnqueue(): bool
+  {
+    return true;
+  }
+
   public function getHook(): string
   {
     return 'wp_enqueue_scripts';