hbz / nwbib

Die Nordrhein-Westfälische Bibliographie
http://nwbib.de
3 stars 2 forks source link

Get type for all Stadtbezirke #532

Closed acka47 closed 4 years ago

acka47 commented 4 years ago

...including those special Stadtbezirk classes for individual cities (Düsseldorf, Köln, Wuppertal). Resolves https://github.com/hbz/nwbib/issues/531

acka47 commented 4 years ago

Wie gewünscht, wird jetzt bei allen Stadtbezirkklassen das instanceOf ergänzt und nur bei diesen. Prinzipiell ist es bei solchen Dingen glaube ich besser VALUES anstatt FILTER zu nutzen, weil das performanter ist.

Das heißt, der Code muss so angepasst werden, dass er schaut, ob in der Spalte instanceOf etwas steht und dann – wenn nicht bereits vorhanden – den String " (Stadtbezirk)" anhängt.

acka47 commented 4 years ago

Wie gewünscht habe ich die Anfrage nun angepasst, so dass generisch nach allen Subtypen von "Stadtbezirk in Deutshcland" (Q2740635) gefragt wird. Diese Lösung sollte also auch funktionieren, falls für die Stadtbezirke anderer Städte in Zukunft eine eigene Klasse ergänzt werden sollte.

fsteeg commented 4 years ago

Deployed to test:

https://test.nwbib.de/spatial#Q114402 https://github.com/hbz/lobid-vocabs/pull/113/files

acka47 commented 4 years ago

+1