[spip-dev] écran de sécurité, demande avant PR

Dev
1

Bonjour,
Je sollicite votre avis avant de déposer une pr. J'aurai besoin afin d'éviter des erreurs au niveau notice de changer sur l'écran de sécu define('_ECRAN_SECURITE', '1.3.11'); par
if (!defined('_ECRAN_SECURITE')) {
     define('_ECRAN_SECURITE', '1.3.11'); // 2019-04-08
}
est-ce que cette demande serait pertinente ou enfreindrait-elle une règle de sécurité ?
Merci

2

je dirais que le principe d'une pull request, c'est précisement d'être une demande. Donc fait ta pull request, et on te répondra.

Mais je vois pas pourquoi tu aurais un _ECRAN_SECURITE defini AVANT l'écran de sécurité lui même.

3

Par ce que comme un idiot je ne l'avais pas viré de ./config alors que je le centralise ailleurs et l'appelle depuis mes options. Donc deux appels, un par le core et un autre dans mes options, mea culpa, je viens juste de réaliser mon erreur (mon oubli de l'avoir viré de config). Et j'avais donc ce notice.
Désolé pour le bruit et bonne soirée.