Sniff idea: Detect when constant is being used in constant()
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 42/100
Direzione di ricerca
Inizia esaminando le convenzioni esistenti degli sniff di VIPCS-PHP_CodeSniffer e il modo in cui vengono rappresentate le chiamate a constant(). Verifica che constant(FOO_BAR) venga segnalato, mentre le stringhe tra virgolette e le variabili non vengano segnalate, e verifica il comportamento con la suite di test dello sniff pertinente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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 );
- Lingua principale
- PHP
- Stelle
- 261
- Fork
- 44
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Automattic/VIP-Coding-Standards
-
AlwaysReturnInFilter: isInsideIfConditonal() guards the conditions array after reading itForse già presa @tomjn l’ha presa 4 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its ifForse già presa @tomjn l’ha presa 4 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 76/100
-
Suppress filters in get_posts false positiveForse già presa @tomjn l’ha presa 5 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
-
Breaking Change Type: Maintenance
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Automattic/VIP-Coding-Standards#849 · 1 commento ·
Tutte le issue di Automattic/VIP-Coding-Standards
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Awaiting Triage bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
product / databases
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
VilnaCRM-Org/user-service#525 ·
I maintainer di solito rispondono entro 21 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
shukiv/jabali-panel#2029 ·
I maintainer di solito rispondono entro 1 giorno