Envole » Envole-Mutualisé » eole-dispatcher

Tipunch sur IRC me fait remarquer que lorsqu'il fait un reconfigure, il perd les icones qu'il avait créées ou modifiées

Au départ je pensais que c'était un pb de config locale mais j'arrive à reproduire le pb chez moi en test (on ne fait pratiquement jamais de reconf)

Après analyse , le coupable serait le script utils/icon2SepTable.php

Ce script est censé modifier la structure de la table au cas ou les icones seraient toujours dans un champ "icon" de la table
Il transforme alors le champ "icon" en "old_icon", il crée une nouvelle entrée dans la table "icon" et il crée un nouveau champ "icon_id" qui fait référence à l'entrée créée précédemment dans cette table

Sauf que ce script commence par :

foreach ($apps as $app ) {
    if ($app->old_icon) {
        $icone = $app->old_icon;
    }

(Et pareil pour les urls plus bas)

Cela signifie que par défaut il va assigner à la nouvelle icone (celle qu'il va mettre dans la table "icon") la valeur de "old_icon"
Au début ce n'est pas grave : forcément, old_icon a d'abord la valeur de l'ancien icon
Mais si on change cette icone : ce sera icon_id qui changera. Mais PAS old_icon (on n'y touche plus). old_icon reste à l'icone initiale, c'est en quelque sorte un snapshot à un instant T de l'icone
Et si le script repasse derrière : il va remettre l'icone à old_icon qui est une ancienne version de l'icone...

Potentiellement c'est assez destructeur, car si un reconfigure est fait alors que beaucoup d’icônes ont été modifiées, celles-ci seront perdues

Je ne comprends pas pourquoi j'ai assigné cette valeur "old_icon" à $icone : cela n'a a priori aucun sens...
La seule explication que je vois, ce serait que j'avais prévu, lors d'une modif d'icone, dans le code applicatif, de modifier également old_icon.
Là, cela aurait un sens : on continue pendant quelques temps à faire cohabiter les 2 façons de faire : icone directement ds la table app + icone dans table séparée, mais je ne suis probablement pas allé jusqu'au bout

Ce qu'il faudrait faire, c'est simplement si on trouve le champ old_icon, on s'arrête, cela veut dire que le reste a été fait

foreach ($apps as $app ) {
    if ($app->old_icon) {
        break;
    }

Je t'assigne la demande pour avoir un avis sur la question, si des fois tu te souviens de quelque chose. Cela m'étonne qu'on ait laissé passer un truc aussi gros. Et cela m'étonne aussi qu'on n'ait eu aucune remontée depuis
D'un autre côté on ne fait pratiquement jamais de reconfigure en prod, j'imagine que les autres non plus...