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

Is this module usable in Drupal 8? #2

Closed
ebremer opened this issue Dec 6, 2017 · 11 comments
Closed

Is this module usable in Drupal 8? #2

ebremer opened this issue Dec 6, 2017 · 11 comments

Comments

@ebremer
Copy link

ebremer commented Dec 6, 2017

Is this module usable in Drupal 8?

@whikloj
Copy link
Member

whikloj commented Dec 6, 2017

Yes, though I'm not sure how much testing has been done so your mileage may vary 😄 .

@ebremer
Copy link
Author

ebremer commented Dec 6, 2017

I got some errors going into the configuration pages for the module, (nothing of the page displayed) I'm using 8.4.2.
PHP Fatal error: Default value for parameters with a class type hint can only be NULL in /www/drupal/modules/openseadragon/src/Config.php on line 70

This occurred in four places in the code. For grins and giggles, I change the four places to:
bool $filterNull = NULL. I can access the configuration page now, but I'm not entirely sure if my "fix" was even accurate. My Drupal runs on CentOS 7.X with php56w.

Curious, in using the field formatter, what field type is used for a file upload and which type is used for a URL for iiif?

@whikloj
Copy link
Member

whikloj commented Dec 6, 2017

@ebremer sorry this was built in Drupal 8.2 and I keep forgetting how fast they are moving. I'll try to update it to match the latest changes.

As for your other question, we get the URI here from the file on the filesystem. So this might not work as a formatter for a URI to an alternate IIIF server...that is a good point though never thought of that.

@ebremer
Copy link
Author

ebremer commented Dec 6, 2017

So, in your set-up, is your Drupal and iiif server on the same system?

@whikloj
Copy link
Member

whikloj commented Dec 6, 2017

Yes, currently we are using Cantaloupe running locally for development purposes. You can see the entire system here https://github.com/Islandora-Devops/claw-playbook

@ebremer
Copy link
Author

ebremer commented Dec 6, 2017

So you use Drupal as the image uploader and display engine (using OSD) and Cantaloupe as the image tiling engine (iiif)?

@whikloj
Copy link
Member

whikloj commented Dec 7, 2017

I think that is correct, but I'll be honest I'm fairly new to this.

That being said I can see your use case and how you could alter this to check the field type.

This field formatter is only meant for image and file types and I don't think you would have a Url in an image field.

So we would need to expand it to cover a Url field.

@ebremer
Copy link
Author

ebremer commented Dec 7, 2017

I am working with gigapixel images. The URI would be for an image on a iiif-type server, in my case, IIP. Having the ability to include local and remote images would be very handy.

@ebremer
Copy link
Author

ebremer commented Dec 7, 2017

We use OpenSeadragon now to access remote images via iiif protocol, but dropping OpenSeadragon into the context of Drupal would have many benefits which is what brought me to your module in the first place.

@whikloj
Copy link
Member

whikloj commented Dec 8, 2017

Cool, I'll see if I can dedicate some time to that. No guarantees in the short term, but if I get something working I'll ping you to test it out.

@rosiel
Copy link
Member

rosiel commented Jul 20, 2022

To answer this question (several years later) we now provide a block that takes (when placed) an IIIF manifest. For our uses, we often provide it node/[node:nid]/manifest which we set up to be that node's children, using an IIIF view formatter. But if you drop an external manifest URL into the block, it works - sometimes.

Here's an example of it working with the manifest at https://digital.library.villanova.edu/Item/vudl:92879/Manifest

Screen Shot 2022-07-20 at 1 35 06 PM

However some other manifests manifestly don't work. For example, https://iiif.europeana.eu/presentation/9200396/BibliographicResource_3000118436165/manifest yeilds "Unable to load TileSource".

I can not figure out how to set this up to debug javascript to figure out why that manifest doesn't work.

@rosiel rosiel closed this as completed Jul 20, 2022
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

No branches or pull requests

3 participants