Check for local actions only

Fixes issue #305

Author: sebastianfeldmann

src/Config.php

@@ -398,7 +398,7 @@ public function getJsonData(): array
         }
 
         foreach (Hooks::getValidHooks() as $hook => $value) {
-            if ($this->hooks[$hook]->hasActions()) {
+            if ($this->hooks[$hook]->hasLocalActions()) {
                 $data[$hook] = $this->hooks[$hook]->getJsonData();
             }
         }

src/Config/Hook.php

@@ -96,6 +96,24 @@ public function hasActions(): bool
         return !empty($this->actions);
     }
 
+    /**
+     * Check if a hook config has local actions that were not loaded from other config files
+     *
+     * @return bool
+     */
+    public function hasLocalActions(): bool
+    {
+        if (!$this->hasActions()) {
+            return false;
+        }
+        foreach ($this->actions as $action) {
+            if (!$action->isIncluded()) {
+                return true;
+            }
+        }
+        return false;
+    }
+
     /**
      * Add an action to the list
      *

tests/unit/Config/HookTest.php

@@ -53,6 +53,27 @@ public function testAddAction(): void
         $this->assertCount(1, $config['actions']);
     }
 
+    public function testCanDetermineIfItHasActionsThatWereNotIncluded(): void
+    {
+        $localAction = new Action('\\Foo\\Bar');
+        $includedAction = new Action('\\Foo\\Bar');
+        $includedAction->markIncluded();
+
+        $hook   = new Hook('pre-commit');
+        $hook->addAction($localAction, $includedAction);
+        $this->assertTrue($hook->hasLocalActions());
+    }
+
+    public function testCanDetermineIfItHasIncludedActionsOnly(): void
+    {
+        $includedAction = new Action('\\Foo\\Bar');
+        $includedAction->markIncluded();
+
+        $hook   = new Hook('pre-commit');
+        $hook->addAction($includedAction);
+        $this->assertFalse($hook->hasLocalActions());
+    }
+
     public function testAddMultiAction(): void
     {
         $hook   = new Hook('pre-commit');