ArdentParameterBundle.php

@@ -0,0 +1,13 @@
+<?php
+
+
+namespace Ardent\ParameterBundle;
+
+use Symfony\Component\HttpKernel\Bundle\Bundle;
+
+/**
+ * Bundle.
+ */
+class ArdentParameterBundle extends Bundle
+{
+}

Controller/BaseController.php

@@ -9,17 +9,21 @@ use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpFoundation\Response;
 
 abstract class BaseController extends AbstractController
 {
-    public function baseIndex(ParameterService $param, Request $request, array $configuration): Response
+    /**
+     * @param ParameterService $param
+     * @param Request          $request
+     * @param []               $configuration
+     */
+    public function baseIndex(ParameterService $param, Request $request, $configuration)
     {
         // gather the values
         $data = [];
         foreach ($configuration as $config) {
             $name = $config['name'];
-            $data[$name] = $param->get($name, false);
+            $data[$name] = $this->valueParser($config['type'], $param->get($name, false));
         }
 
         // build the form
@@ -51,22 +55,30 @@ abstract class BaseController extends AbstractController
             ]);
     }
 
+    private function valueParser($type, $value)
+    {
+        switch ($type) {
+            case CheckboxType::class:
+                return boolval($value);
+            default:
+                return $value;
+        }
+    }
+
     /**
      * @param $config
      * @param FormBuilderInterface $formBuilder
      */
-    private function configParser($config, FormBuilderInterface $formBuilder): void
+    private function configParser($config, &$formBuilder)
     {
         $type = $config['type'];
         $options = [];
         switch ($type) {
             case ChoiceType::class:
-                $choices = array_combine($config['choices'], $config['choices']);
-
-                if (is_callable($choices)) {
-                    $options['choices'] = $choices();
+                if (is_callable($config['choices'])) {
+                    $options['choices'] = $config['choices']();
                 } else {
-                    $options['choices'] = $choices;
+                    $options['choices'] = $config['choices'];
                 }
                 break;
             case CheckboxType::class:

Controller/ParameterController.php

@@ -6,17 +6,16 @@ namespace Ardent\ParameterBundle\Controller;
 
 use Ardent\ParameterBundle\Service\ParameterService;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpFoundation\Response;
-use Symfony\Component\Routing\Attribute\Route;
+use Symfony\Component\Routing\Annotation\Route;
 
 class ParameterController extends BaseController
 {
-    #[Route('/{category}', name: 'parameter_bundle_parameters')]
-    public function parameters(
+    /**
+     * @Route("/{category}", name="parameter_bundle_parameters")
+     */
+    public function parameters($category = 'categories',
                                ParameterService $param,
-        Request          $request,
-        ?string          $category = 'categories',
-    ): Response
+                               Request $request)
     {
         $configs = $param->getConfig();
 

DependencyInjection/ArdentParameterExtension.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Ardent\ParameterBundle\DependencyInjection;
+
+use Symfony\Component\Config\FileLocator;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Extension\Extension;
+use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
+
+class ArdentParameterExtension extends Extension
+{
+    public function load(array $configs, ContainerBuilder $container)
+    {
+        $loader = new YamlFileLoader(
+            $container,
+            new FileLocator(dirname(__DIR__).'/Resources/config')
+        );
+        $loader->load('services.yaml');
+
+        $configuration = new Configuration();
+        $config = $this->processConfiguration($configuration, $configs);
+
+        $def = $container->findDefinition('ardent.parameter');
+        $def->setArgument('$config', $config['parameters']);
+    }
+}

DependencyInjection/Configuration.php

@@ -0,0 +1,39 @@
+<?php
+
+
+namespace Ardent\ParameterBundle\DependencyInjection;
+
+
+use Symfony\Component\Config\Definition\Builder\TreeBuilder;
+use Symfony\Component\Config\Definition\ConfigurationInterface;
+
+class Configuration implements ConfigurationInterface
+{
+    public const TYPE_TEXT = 'text';
+    public const TYPE_NUMBER = 'number';
+    public const TYPE_BOOL = 'bool';
+
+    public function getConfigTreeBuilder()
+    {
+        $treeBuilder = new TreeBuilder('ardent_parameter');
+
+        $treeBuilder->getRootNode()
+            ->children()
+                ->arrayNode('parameters')
+                    ->arrayPrototype()
+                        ->arrayPrototype()
+                            ->children()
+                                ->scalarNode('name')->end()
+                                ->enumNode('type')
+                                    ->values([self::TYPE_TEXT, self::TYPE_NUMBER, self::TYPE_BOOL])
+                                    ->defaultValue(self::TYPE_TEXT)
+                                ->end()
+                            ->end()
+                        ->end()
+                    ->end()
+                ->end()
+            ->end();
+
+        return $treeBuilder;
+    }
+}

Entity/Parameter.php

@@ -0,0 +1,75 @@
+<?php
+
+namespace Ardent\ParameterBundle\Entity;
+
+use Doctrine\ORM\Mapping as ORM;
+
+/**
+ * @ORM\Entity
+ * @ORM\Table(name="parameter_bundle_parameters")
+ */
+class Parameter
+{
+    /**
+     * @ORM\Id
+     * @ORM\GeneratedValue(strategy="AUTO")
+     * @ORM\Column(type="integer")
+     */
+    private $id;
+
+    /**
+     * @ORM\Column(type="string")
+     */
+    private $name;
+
+    /**
+     * @ORM\Column(type="string")
+     */
+    private $value;
+
+    /**
+     * @return mixed
+     */
+    public function getId()
+    {
+        return $this->id;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getName()
+    {
+        return $this->name;
+    }
+
+    /**
+     * @param mixed $name
+     * @return Parameter
+     */
+    public function setName($name)
+    {
+        $this->name = $name;
+        return $this;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getValue()
+    {
+        return $this->value;
+    }
+
+    /**
+     * @param mixed $value
+     * @return Parameter
+     */
+    public function setValue($value)
+    {
+        $this->value = $value;
+        return $this;
+    }
+
+
+}

Resources/config/services.yaml

@@ -7,6 +7,6 @@ services:
     autowire: true
 
   Ardent\ParameterBundle\Controller\:
-    resource: '../src/Controller'
-    tags: ['controller.service_arguments', 'container.service_subscriber']
+    resource: '../../Controller'
+    tags: ['controller.service_arguments']
     autowire: true

Service/ParameterService.php

@@ -2,44 +2,46 @@
 
 namespace Ardent\ParameterBundle\Service;
 
-use Ardent\ParameterBundle\ArdentParameterBundle;
+use Ardent\ParameterBundle\DependencyInjection\Configuration;
 use Ardent\ParameterBundle\Entity\Parameter;
 use Doctrine\ORM\EntityManagerInterface;
 use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
 use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
-use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\Extension\Core\Type\NumberType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 
 class ParameterService
 {
-    private array $config;
+    /** @var EntityManagerInterface */
+    private $em;
 
-    private array $cache = [];
+    /** @var ParameterBagInterface */
+    private $parameter;
+    private $config;
 
-    public function __construct(
-        private readonly EntityManagerInterface $em,
-        private readonly ParameterBagInterface  $parameter,
-        array                                   $config
-    )
+    /**
+     * ParameterService constructor.
+     *
+     * @param EntityManagerInterface $em
+     * @param ParameterBagInterface $parameter
+     * @param $config
+     */
+    public function __construct(EntityManagerInterface $em, ParameterBagInterface $parameter, $config)
     {
+        $this->em = $em;
+        $this->parameter = $parameter;
         $this->config = $this->supplementConfig($config);
     }
 
     /**
      * @param string $name The name of the parameter in the format <group>_<parameter>
+     * @param bool $parse Whether to parse embedded %parameter%'s
+     *
+     * @return mixed
      */
-    public function get(string $name, bool $parse = true): mixed
+    public function get($name, $parse = true)
     {
-        if (array_key_exists($name, $this->cache)) {
-            return $this->cache[$name];
-        }
-
-        $split = explode('_', $name);
-        $category = array_shift($split);
-
         $paramRepo = $this->em->getRepository(Parameter::class);
-        /** @var Parameter $parameter */
         $parameter = $paramRepo->findOneBy(['name' => $name]);
         if ($parameter) {
             if ($parse) {
@@ -59,8 +61,7 @@ class ParameterService
                 $value = $parameter->getValue();
             }
 
-            $this->cache[$name] = Parameter::parseValue($value, $this->config[$category][$name]['type']);
-            return $this->cache[$name];
+            return $value;
         } else {
             return null;
         }
@@ -68,8 +69,9 @@ class ParameterService
 
     /**
      * @param string $name The name of the parameter in the format <group>_<parameter>
+     * @param mixed $value
      */
-    public function set(string $name, mixed $value): void
+    public function set($name, $value)
     {
         $paramRepo = $this->em->getRepository(Parameter::class);
         /* @var Parameter $parameter */
@@ -80,39 +82,37 @@ class ParameterService
         $parameter->setValue($value);
         $this->em->persist($parameter);
         $this->em->flush();
-        $this->cache[$name] = $value;
     }
 
-    private function supplementConfig(array $rawConfig): array
+    /**
+     * Replaces the types in the config by their classes
+     *
+     * @param $config
+     * @return array
+     */
+    private function supplementConfig($config)
     {
-        $config = [];
-        foreach ($rawConfig as $groupName => $rawGroup) {
-            $group = [];
-            foreach ($rawGroup as &$value) {
+        foreach ($config as $groupName => &$group) {
+            foreach ($group as &$value) {
                 switch ($value['type']) {
                     default:
-                    case ArdentParameterBundle::TYPE_TEXT:
+                    case Configuration::TYPE_TEXT:
                         $class = TextType::class;
                         break;
-                    case ArdentParameterBundle::TYPE_NUMBER:
+                    case Configuration::TYPE_NUMBER:
                         $class = NumberType::class;
                         break;
-                    case ArdentParameterBundle::TYPE_BOOL:
+                    case Configuration::TYPE_BOOL:
                         $class = CheckboxType::class;
                         break;
-                    case ArdentParameterBundle::TYPE_CHOICE:
-                        $class = ChoiceType::class;
-                        break;
                 }
 
                 // Add the group name in front of the parameter name
                 $value['name'] = sprintf('%s_%s', $groupName, $value['name']);
                 $value['type'] = $class;
-
-                $group[$value['name']] = $value;
                 unset($value);
             }
-            $config[$groupName] = $group;
+            unset($group);
         }
         return $config;
     }

composer.json

@@ -3,14 +3,13 @@
     "description": "Simple bundle for storing parameters in a database",
     "type": "symfony-bundle",
     "require": {
-        "php": "^8.1",
         "doctrine/orm": "^2.0",
-        "symfony/form": "^5.0|^6.0|^7.0",
-        "symfony/twig-bundle": "^5.0|^6.0|^7.0"
+        "symfony/form": "^5.0|^6.0",
+        "symfony/twig-bundle": "^5.0|^6.0"
     },
     "autoload": {
         "psr-4": {
-            "Ardent\\ParameterBundle\\": "src/"
+            "Ardent\\ParameterBundle\\": ""
         }
     },
     "license": "GPL-3.0-only"

src/ArdentParameterBundle.php

@@ -1,52 +0,0 @@
-<?php
-
-
-namespace Ardent\ParameterBundle;
-
-use Symfony\Component\Config\Definition\Configurator\DefinitionConfigurator;
-use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
-use Symfony\Component\HttpKernel\Bundle\AbstractBundle;
-
-class ArdentParameterBundle extends AbstractBundle
-{
-
-    public const TYPE_TEXT = 'text';
-    public const TYPE_NUMBER = 'number';
-    public const TYPE_BOOL = 'bool';
-    public const TYPE_CHOICE = 'choice';
-
-    public function configure(DefinitionConfigurator $definition): void
-    {
-//        $treeBuilder = new TreeBuilder('ardent_parameter');
-
-        $definition->rootNode()
-            ->children()
-                ->arrayNode('parameters')
-                    ->arrayPrototype()
-                        ->arrayPrototype()
-                            ->children()
-                                ->scalarNode('name')->end()
-                                ->enumNode('type')
-                                    ->values([self::TYPE_TEXT, self::TYPE_NUMBER, self::TYPE_BOOL, self::TYPE_CHOICE])
-                                    ->defaultValue(self::TYPE_TEXT)
-                                ->end()
-                                ->arrayNode('choices')
-                                    ->scalarPrototype()->end()
-                                ->end()
-                            ->end()
-                        ->end()
-                    ->end()
-                ->end()
-            ->end();
-    }
-
-    public function loadExtension(array $config, ContainerConfigurator $container, ContainerBuilder $builder): void
-    {
-        $container->import('../config/services.yaml');
-
-        // Retrieves the parameters from the config and injects them info the ardent.parameter service
-        $def = $builder->findDefinition('ardent.parameter');
-        $def->setArgument('$config', $config['parameters']);
-    }
-}

src/Entity/Parameter.php

@@ -1,61 +0,0 @@
-<?php
-
-namespace Ardent\ParameterBundle\Entity;
-
-use Doctrine\ORM\Mapping as ORM;
-use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
-use Symfony\Component\Form\Extension\Core\Type\NumberType;
-
-#[ORM\Entity()]
-#[ORM\Table(name: 'parameter_bundle_parameters')]
-class Parameter
-{
-    #[ORM\Id]
-    #[ORM\GeneratedValue]
-    #[ORM\Column]
-    private ?int $id = null;
-
-    #[ORM\Column]
-    private ?string $name;
-
-    #[ORM\Column(type: 'string')]
-    private mixed $value;
-
-    public function getId(): int
-    {
-        return $this->id;
-    }
-
-    public function getName(): string
-    {
-        return $this->name;
-    }
-
-    public function setName(string $name): self
-    {
-        $this->name = $name;
-        return $this;
-    }
-
-    public function getValue(): mixed
-    {
-        return $this->value;
-    }
-
-    static function parseValue(mixed $value, string $type): mixed
-    {
-        return match ($type) {
-            NumberType::class => intval($value),
-            CheckboxType::class => boolval($value),
-            default => $value,
-        };
-    }
-
-    public function setValue(mixed $value): self
-    {
-        $this->value = $value;
-        return $this;
-    }
-
-
-}