gazeteer namespace ist gehardcoded (dog)

Issue #584 resolved
Dominik Fröhlich created an issue

Hallo,

die Gazeteer-Suche ist super! So viel schonmal vorweg. Ich muss allerdings immer im code den wohl in Hamburg gebräuchlichen Namespace “dog” ersetzen. Hier wäre ein Setzen über die config statt im code sinnvoll (existiert bereits?).

Gruß

Dominik

Comments (9)

  1. Dennis Sen

    Hallo @Dominik Fröhlich ,

    ich denke, dass man hier den Namespace recht einfach über die Config setzen können sollte. Tatsächlich ist im Modul auch bereits eine Variable Namespace vorhanden, aus der man die Präfixe (bisher fix “dog:”) generieren könnte.

    Würden Sie mir bitte ihre Konfiguration in dieses Issue laden? Dann kann ich das darauf basierend einmal implementieren und direkt gegen den Gazetteer testen.

    Viele Grüße

  2. Dominik Fröhlich reporter

    Meine Konfiguration? Was benötigen Sie denn da genau?
    Wir nutzen hauptsächlich mapserver. Der Namespace wäre bei uns daher meistens “ms”.
    Masterportalkonfigurationen haben wir einige öffentlich, z.B.:

    https://geoportal.freiburg.de/freigis/config.js

    https://geoportal.freiburg.de/freigis/config.json

    https://geoportal.freiburg.de/freigis/ressources/services-internet.json

    Ist das was sie suchen?

    EDIT: Ah, man kann den jetzt schon setzen? Das ist natürlich super! Dann schaue ich gleich mal.

    EDIT2: Nö, ist immer noch gehardcoded, z.B. /modules/searchbar/gaz/model.js::134

  3. Dennis Sen

    Genau, mit Konfiguration meine ich die verschiedenen Config-Dateien, Services-Dateien, usw., mit denen man das Problem nachstellen kann.

    Da habe ich mich ungünstig ausgedrückt. Statt “über die Config setzen können sollte” hätte ich “über die Config setzbar machen können sollte” schreiben sollen.

    Das ist nun jedenfalls auch umgesetzt und ein Pull Request läuft gegen den Dev-Stand: https://bitbucket.org/geowerkstatt-hamburg/masterportal/pull-requests/3182 Mittels der Konfiguration habe ich überprüft, dass z.B. für die Eingabe “Stefan” auch die “Stefan-Meier-Straße” gefunden wird.

    Sobald der PR angenommen ist, gibt es in der folgenden Version auch dieses Feature.

  4. Dennis Sen

    Der PR ist schon durch. Ich schließe das Ticket von daher einmal, da damit alles erledigt sein sollte. Falls doch noch Probleme auftreten oder es Rückfragen gibt, gern wieder öffnen bzw. schreiben.

    Viele Grüße

  5. Log in to comment