search

lucascranach / importer

MIT License
1 stars 1 forks source link

Basis URL für Grafik Repräsentanten falsch … #17

Closed cnoss closed 2 years ago

cnoss commented 3 years ago

Hi,

die haben ein GWN Prefix, statt ein G, also: https://lucascranach.org/imageserver-2021/GWN_ANO_H-NONE-003/01_Overall/GWN_ANO_H-NONE-003_Overall-s.jpg

In den JSONS steht aber noch: https://lucascranach.org/imageserver-2021/GWN_ANO_H-NONE-003/01_Overall/GWN_ANO_H-NONE-003_Overall-s.jpg

Danke fürs fixn, c

jhfpereira commented 3 years ago

Das Prefix-Problem bin ich bereits vor kurzem angegangen. Dazu gibt es bereits Anpassungen im Importer:

Replace-Muster: Modules/Graphics/Entities/Graphic.php#L27

Image-ID: Modules/Graphics/Entities/Graphic.php#L83

Setzen / Aktualisieren der Inventarnummer: Modules/Graphics/Entities/Graphic.php#L170

Die Image-ID ist die ID, die für die Konstruktion des Pfades hin zu der imageData-1.1.json herangezogen wird. Diese enthält dann sicherheitshalber das Prefix (sofern gegeben), um mit den Ordnernamen auf dem Image-Server zu harmonieren.

Die API setzt aktuell noch auf ältere Daten, die noch nicht mit diesem Fix importiert wurden. Wenn neue Daten im ElasticSearch importiert werden, sollte sich das Problem von alleine gelöst haben. Dazu muss erst der Integration-Branch des API-Repos einmal in den Hauptzweig geführt werden, da nur dieser Stand mit leichten Änderungen an der Struktur der Daten klarkommt.

Würde das mit dem Volker besprechen und ggf. angehen.

Lasse das Issue solange auf, bis die API die neuen Datenstände zurückgibt.

jhfpereira commented 2 years ago

Würde das Issue schließen, da die Daten mittlerweile im ElasticSearch aktualisiert wurden.