Automattic / Automattic/VIP-Coding-Standards
Sniff idea: Detect when constant is being used in constant()
- Lingua principale
- PHP
- Stelle
- 261
- Fork
- 44
- Merge medio
- 19m
- PR unite (30g)
- 1
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 );
```
Guida per i contributori
Apri la guida per i contributori
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.
Valutazione
- Stack tecnologico
- php
- Ambito
- tooling
- Tipo di issue
- Funzionalità
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100