Decoupled calls among parent and heir classes.

Moved declaration of common interfaces to the acme/contracts package.

composer.json

@@ -5,7 +5,8 @@
         "php": "^8.0",
         "symfony/dependency-injection": "^5.2",
         "symfony/config": "^5.2",
-        "acme/image": "^0.1"
+        "acme/image": "^0.1",
+        "acme/contracts": "0.1"
     },
     "autoload": {
         "psr-4": {
@@ -18,7 +19,7 @@
     "repositories": [
         {
             "type": "path",
-            "url": "./packages/acme-image"
+            "url": "./packages/*"
         }
     ],    "license": "MIT",
     "authors": [

packages/acme-contracts/composer.json

@@ -0,0 +1,17 @@
+{
+  "name": "acme/contracts",
+  "description": "a package for contracts between application and acme/image",
+  "license": "MIT",
+  "version": "0.1",
+  "authors": [
+    {
+      "name": "Grigori Kochanov",
+      "email": "public@grik.net"
+    }
+  ],
+  "autoload": {
+    "psr-4": {
+      "Acme\\Contracts\\": "src/"
+    }
+  }
+}

packages/acme-contracts/src/BaseModelInterface.php

@@ -0,0 +1,11 @@
+<?php
+
+
+namespace Acme\Contracts;
+
+
+interface BaseModelInterface
+{
+    public function foo();
+    public function callBar();
+}

packages/acme-contracts/src/FooImplementationInterface.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace Acme\Contracts;
+
+interface FooImplementationInterface
+{
+    public function withBaseModel(BaseModelInterface $baseModel): void;
+
+    public function doFoo(): mixed;
+}

packages/acme-image/src/Contracts/ImageInterface.php → packages/acme-contracts/src/ImageInterface.php

@@ -1,13 +1,13 @@
 <?php
 
 
-namespace Acme\Image\Contracts;
+namespace Acme\Contracts;
 
 /**
  * Interface ImageInterface
  * @package Acme\Image
  */
 interface ImageInterface
 {
-    public function load(string $tmp_name): mixed;
+    public function load(string $id): mixed;
 }

packages/acme-image/src/ImageService.php

@@ -3,8 +3,13 @@
 
 namespace Acme\Image;
 
-use Acme\Image\Contracts\ImageInterface;
+use Acme\Contracts\{
+    FooImplementationInterface,
+    BaseModelInterface,
+    ImageInterface
+};
 use Acme\Image\lib\Image;
+use Symfony\Contracts\Service\Attribute\Required;
 
 /**
  * Class ImageService provides an API for the application.
@@ -14,24 +19,33 @@
  * @package Acme\Image
  * @api
  */
-class ImageService  implements ImageInterface
+final class ImageService  implements ImageInterface, FooImplementationInterface
 {
 
     /**
      * @var Image
      */
     private Image $imageLib;
 
-    public function __construct()
+    /**
+     * @param string $id
+     * @return mixed
+     */
+    public function load(string $id): mixed
     {
-        $this->imageLib = new Image();
+        return $this->imageLib->load($id);
     }
 
-    /**
-     * @param string $tmp_name
-     */
-    public function load(string $tmp_name): mixed
+    #[Required]
+    public function withBaseModel(BaseModelInterface $baseModel): void
     {
-        return $this->imageLib->load($tmp_name);
+        $this->imageLib = new Image($baseModel);
+        $baseModel->init();
     }
+
+    public function doFoo(): mixed
+    {
+        return $this->imageLib->foo();
+    }
+
 }

packages/acme-image/src/lib/Image.php

@@ -2,25 +2,29 @@
 
 namespace Acme\Image\lib;
 
+use Acme\Contracts\BaseModelInterface;
+
 /**
- * This class is tightly coupled with \BaseModel
+ * This class is the code is not coupled with a \BaseModel class anymore
  *
  * @internal
  * @package Acme\Image
  */
-class Image extends \BaseModel
+class Image
 {
+    public function __construct(private BaseModelInterface $baseModel)
+    {
+    }
 
     /**
-     * called from the \BaseModel
+     * called from the \BaseModel through adapters
      */
-    protected function foo()
+    public function foo()
     {
-        //an inherited final method
-        $this->bar();
+        $this->baseModel->callBar();
     }
 
-    public function load($tmp_name)
+    public function load($id)
     {
     }
 }

services.php

@@ -2,16 +2,28 @@
 
 namespace Symfony\Component\DependencyInjection\Loader\Configurator;
 
-use Acme\Image\{Contracts\ImageInterface,ImageService};
+use Acme\BaseModelAdapter;
+use Acme\ImageResizeController;
+use Acme\Contracts\{
+    FooImplementationInterface,
+    BaseModelInterface,
+    ImageInterface
+};
+use Acme\Image\ImageService;
+
+use Symfony\Component\DependencyInjection\Reference;
 
 return function(ContainerConfigurator $configurator) {
     /**
      * @see https://symfony.com/doc/current/service_container/autowiring.html
      */
     $services = $configurator->services()->defaults()->autowire()->public();
 
-    $services->load('Acme\\', './src/*')
-        ->exclude('./src/BaseModel.php');
+    $services->set(ImageResizeController::class);
 
     $services->set(ImageInterface::class, ImageService::class);
+
+    $services->set(BaseModelInterface::class, BaseModelAdapter::class)
+        ->bind(FooImplementationInterface::class, service(ImageInterface::class))
+        ;
 };

src/BaseModelAdapter.php

@@ -0,0 +1,49 @@
+<?php
+declare(strict_types=1);
+
+namespace Acme;
+
+use Acme\Contracts\{BaseModelInterface,FooImplementationInterface};
+
+/**
+ * An adapter service for BaseModel to decouple inheritance into composition
+ *
+ * @package Acme
+ */
+final class BaseModelAdapter extends \BaseModel implements BaseModelInterface
+{
+    public function __construct(private FooImplementationInterface $fooInstance)
+    {
+    }
+
+    public function init():void
+    {
+        parent::__construct();
+    }
+
+    public function getX(): int
+    {
+        return $this->x;
+    }
+
+    /**
+     * Required by the interface of the BaseModel abstract class
+     */
+    public function foo()
+    {
+        return $this->fooInstance->doFoo($this);
+    }
+
+    /**
+     * An adapter for a final protected method bar()
+     */
+    public function callBar()
+    {
+        //print call trace after decoupling
+        foreach (debug_backtrace() as ['function'=>$f,'class'=>$c]) {
+            echo "$c::$f()\n";
+        }
+
+        return $this->bar();
+    }
+}

src/ImageResizeController.php

@@ -4,13 +4,12 @@
 namespace Acme;
 
 
-use Acme\Image\Contracts\ImageInterface;
+use Acme\Contracts\ImageInterface;
 
 class ImageResizeController
 {
-    public function __construct(
-        public ImageInterface $image
-    ){
+    public function __construct(public ImageInterface $image)
+    {
     }
 
     public function action(?string $id)