search

fossgis / openstreetmap.de

Source of openstreetmap.de website
https://openstreetmap.de
8 stars 17 forks source link

Pre-release Optimierungen #58

Closed rramthun closed 1 week ago

rramthun commented 1 week ago

siehe Commit-Kommentare

joto commented 1 week ago

Ich hab den PR jetzt nicht gemerged, aber alle Änderungen soweit eingebaut. An den Stellen, wo Du TODOs auskommentiert hast, habe ich stattdessen die entsprechenden Texte/Seiten ergänzt. Also außer der .File.Path vs. .Path-Geschichte ist alles soweit erledigt.

rramthun commented 1 week ago

Schön, danke.

Bzgl. des .File-Path verstehe ich die Funktionsweise von Hugo so: In einem {{ with .File }}-Block wird File an den Punkt "." gebunden - d.h. wenn ich innerhalb des Blocks . schreibe, ist nach dem with damit .File gemeint. .File.Path würde damit zu .File.File.Path, was natürlich nicht klappt.

Im aktuellen staging-Branch der Webseite meldet Hugo Version 0.127 folglich auch diesen Fehler:

ERROR render of "page" failed: "/tmp/openstreetmap.de/layouts/_default/baseof.html:5:11": execute of template failed: template: _default/single.html:5:11: executing "_default/single.html" at <partial "header.html" .>: error calling partial: "/tmp/openstreetmap.de/layouts/partials/header.html:9:7": execute of template failed: template: partials/header.html:9:7: executing "partials/header.html" at <partial "menu.html" .>: error calling partial: "/tmp/openstreetmap.de/layouts/partials/menu.html:4:40": execute of template failed: template: partials/menu.html:4:40: executing "partials/menu.html" at <.File.Path>: can't evaluate field File in type *source.File

joto commented 1 week ago

Du hast recht. Komischerweise hat das bei mir und auf staging wunderbar funktioniert!? Habs jetzt geändert.

joto commented 1 week ago

Damit ist das alles soweit drin. Danke Roland!