Skip to content

Allow the edition of berry-related data of items directly from Studio#554

Merged
AerunDev merged 14 commits intodevelopfrom
82-edit-the-berries-data-right-from-the-database-item-ui
Aug 12, 2025
Merged

Allow the edition of berry-related data of items directly from Studio#554
AerunDev merged 14 commits intodevelopfrom
82-edit-the-berries-data-right-from-the-database-item-ui

Conversation

@Aelysya
Copy link
Collaborator

@Aelysya Aelysya commented Jul 27, 2025

Description

This PR adds two new datablocks to the items database page to allow defining berry-related data in items such as growth time, berry yield, flavors etc...

closes #82

Tests to perform

  • Opening a pre-2.6.1 project correctly launches the migration.
  • Open the database page of any berry item, the berry-related datablocks should have been filled by the migration.
  • Open the database page of any non-berry item, the berry-related datablocks should be empty.
  • The new datablocks are active when the item has the "Berry" parameter.
  • The new datablocks are not active when the item doesn't have the "Berry" parameter.
  • Create a new item, the berry-related data should be empty on the page after you confirm the creation.
  • Create a new item using the importation feature with a berry item, the berry-related datablocks should correctly be filled.
  • Create a new item using the importation feature with a non-berry item, the berry-related datablocks should be empty.
  • The new editors are functional and work as intended, correctly filling their datablock with chosen options when you exit them.
  • The JSON files are correctly updated and formatted after each modification.
  • Adding the "Berry" parameter to an item correctly enables the new datablocks.
  • Removing the "Berry" parameter from an item disables the new datablocks and the related data are removed from the JSON file.

@Aelysya Aelysya self-assigned this Jul 27, 2025
@Aelysya Aelysya linked an issue Jul 27, 2025 that may be closed by this pull request
3 tasks
@Palbolsky
Copy link
Collaborator

Palbolsky commented Jul 28, 2025

Salut. J'ai tout testé. GG pour le travail.

J'ai trouvé des problèmes (et amélioration rapide à faire), que je note ici :

  • Les composants baies et préparations devraient être utilisable que pour les baies, car même si les données sont vides par défaut, entrer dans les éditeurs valorisent les données et il n'y a ensuite aucun moyen de revenir en arrière. Sauf que l'objet de type Baie n'existe pas, donc je pense qu'il faut le créer (ça se discute). Il pourrait dériver des objets de type Soin ;
  • La sauvegarde détecte une modification de données quand on ouvre et ferme le composant Baies sans qu'aucune modification n'ait été apportée par l'utilisateur ;
  • Lorsqu'on ne met rien dans les inputs Dimension, Baies, Croissance et Durée de l'arrosage, la valeur est par défaut mise à 0. Si on réouvre l'éditeur, 0 est en rouge car c'est une valeur non autorisée et on ne peut pas quitter l'éditeur. Donc par défaut il faut par exemple 0.1 pour Dimension et 1 pour les autres ;
  • Problème un peu particulier pour la Puissance de Don Naturel. Si on met n'importe quelle valeur et qu'on ferme "Données de Don Naturel", on peut fermer l'éditeur sans restriction (donc je peux écrire -50 par exemple).
  • Dans le composant Baies, pour le Nombre de baies, si les deux nombres sont égaux, on peut simplement en mettre qu'un. (donc au lieu d'écrire de 2 à 2 on peut écrire simplement 2)

Copy link
Collaborator

@AerunDev AerunDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salut ! :D J'ai pu tester complètement ta PR et globalement c'est nickel, bien joué. 😄
J'ai juste corrigé une typo et j'en ai profité pour remplacer "Pokémon" par "creature" sur cette chaîne de caractères.

Je note juste quelques détails pour améliorer les choses et être clean par rapport à l'attendu en matière de design et UX :

  • L'input "Pokébloc amélioré" devrait afficher l'unité "%", comme c'est le cas par exemple avec l'input "Effet secondaire" dans le bloc "Données" d'une capacité
  • Quand je change le paramètre "Est une baie" sur un objet, Studio affiche des "---" sur les données des blocs Baies et Préparations et si je sauvegarde, les données des blocs ne sont pas dans le JSON. Si par contre j'ouvre les blocs et les referme, j'ai bien les données affichées, et une sauvegarde les ajoute bien au JSON. Est-ce que c'est volontaire ?

@Aelysya
Copy link
Collaborator Author

Aelysya commented Aug 3, 2025

Pour les données non-présentes dans le JSON c'était le comportement à l'origine quand les editeurs étaient dispo pour tous les objets. J'ai modifié pour mettre des valeurs par défaut quand on rajoute le paramètre à un objet qui ne l'avait pas.

Copy link
Collaborator

@AerunDev AerunDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est tout bon comme ça, bien joué !

@AerunDev AerunDev merged commit c227313 into develop Aug 12, 2025
5 checks passed
@AerunDev AerunDev deleted the 82-edit-the-berries-data-right-from-the-database-item-ui branch August 12, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Edit the berries data right from the database Item UI

3 participants