Avertissement perte de données avec métadonnées calculées en mode lecture

[alhyss]

Plume renvoie un avertissement "Si vous poursuivez, les modifications non enregistrées seront perdues." dans les conditions suivantes :

• en mode lecture ;
• avec un modèle qui inclut du calcul automatique ;
• l'utilisateur change de langue ou de modèle.

Il n'y a pas d'avertissement lors du passage en mode édition, pour autant que je puisse voir.

C'est tout à fait logique sur le fond. Le dictionnaire a bel et bien été modifié par le calcul automatique. Néanmoins, il me semble qu'il ne devrait jamais y avoir d'avertissement de ce type en mode lecture, où il n'est de toute façon pas possible d'enregistrer quoi que ce soit.

@WREATCHED Je m'interroge sur la manière la plus propre de gérer ça :

• Je peux m'assurer que l'attribut modified du dictionnaire de widgets vaille toujours False en mode lecture.
• Tu peux certainement ajouter un test de ton côté pour ne même pas aller contrôler la valeur de modified en mode lecture.

L'avantage de la première solution, c'est que ça revient à remplacer ça : https://github.com/MTES-MCT/metadata-postgresql/blob/92535d0d151b444249b75523658422069152f60d/plume/rdf/widgetsdict.py#L1319

Par :

        if self.edit:
            self.modified = True

On fait difficilement plus simple.

L'inconvénient, c'est que dans l'absolu le dictionnaire a bien été modifié, ce qui est censé être l'information fournie par modified. Je ne crois pas que ce soit utilisé ailleurs pour l'instant, mais il pourrait être préférable pour la suite que la valeur de l'attribut reste conforme à sa définition...

[WREATCHED]

Corrections apportées

Partout où nous avions le test unique self.zoneConfirmMessage : Modifié par if self.mode == "edit" and self.zoneConfirmMessage :

[alhyss]

Ça m'a l'air tout bon !

Je pense que l'ajout de la condition if self.mode == "edit" n'était pas nécessaire sur l'activation du mode traduction (ligne 391), puisque ce bouton n'est pas activable en mode lecture de toute façon. Mais pas de raison que ça pose problème, et ça pourrait être mieux comme ça si un jour on permet de visualiser toutes les traductions en mode lecture grâce à ce bouton.

[WREATCHED]

Ok 👍