normalize_url() relies on @ to hide an undefined index for URLs without a query string
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 85/100
Direzione di ricerca
Individua normalize_url() ed esamina come suddivide gli URL in corrispondenza di ?; l’issue descrive il caso che non funziona e suggerisce un ritorno anticipato. Controlla il comportamento esistente di check_topic() per confermare che entrambi gli URL corrispondano ancora dopo la normalizzazione. Il lavoro è completato quando gli URL senza stringa di query non attivano più l’avviso e il comportamento di confronto rimane invariato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
For a URL with no ?, @list( $path, $query ) = explode( '?', $url ); reads index 1 of a one-element array. The @ hides the "Undefined array key 1" warning, but a custom error handler is still called, so debug and error loggers record it on every hub request.
Fix: return early when there's no query string, e.g.
if ( false === strpos( $url, '?' ) ) {
return rtrim( $url, '/' ) . '/';
}
Note this returns path/ instead of the current path/?. Both sides of the comparison in check_topic() go through normalize_url(), so matching is unaffected.
- Lingua principale
- PHP
- Stelle
- 15
- Fork
- 10
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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/pushpress
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
Automattic/pushpress#17 ·
-
GUID normalisation bug, leading to duplicatesForse già presa @Alkarex l’ha presa 420 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
Automattic/pushpress#15 · 1 commento · 1 reazione ·
-
enhancement help wanted
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
Automattic/pushpress#14 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Automattic/pushpress#2 · 1 commento ·
Tutte le issue di Automattic/pushpress
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 3 giorni
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
[Sync EN] Formatting improvements to "Large Objects (LOBs)" docs (#6056)Forse già presa Una pull request collegata a questa issue è aperta o già unita. Apertasync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Перевод устарел
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
I maintainer di solito rispondono entro 1 giorno
-
repo:cms
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
craftcms/cms#20613 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno