scholarslab / FedoraConnector

This is an Omeka plugin for importing items from a Fedora Commons repository.
Apache License 2.0
11 stars 7 forks source link

Must images in Fedora all have a DSID of "SCREEN" #23

Open petermacdonald opened 8 years ago

petermacdonald commented 8 years ago

Lovely plugin BTW.

I've got the Fedora plugin working, but Omeka was only display images if the Fedora record has a datastream with the DSID of "SCREEN". That is, I don't see a way in the Fedora plugin to select, say, "OBJ" or "TN" or other commonly used DSIDs to display in Omeka.

Am I correct the the datastream must be called "SCREEN"?

To get around this, I have hacked the image renderer file "100-Image.php" and hardcoded "TN" where "SCREEN" is found. Is that the only way to handle this?

I just want to be sure I haven't missed seeing the way to configure Omeka to find the DSID I want to display.

Thanks, Peter MacDonald

erochest commented 8 years ago

Hi Peter,

You're correct: this does appear to be hard coded.

We've been testing this against our local Fedora installation, and against others as users do and file bug reports.

I'll leave this open for the time being. I'm not sure whether the best way to approach this would be to have a series of fall-backs or a configuration option. Thoughts?

I'll try to put some thought into this in the next month or so to see what appears to be the best solution.

Thanks, Eric