fix(FormManager): use constante from ImageField

YesWiki / yeswiki

YesWiki is a wiki system written in PHP, including extensions making collaboration more simple (databases, maps, easy editing, bootstrap themes,...).

https://yeswiki.net

GNU Affero General Public License v3.0

107 stars 55 forks source link

fix(FormManager): use constante from ImageField #1182

Closed J9rem closed 1 month ago

J9rem commented 2 months ago

A recent commit (https://github.com/YesWiki/yeswiki/commit/4cede2fcd1627b064f6913c22dccb54be32b7a49) 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 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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