Nettoyage de globales flag_* #4881

Open
opened 2 weeks ago by marcimat · 3 comments
marcimat commented 2 weeks ago
Owner

Suite à #4878

On voit qu’on déclare (de façon assez désuète) des globales flag_* dans inc/utils :  https://git-mirror.spip.net/spip/spip/-/blob/master/ecrire/inc/utils.php#L2590

A priori on peut s’en passer ou réécrire si besoin dans des fonctions avec une varible statique, ou même une constante…

Bref, je pense que tout ça peut partir.

Suite à #4878 On voit qu’on déclare (de façon assez désuète) des globales flag_* dans inc/utils :  https://git-mirror.spip.net/spip/spip/-/blob/master/ecrire/inc/utils.php#L2590 A priori on peut s’en passer ou réécrire si besoin dans des fonctions avec une varible statique, ou même une constante… - flag_ob : utilisée dans https://git-mirror.spip.net/spip/spip/-/blob/master/ecrire/public/assembler.php#L159 ; introduit il y a longtemps le @ sur https://git-mirror.spip.net/spip/spip/-/commit/9080c0df84d , le test disable là https://git-mirror.spip.net/spip/spip/-/commit/3d02bec12ade93e59372dc04b69fe0cf5517616c et le flag_ob là https://git-mirror.spip.net/spip/spip/-/commit/ac57e525c940f75db3821d29db71c4f48502c3e0 (il y a 19 ans donc) : m’est avis que 1) ob_* est toujours présent… et 2) je doute en plus que les hébergeurs bloquent cette fonction maintenant… - flag_sapi_name : Il reste une occurrence de flag_sapi_name en globale dans le formulaire de login là https://git-mirror.spip.net/spip/spip/-/blob/master/prive/formulaires/login.php#L186 ; Je pense que ça peut virer car introduit il y a 14 ans, à une époque où certainement php_sapi_name() n’était peut être pas tout le temps présent ? https://git-mirror.spip.net/spip/spip/-/commit/7ce2c047f7c8 ; de plus il existe `PHP_SAPI` en constante maintenant au besoin - flag_get_cfg_var : utilisé juste pour le flag_upload suivant - flag_upload : aucune occurrence nulle part… Bref, je pense que tout ça peut partir.
Poster
Owner

Mais en fait il reste 1 occurrence dans https://git-mirror.spip.net/spip/medias/-/blob/master/formulaires/joindre_document.php#115 qui est trainé depuis très longtemps. La question pour lui est est-ce que le test (qui définit ce flag) get_cfg_var('upload_max_filesize') > 0 a encore un sens

Mais en fait il reste 1 occurrence dans https://git-mirror.spip.net/spip/medias/-/blob/master/formulaires/joindre_document.php#115 qui est trainé depuis très longtemps. La question pour lui est est-ce que le test (qui définit ce flag) `get_cfg_var('upload_max_filesize') > 0` a encore un sens
Collaborator

Il me semble que la globale $ldap_present n'est plus utilisée

Il me semble que la globale `$ldap_present` n'est plus utilisée
b_b added this to the 4.1 milestone 2 weeks ago
Poster
Owner

Sur flag_ob je pense qu’on peut vraiment le virer : c’était vraiment manifestement pour certains vieux serveurs mutualisés…

Par ailleurs il y a une occurrence dans le compresseur là https://git.spip.net/spip/compresseur/src/branch/master/compresseur_options.php#L14 sur un code qui semble t’il pourrait virer aussi : un avis @cerdic pour le compresseur ?

Sur `flag_ob` je pense qu’on peut vraiment le virer : c’était vraiment manifestement pour certains vieux serveurs mutualisés… Par ailleurs il y a une occurrence dans le compresseur là https://git.spip.net/spip/compresseur/src/branch/master/compresseur_options.php#L14 sur un code qui semble t’il pourrait virer aussi : un avis @cerdic pour le compresseur ?
b_b added the
amélioration
label 17 hours ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.