From d704ce8c079562a9b31e08bd554d8a2502d179c2 Mon Sep 17 00:00:00 2001
From: Jephte Clain <Jephte.Clain@univ-reunion.fr>
Date: Thu, 12 Jun 2025 15:54:20 +0400
Subject: [PATCH] =?UTF-8?q?possibilit=C3=A9=20de=20changer=20items=20dans?=
 =?UTF-8?q?=20getItemValues?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 php/src/db/CapacitorChannel.php | 6 +++++-
 php/src/db/CapacitorStorage.php | 4 ++++
 php/src/db/_private/_update.php | 1 -
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/php/src/db/CapacitorChannel.php b/php/src/db/CapacitorChannel.php
index 105937a..3bbe263 100644
--- a/php/src/db/CapacitorChannel.php
+++ b/php/src/db/CapacitorChannel.php
@@ -257,6 +257,10 @@ class CapacitorChannel implements ITransactor {
    * Retourner la clé primaire par cette méthode est l'unique moyen de
    * déclencher une mise à jour plutôt qu'une nouvelle création.
    *
+   * Bien que ce ne soit pas prévu à la base, si on veut modifier $item dans
+   * cette méthode pour des raisons pratiques, il suffit de retournerla valeur
+   * modifiée avec la clé "item"
+   *
    * Retourner [false] pour annuler le chargement
    */
   function getItemValues($item): ?array {
@@ -337,7 +341,7 @@ class CapacitorChannel implements ITransactor {
    * Retourner [false] pour annuler le chargement (la ligne n'est pas créée)
    *
    * Si $item est modifié dans cette méthode, il est possible de le retourner
-   * avec la clé "item" pour mettre à jour la ligne correspondante.
+   * avec la clé "item" pour mettre à jour la colonne correspondante.
    *
    * la création ou la mise à jour est uniquement décidée en fonction des
    * valeurs calculées par {@link self::getItemValues()}. Bien que cette méthode
diff --git a/php/src/db/CapacitorStorage.php b/php/src/db/CapacitorStorage.php
index df39422..114fdfb 100644
--- a/php/src/db/CapacitorStorage.php
+++ b/php/src/db/CapacitorStorage.php
@@ -1,6 +1,7 @@
 <?php
 namespace nulib\db;
 
+use nulib\A;
 use nulib\cl;
 use nulib\db\_private\_migration;
 use nulib\php\func;
@@ -364,6 +365,9 @@ abstract class CapacitorStorage {
     $values = func::call([$channel, "getItemValues"], $item, ...$args);
     if ($values === [false]) return 0;
 
+    $newItem = A::pop($values, "item");
+    if ($newItem !== null) $item = $newItem;
+
     $row = cl::merge(
       $channel->getSum("item", $item),
       $this->serialize($channel, $values));
diff --git a/php/src/db/_private/_update.php b/php/src/db/_private/_update.php
index 7b297c4..5791387 100644
--- a/php/src/db/_private/_update.php
+++ b/php/src/db/_private/_update.php
@@ -6,7 +6,6 @@ class _update extends _common {
     "prefix" => "?string",
     "table" => "?string",
     "schema" => "?array",
-    "cols" => "?array",
     "values" => "?array",
     "where" => "?array",
     "suffix" => "?string",