addyosmani/critical

Remove query string from stylesheet path

Aperta

#551 aperta il 31 gen 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (457 fork)batch import
bughelp wanted

Metriche repository

Star
 (9902 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In my simple PHP file I reference my stylesheet as follows:

<link rel="stylesheet" href="css/style.css?v=<?php echo filemtime( 'css/style.css' ); ?>"

When using the CLI, I get the following error:

Error: File not found: css/style.css?v=<?php echo filemtime( 'css/style.css' ); ?>

This could be addressed by automatically removing query strings like ?v before trying to access the file. After that it works as expected.

Guida contributor

Direzione di ricerca
Trova la funzione di lettura del file nel pacchetto critical, rimuovi la stringa di query dal percorso prima di passarlo al filesystem.
Tech stack
javascript
Dominio
devtools
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85