Automattic / Automattic/VIP-Coding-Standards

Sniff idea: Detect when constant is being used in constant()

Aperta
#717 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.