FriendsOfREDAXO / geolocation

Tile-Proxy/Cache und Kartendarstellung mit Leaflet für REDAXO 5.13+
MIT License
26 stars 3 forks source link

Backslash aus Pfad in Schrägstrich ersetzen #17

Closed dtpop closed 2 years ago

dtpop commented 2 years ago

Ich hatte unter Windows (xampp php8) bei der Installation den Fehler AddOn geolocation konnte aus folgendem Grund nicht installiert werden: AssetPacker: "C:\www\myproject\app\redaxo\src\addons\geolocation/install/vendor/leaflet/leaflet.css" not minimized [C:wwwmyproject ppredaxosrc૝ons¾_stylepluginsredaxoscss_variables file not found for @import: line: 1, column: 0]! Minificaton stopped. (file C:\www\myproject\app\redaxo\src\addons\geolocation\lib\asset_packer.php line 353) Die ergänzte Zeile löste bei mir das Problem.

christophboecker commented 2 years ago

Wechen Wert hat denn styles vor Deiner neuen Zeile? Bei mir ist das auf dem Mac

"styles" => "@import '/Users/xyz/Sites/rexdev/geolocation/redaxo/src/addons/be_style/plugins/redaxo/scss/_variables';"
christophboecker commented 2 years ago

Mal abgesehen davon, dass die Änderung das Problem auf Deinem Windows-System löst und auf Unix-Systemen nicht schadet, ist es immer noch ein Workaround. Da stellt sich mir die Frage, wer der Problembär ist und die Lösung liefern müsste. Aktiv beteiligt sind

Keine Ahnung wo es da klemmen könnte. @import kommt bei SCSS an allen Ecken vor. Wenn ScssPHP mit dem von rex_path bereitgestellten Pfad hadert, wäre durchaus die Frage, wo man ansetzen muss. Kann ScssPHP nicht mit den Windows-Pfaden umgeben oder liefert rex_path einen suboptimalen Pfad (was vermutlich schon längst aufgefallen sein müsste). Mangels Windows-System kann ich leider nichts testen und ausprobieren.

dtpop commented 2 years ago

Ich schätze, dass der Scssphp Kompiler als Standard den Schrägstrich hat und den Backslash als Exoten nicht kennt.

dtpop commented 2 years ago

Danke für den Merge und gutes 2022!