KnpLabs / KnpLabs/php-github-api
Do not "help" the user by changing parameters
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 2.2k
- Fork
- 585
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
On 30+ places we have code like this:
if (!in_array($state, array('open', 'closed'))) {
$state = 'open';
}
If the user misspelled the $state we should not help them. That may cause unpredicted actions. We should throw an exception or let the API call fail.
This is a BC break. I've marked it in milestone 3.0.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
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.
Direzione di ricerca
Cerca nel codice sorgente PHP il pattern ripetuto di normalizzazione dei parametri con in_array e verifica come falliscono attualmente i parametri API non validi. Determina il comportamento coerente in caso di errore e identifica tutte le posizioni interessate; il lavoro è completato quando i parametri scritti in modo errato non vengono più modificati silenziosamente e la suite di test esistente passa.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- api, backend
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100