Sniff idea: Detect when constant is being used in constant()
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 42/100
Piste de recherche
Commencez par examiner les conventions existantes des sniffs de VIPCS-PHP_CodeSniffer et la manière dont les appels à constant() sont représentés. Confirmez que constant(FOO_BAR) est signalé, tandis que les chaînes entre guillemets et les variables ne le sont pas, et vérifiez le comportement avec la suite de tests du sniff concerné.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Describe the solution you'd like
Not sure if VIPCS is the best way forward, but I think this would be a worthy sniff.
When using constant(), we should throw a warning if a string is not being inputted. Of course it is valid PHP to do something like:
constant( FOO_BAR );
But I think most likely, what the user wants to do is:
constant( 'FOO_BAR' );
What code should be reported as a violation?
constant( FOO_BAR );
What code should not be reported as a violation?
constant( 'FOO_BAR' );
constant( "FOO_BAR" );
constant( $foo_bar );
- Langage dominant
- PHP
- Étoiles
- 261
- Forks
- 44
- Merge moyen
- 3 j 9 h
- PR mergées (30 j)
- 10
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Automattic/VIP-Coding-Standards
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
Automattic/VIP-Coding-Standards#876 ·
Les mainteneurs répondent en général sous 1 jour
-
Breaking Change Type: Maintenance
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Automattic/VIP-Coding-Standards#849 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#832 ·
Les mainteneurs répondent en général sous 1 jour
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
Automattic/VIP-Coding-Standards#823 ·
Les mainteneurs répondent en général sous 1 jour
-
Status: Awaiting feedback
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#754 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Automattic/VIP-Coding-Standards
Issues similaires
-
[Sync EN] Add more details about the array returned from openssl_x509_parse (php#2666) (#6013)Ouvertesync-en
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
fix(proxy): avoid exposing Reverb HTTP API (/apps) publicly in generated Traefik dynamic configOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
coollabsio/coolify#12390 ·
Les mainteneurs répondent en général sous 1 jour
-
win_build: parallel PostBuildEvent race in example_app_atiopencl and example_app_nvopencl copying openclapp_kernels.clPeut-être pris @qixuancao l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
verbb/formie#3072 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour