Skip to content

fix(FormManager): use constante from ImageField#1182

Merged
mrflos merged 1 commit intoYesWiki:doryphore-devfrom
J9rem:fix/form-manager-const
Jul 19, 2024
Merged

fix(FormManager): use constante from ImageField#1182
mrflos merged 1 commit intoYesWiki:doryphore-devfrom
J9rem:fix/form-manager-const

Conversation

@J9rem
Copy link
Contributor

@J9rem J9rem commented Jul 9, 2024

A recent commit (4cede2f) seems to have introduced errors in FormManager (usage of $this:: to access to a constant instead of self::).

What do this PR:

  • make public the constant FIELD_IMAGE_DEFAULT in ImageField
  • use it in FormManager

How to test:

  • without PR : errors
  • with PR : no error

@mrflos
Copy link
Contributor

mrflos commented Jul 19, 2024

Plutot que de rendre publique la propriété, c'est pas plus simple de faire des self::FIELD_IMAGE_DEFAULT au lieu de $this::FIELD_IMAGE_DEFAULT ?

@J9rem
Copy link
Contributor Author

J9rem commented Jul 19, 2024

En fait, le code qui utilise cette constante est située dans FormManager alors que la constante est définie dans ImageField.
Je trouve que ça rajoute de la dette technique de dupliquer la définition de la constante dans FormManager et ImageField alors qu'en la rendant publique dans ImageField, ça permet de l'utiliser dans FormManager avec une définition unique.

@mrflos
Copy link
Contributor

mrflos commented Jul 19, 2024

Ok j'avais pas vu que c'était dans une autre classe

@mrflos mrflos merged commit 46e8924 into YesWiki:doryphore-dev Jul 19, 2024
@J9rem J9rem deleted the fix/form-manager-const branch July 19, 2024 13:21
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.

2 participants