search

FriendsOfREDAXO / minify

Minifiziert und bündelt CSS/Sass, JavaScript, HTML und Bilder im Medienpool.
MIT License
43 stars 5 forks source link

Update sets.php #54

Closed pschuchmann closed 4 years ago

pschuchmann commented 5 years ago

JS Dateien wurde nicht gelöscht, da Pfad nicht vollständig.

phoebusryan commented 5 years ago

Den Slash musst du in der Konfiguration (CSS und JS Pfad) am Ende einfügen.

pschuchmann commented 5 years ago

Das wäre die andere Lösung, aber dann ist der Kommentar irreführend, deswegen hatte ich das mal als Pull Request rein. Sonst funktioniert ja alles super :)

grafik

rkemmere commented 5 years ago

Wenn der Slash im Pfad mit angegeben wird, passt der Kommentar ja nicht mehr! Viel schlimmer ist aber, dass im Frontend der Pfad dann mit // ausgegeben wird.

In Zeile 115 wird es für CSS ja bereits so gemacht, nur bei JS war es fehlerhaft!

Also am Besten einfach den PR annehmen ;) der klappt nämlich!

Vg

alxndr-w commented 5 years ago

Gibt es hier noch etwas zu lösen?

pschuchmann commented 5 years ago

Ja, mit dem aktuellen Code werden die JS Dateien nicht gelöscht. Deswegen hatte ich den PR aufgemacht @alexplusde

alxndr-w commented 4 years ago

Ich brauche hier noch mehr Infos. Ich würde den PR gerne annehmen, bei mir funktioniert jedoch bisher alles ohne Probleme und ohne den PR.

rkemmere commented 4 years ago

Einfach mal versuchen den Cache der Dateien zu löschen. Steht zwar gelöscht, die JS Dateien sind aber file basiert im Cache Ordner noch vorhanden.