Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

option to add search to the manifest #16

Merged
merged 3 commits into from
Nov 4, 2019
Merged

option to add search to the manifest #16

merged 3 commits into from
Nov 4, 2019

Conversation

symac
Copy link
Contributor

@symac symac commented Oct 28, 2019

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.

symac added 2 commits October 28, 2019 13:53
can check if there is an XML in order to disable search for items that
only contain images for example.
@Daniel-KM
Copy link
Owner

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
Copy link
Contributor Author

symac commented Nov 4, 2019

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 Daniel-KM merged commit 270ee02 into Daniel-KM:master Nov 4, 2019
@Daniel-KM
Copy link
Owner

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants