Fixed Hook Registration

src/Hooks/UsedEmptyHook.php

@@ -0,0 +1,29 @@
+<?php declare(strict_types=1);
+
+namespace Marartner\PsalmNoEmpty\Hooks;
+
+use Marartner\PsalmNoEmpty\Issue\UsedEmptyConstruct;
+use PhpParser\Node\Expr;
+use Psalm\Codebase;
+use Psalm\CodeLocation;
+use Psalm\Context;
+use Psalm\IssueBuffer;
+use Psalm\Plugin\Hook\AfterExpressionAnalysisInterface;
+use Psalm\StatementsSource;
+
+final class UsedEmptyHook implements AfterExpressionAnalysisInterface
+{
+    public static function afterExpressionAnalysis(Expr $expr, Context $context, StatementsSource $statements_source, Codebase $codebase, array &$file_replacements = []): ?bool
+    {
+        if ($expr instanceof Expr\Empty_) {
+            IssueBuffer::accepts(
+                new UsedEmptyConstruct(
+                    'Do not use the empty() construct',
+                    new CodeLocation($statements_source->getSource(), $expr)
+                ),
+                $statements_source->getSuppressedIssues()
+            );
+        }
+        return null;
+    }
+}

src/Plugin.php

@@ -1,36 +1,20 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace Marartner\PsalmNoEmpty;
 
-use Marartner\PsalmNoEmpty\Issue\UsedEmptyConstruct;
-use PhpParser\Node\Expr;
-use Psalm\Codebase;
-use Psalm\CodeLocation;
-use Psalm\Context;
-use Psalm\IssueBuffer;
-use Psalm\Plugin\Hook\AfterExpressionAnalysisInterface;
-use Psalm\StatementsSource;
+use Marartner\PsalmNoEmpty\Hooks\UsedEmptyHook;
 use SimpleXMLElement;
 use Psalm\Plugin\PluginEntryPointInterface;
 use Psalm\Plugin\RegistrationInterface;
+use function class_exists;
 
-class Plugin implements PluginEntryPointInterface, AfterExpressionAnalysisInterface
+class Plugin implements PluginEntryPointInterface
 {
     public function __invoke(RegistrationInterface $registration, ?SimpleXMLElement $config = null): void
     {
-        // TODO: Implement __invoke() method.
-    }
-
-    public static function afterExpressionAnalysis(Expr $expr, Context $context, StatementsSource $statements_source, Codebase $codebase, array &$file_replacements = []): ?bool
-    {
-        if ($expr instanceof Expr\Empty_) {
-            IssueBuffer::accepts(
-                new UsedEmptyConstruct(
-                    'Do not use the empty() construct',
-                    new CodeLocation($statements_source->getSource(), $expr)
-                ),
-                $statements_source->getSuppressedIssues()
-            );
+        if(class_exists(UsedEmptyHook::class)){
+            $registration->registerHooksFromClass(UsedEmptyHook::class);
         }
     }
 }
+