Restore junction schema extension compatibility

composer.json

@@ -32,6 +32,7 @@
         "phpnomad/logger": "^1.0"
     },
     "require-dev": {
+        "phpnomad/event": "^1.0",
         "phpnomad/tests": "^0.1.0 || ^0.3.0"
     },
     "config": {

lib/Abstracts/JunctionTable.php

@@ -25,7 +25,7 @@
     /**
      * @var mixed|null
      */
     protected LoggerStrategy $logger;
 
     public function __construct(
         HasLocalDatabasePrefix  $localPrefixProvider,
@@ -39,10 +39,10 @@
     )
     {
         $args = func_get_args();
         $this->logger = array_pop($args);
         $this->rightTable = array_pop($args);
         $this->leftTable = array_pop($args);
         parent::__construct(...$args);
     }
 
     /**
@@ -104,7 +104,7 @@
      */
     public function getLeftColumnName(): string
     {
-        return $this->tableSchemaService->getJunctionColumnNameFromTableUncached($this->leftTable);
+        return $this->tableSchemaService->getJunctionColumnNameFromTable($this->leftTable);
     }
 
     /**
@@ -114,7 +114,7 @@
      */
     public function getRightColumnName(): string
     {
-        return $this->tableSchemaService->getJunctionColumnNameFromTableUncached($this->rightTable);
+        return $this->tableSchemaService->getJunctionColumnNameFromTable($this->rightTable);
     }
 
     /**
@@ -146,7 +146,7 @@
     {
         try {
             // Find the corresponding primary column by the column name
-            $primaryColumn = $this->tableSchemaService->getPrimaryColumnNameForTableUncached($primaryTable);
+            $primaryColumn = $this->tableSchemaService->getPrimaryColumnNameForTable($primaryTable);
         }catch(ColumnNotFoundException $e){
             $this->logger->logException($e, '', [], LoggerLevel::Emergency);
             throw $e;

lib/Services/TableSchemaService.php

@@ -34,11 +34,11 @@ public function getPrimaryColumnsForTable(TableInterface $table): array
     }
 
     /**
-     * Read the supplied descriptor without consulting or changing shared cache.
-     * Returns the same primary-column metadata that the cached lookup computes
-     * after a cache miss.
+     * Read the supplied descriptor without this service consulting or changing
+     * its shared cache. Returns the same primary-column metadata that the cached
+     * lookup computes after a cache miss.
      * Descriptor failures propagate unchanged. This does not inspect storage.
-     * Custom descriptors must provide stable, side-effect-free metadata.
+     * Descriptor callbacks may invoke cached methods or perform other side effects.
      *
      * @return Column[]
      */
@@ -48,8 +48,9 @@ public function getPrimaryColumnsForTableUncached(TableInterface $table): array
     }
 
     /**
-     * Return the descriptor's sole primary Column without shared cache access.
-     * Preserves the cached helper's cardinality rule and original Column object.
+     * Return the descriptor's sole primary Column without this service accessing
+     * its shared cache. Preserves the cached helper's cardinality rule and original
+     * Column object. Descriptor callbacks may cross their own cache boundaries.
      *
      * @throws ColumnNotFoundException When there is not exactly one primary column.
      */
@@ -68,8 +69,9 @@ public function getPrimaryColumnNameForTableUncached(TableInterface $table): Col
     }
 
     /**
-     * Compose the existing junction-column name without shared cache access.
-     * Descriptor failures propagate unchanged.
+     * Compose the existing junction-column name without this service accessing
+     * its shared cache. Descriptor failures propagate unchanged. Descriptor
+     * callbacks may cross their own cache boundaries.
      *
      * @throws ColumnNotFoundException When there is not exactly one primary column.
      */