option to add search to the manifest

Daniel-KM / Omeka-S-module-IiifServer

IIIF Server is a module for Omeka S that adds the IIIF specifications to serve any images and medias.

Other

16 stars 11 forks source link

option to add search to the manifest #16

Closed symac closed 5 years ago

symac commented 5 years ago

This PR adds two options to the config form :

an Url to an IIIF search service provider, to which the item id will be sent;
a checkbox to tell IiifServer that the search service is provided by IiifSearch and extractOcr;

When an URL is set, IiifServer module will add a search service to the manifest. In case the checkbox is ticked, the module will first check if an XML file is attached to the item. It will prevent the search box to display on items where no OCR has been extracted and therefore no search results can be returned.

Daniel-KM commented 5 years ago

Ok pour le premier point. Pour le second, est-ce que la vérification ne pourrait pas être automatique plutôt que d'ajouter une option ? Par ailleurs, pour homogénéiser les clés de configuration, serait-il possible de les mettre en minuscule ?

symac commented 5 years ago

Merci @Daniel-KM pour les commentaires. J'ai harmonisé la casse des clés de configuration.

Et tout à fait d'accord avec la suggestion de vérifier automatiquement la présence du module ExtractOcr, je n'avais jamais cherché comment détecter la présence d'un autre module, c'est désormais chose faite.

Daniel-KM commented 5 years ago

Ok, c'est intégré. A noter qu'il y a un test sur les fichiers xml : c'est assez difficile et j'ai justement fait un pr sur ce point : omeka/omeka-s#1464. C'est sans doute de l'alto ? Le pr est disponible dans le module Next en attendant.