search

ItalyCookieChoices / italy-cookie-choices

A simple way to show how your website complies with the EU Cookie Law.
53 stars 12 forks source link

Alcune incompatibilità con il plugin Cache Enabler #204

Open manueleone opened 6 years ago

manueleone commented 6 years ago

Ciao @overclokk, volevo segnalarti un piccolo problema di incompatibilità con il plugin Cache Enabler – WordPress Cache.

Andando a rimpiazzare la url della cdn, crea un errore quando nei file cookiechoices.php incontra un'espressione regolare senza il costruttore new RegExp.

Es: e=/<script.*?src="(.*?)"/ diventa: e=/<script.*?src="(.*?)"//urlcnd.dominio

Non ho aperto una pull-request perché i file cookiechoices.php sembrano minimizzati, magari hai usato altro per generarli e fai prima a verificare tu :)

Ciao e grazie ancora per questo utilissimo script!

overclokk commented 6 years ago

Ciao @manueleone grazie della segnalazione, sa hai anche la soluzione al problema puoi fare una pull per il file cookiechoices.js che è quello utilizzato per generare quello minificato, poi vedo di fare un test e integrarlo in nella prossima release.

Grazie mille. Ciao Enea

manueleone commented 6 years ago

Cia @overclokk,

avevo dato uno sguardo al file cookiechoices.js che mi sembrava corretto. Probabilmente il "problema" lo crea proprio uglify trasformando le regex nella versione "literal". Forse basta impostare "mangle: false" nello script grunt. Poi provo, intanto grazie :)

overclokk commented 6 years ago

Grazie a te, fammi sapere se la tua soluzione funziona che così la applico :-)