style: Refactor ButtonsResolver for cleariy

Author: jissereitsma

Ui/Button/ButtonsResolver.php

@@ -3,6 +3,8 @@
 
 namespace Loki\AdminComponents\Ui\Button;
 
+use Loki\AdminComponents\Component\Form\FormRepository;
+use Loki\AdminComponents\Component\Grid\GridRepository;
 use Loki\AdminComponents\Provider\FormProviderInterface;
 use Magento\Framework\DataObject;
 use Magento\Framework\View\Element\AbstractBlock;
@@ -18,30 +20,54 @@ public function __construct(
      * @return Button[]
      */
     public function resolve(
+        FormRepository|GridRepository $repository,
         AbstractBlock $block,
-        object $provider,
-        mixed $item
     ): array {
+        $provider = $repository->getProvider();
         if ($provider instanceof FormProviderInterface) {
             return $provider->getForm()->getButtons();
         }
 
+        $buttons = $this->getButtonsFromBlock($block);
+        if (!empty($buttons)) {
+            return $buttons;
+        }
+
+        if ($repository instanceof FormRepository) {
+            return $this->getDefaultFormButtons($repository);
+        }
+
+        if ($repository instanceof GridRepository) {
+            return $this->getDefaultGridButtons($repository);
+        }
+
+        return [];
+    }
+
+    private function getButtonsFromBlock(AbstractBlock $block): array
+    {
         $buttonDefinitions = $block->getButtons();
-        if (!empty($buttonDefinitions)) {
-            $buttons = [];
-            foreach ($buttonDefinitions as $buttonDefinition) {
-                $buttons[] = $this->buttonFactory->create(
-                    method: (string)$buttonDefinition['method'],
-                    label: (string)$buttonDefinition['label'],
-                    cssClass: isset($buttonDefinition['cssClass']) ? (string)$buttonDefinition['cssClass'] : '',
-                    url: isset($buttonDefinition['url']) ? (string)$buttonDefinition['url'] : '',
-                    primary: $buttonDefinition['primary'] ?? false,
-                );
-            }
+        if (empty($buttonDefinitions)) {
+            return [];
+        }
 
-            return $buttons;
+        $buttons = [];
+        foreach ($buttonDefinitions as $buttonDefinition) {
+            $buttons[] = $this->buttonFactory->create(
+                method: (string)$buttonDefinition['method'],
+                label: (string)$buttonDefinition['label'],
+                cssClass: isset($buttonDefinition['cssClass']) ? (string)$buttonDefinition['cssClass'] : '',
+                url: isset($buttonDefinition['url']) ? (string)$buttonDefinition['url'] : '',
+                primary: $buttonDefinition['primary'] ?? false,
+            );
         }
 
+        return $buttons;
+    }
+
+    private function getDefaultFormButtons(FormRepository $repository): array
+    {
+        $item = $repository->getItem();
         if ($item instanceof DataObject && $item->getId() > 0) {
             return [
                 $this->buttonFactory->createCloseAction(),
@@ -59,4 +85,9 @@ public function resolve(
             //$this->buttonFactory->createSaveContinueAction(),
         ];
     }
+
+    private function getDefaultGridButtons(GridRepository $repository): array
+    {
+        return [];
+    }
 }