New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alcune incompatibilità con il plugin Cache Enabler #204

Open
manueleone opened this Issue May 3, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@manueleone

manueleone commented May 3, 2018

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 overclokk added the bug label May 7, 2018

@overclokk overclokk added this to the Version 2.5.x milestone May 7, 2018

@overclokk

This comment has been minimized.

Contributor

overclokk commented May 7, 2018

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

This comment has been minimized.

manueleone commented May 12, 2018

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

This comment has been minimized.

Contributor

overclokk commented May 14, 2018

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

manueleone added a commit to manueleone/italy-cookie-choices that referenced this issue May 18, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment