ICanBoogie/ActiveRecord update

README.md

@@ -32,9 +32,7 @@ $primary_connection = $app->service_for_id('active_record.connection.primary', C
 # or
 $primary_connection = $app->service_for_class(ConnectionProvider::class)->connection_for_id('primary');
 
-$nodes = $app->service_for_class(NodeModel::class);
-# or
-$nodes = $app->service_for_class(ModelProvider::class)->model_for_class(NodeModel::class);
+$nodes = $app->service_for_class(ModelProvider::class)->model_for_record(Node::class);
 ```
 
 

composer.json

@@ -29,8 +29,7 @@
 		"php": ">=8.1",
 		"icanboogie/activerecord": "dev-remove-model-id as 6.0",
 		"icanboogie/icanboogie": "^6.0",
-		"olvlvl/composer-attribute-collector": "^2.0",
-		"symfony/expression-language": "^6.3"
+		"olvlvl/composer-attribute-collector": "^2.0"
 	},
 	"require-dev": {
 		"phpstan/phpstan": "^1.10",

config/services.yml

@@ -22,10 +22,6 @@ services:
   ICanBoogie\ActiveRecord\ModelIterator:
     alias: ICanBoogie\ActiveRecord\ModelCollection
 
-  ICanBoogie\ActiveRecord\ModelResolver:
-    alias: ICanBoogie\ActiveRecord\ModelCollection
-    public: true # required by StaticModelResolver
-
   # Console
 
   ICanBoogie\Binding\ActiveRecord\Console\ListModelsCommand:

lib/Console/ListModelsCommand.php

@@ -23,15 +23,16 @@ protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $rows = [];
 
-        foreach ($this->config->models as $model_id => $attributes) {
+        foreach ($this->config->models as $attributes) {
             $rows[] = [
-                $model_id,
+                $attributes->activerecord_class,
+                $attributes->model_class,
                 $attributes->connection,
             ];
         }
 
         $table = new Table($output);
-        $table->setHeaders([ 'Id', 'Connection' ]);
+        $table->setHeaders([ 'ActiveRecord', 'Model', 'Connection' ]);
         $table->setRows($rows);
         $table->setStyle($this->style);
         $table->render();

lib/ContainerExtension.php

@@ -17,7 +17,6 @@
 use ICanBoogie\ActiveRecord\ModelProvider;
 use ICanBoogie\Application;
 use ICanBoogie\Binding\SymfonyDependencyInjection\ExtensionWithFactory;
-use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\Extension\Extension;
@@ -67,14 +66,15 @@ private function register_connections(ContainerBuilder $container): void
      */
     private function register_models(ContainerBuilder $container): void
     {
-        foreach ($this->config->models as $model) {
-            $class = $model->model_class;
-            $definition = (new Definition($class))
-                ->setFactory([ new Reference(ModelProvider::class), 'model_for_class' ])
-                ->setArguments([ $class ])
+        foreach ($this->config->models as $definition) {
+            $activerecord_class = $definition->activerecord_class;
+
+            $d = (new Definition($definition->model_class))
+                ->setFactory([ new Reference(ModelProvider::class), 'model_for_record' ])
+                ->setArguments([ $activerecord_class ])
                 ->setPublic(true);
 
-            $container->setDefinition($class, $definition);
+            $container->setDefinition(Record::format_service_id($activerecord_class), $d);
         }
     }
 }

lib/Hooks.php

@@ -15,8 +15,7 @@
 use ICanBoogie\ActiveRecord\ActiveRecordCache\RuntimeActiveRecordCache;
 use ICanBoogie\ActiveRecord\Model;
 use ICanBoogie\ActiveRecord\ModelProvider;
-use ICanBoogie\ActiveRecord\ModelResolver;
-use ICanBoogie\ActiveRecord\StaticModelResolver;
+use ICanBoogie\ActiveRecord\StaticModelProvider;
 use ICanBoogie\Application;
 use ICanBoogie\Validate\ValidationErrors;
 
@@ -35,19 +34,11 @@ public static function on_app_boot(Application\BootEvent $event): void
     {
         $app = $event->app;
 
-        ActiveRecord\StaticModelProvider::define(function (string $id) use ($app): Model {
-            static $models;
-
-            $models ??= $app->service_for_class(ModelProvider::class);
-
-            return $models->model_for_id($id);
-        });
-
-        StaticModelResolver::define(
-            static function () use ($app): ModelResolver {
+        StaticModelProvider::define(
+            static function () use ($app): ModelProvider {
                 static $resolver;
 
-                return $resolver ??= $app->service_for_class(ModelResolver::class);
+                return $resolver ??= $app->service_for_class(ModelProvider::class);
             }
         );
     }

lib/Record.php

@@ -6,28 +6,24 @@
 use ICanBoogie\ActiveRecord;
 use Symfony\Component\DependencyInjection\Attribute\Autowire;
 
-use function sprintf;
-use function str_replace;
-
 #[Attribute(Attribute::TARGET_PARAMETER)]
 class Record extends Autowire
 {
+    public const SERVICE_PREFIX = 'active_record.model.';
+
+    public static function format_service_id(string $activerecord_class): string
+    {
+        return self::SERVICE_PREFIX . $activerecord_class;
+    }
+
     /**
      * @param class-string<ActiveRecord> $activerecord_class
      */
     public function __construct( // @phpstan-ignore-line
         public readonly string $activerecord_class,
     ) {
         parent::__construct(
-            expression: sprintf(
-                "service('%s').model_for_activerecord('%s')",
-                self::escape(ActiveRecord\ModelProvider::class),
-                self::escape($this->activerecord_class)
-            )
+            service: self::format_service_id($activerecord_class)
         );
     }
-
-    private static function escape(string $str): string {
-        return str_replace('\\', '\\\\', $str);
-    }
 }

tests/app/all/config/services.yml

@@ -11,8 +11,8 @@ services:
   test.active_record.models:
     alias: ICanBoogie\ActiveRecord\ModelProvider
 
-  test.active_record.model.node_by_class:
-    alias: Test\ICanBoogie\Binding\ActiveRecord\Acme\NodeModel
+  test.active_record.model.node:
+    alias: active_record.model.Test\ICanBoogie\Binding\ActiveRecord\Acme\Node
 
   Test\ICanBoogie\Binding\ActiveRecord\Acme\SampleService:
     autowire: true

tests/lib/Console/ListModelsCommandTest.php

@@ -4,6 +4,7 @@
 
 use ICanBoogie\Binding\ActiveRecord\Console\ListModelsCommand;
 use ICanBoogie\Console\Test\CommandTestCase;
+use Test\ICanBoogie\Binding\ActiveRecord\Acme\Node;
 use Test\ICanBoogie\Binding\ActiveRecord\Acme\NodeModel;
 
 final class ListModelsCommandTest extends CommandTestCase
@@ -17,6 +18,7 @@ public static function provideExecute(): array
                 ListModelsCommand::class,
                 [],
                 [
+                    Node::class,
                     NodeModel::class,
                     'primary'
                 ]

tests/lib/ContainerTest.php

@@ -11,11 +11,10 @@
 
 namespace Test\ICanBoogie\Binding\ActiveRecord;
 
+use ICanBoogie\ActiveRecord\Config;
 use ICanBoogie\ActiveRecord\Connection;
 use ICanBoogie\ActiveRecord\ConnectionProvider;
-use ICanBoogie\ActiveRecord\Model;
 use ICanBoogie\ActiveRecord\ModelProvider;
-use ICanBoogie\ActiveRecord\Config;
 use PHPUnit\Framework\TestCase;
 use Test\ICanBoogie\Binding\ActiveRecord\Acme\NodeModel;
 
@@ -43,7 +42,7 @@ public static function provide_service(): array
             [ 'test.active_record.config', Config::class ],
             [ 'test.active_record.connections', ConnectionProvider::class ],
             [ 'test.active_record.models', ModelProvider::class ],
-            [ 'test.active_record.model.node_by_class', NodeModel::class ],
+            [ 'test.active_record.model.node', NodeModel::class ],
             [ 'active_record.connection.primary', Connection::class ],
             [ 'active_record.connection.cache', Connection::class ],
 

tests/lib/IntegrationTest.php

@@ -5,6 +5,7 @@
 use ICanBoogie\ActiveRecord\ModelProvider;
 use PHPUnit\Framework\TestCase;
 use Test\ICanBoogie\Binding\ActiveRecord\Acme\Article;
+use Test\ICanBoogie\Binding\ActiveRecord\Acme\Node;
 use Test\ICanBoogie\Binding\ActiveRecord\Acme\NodeModel;
 
 use Test\ICanBoogie\Binding\ActiveRecord\Acme\SampleService;
@@ -16,7 +17,7 @@ final class IntegrationTest extends TestCase
     public function test_nodes_model(): void
     {
         $model_provider = app()->service_for_id('test.active_record.models', ModelProvider::class);
-        $nodes = $model_provider->model_for_class(NodeModel::class);
+        $nodes = $model_provider->model_for_record(Node::class);
 
         $this->assertInstanceOf(NodeModel::class, $nodes);
         $this->assertEquals('id', $nodes->schema->primary);