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

Self-transform MODS function #75

Open
bondjimbond opened this issue May 17, 2019 · 9 comments
Open

Self-transform MODS function #75

bondjimbond opened this issue May 17, 2019 · 9 comments

Comments

@bondjimbond
Copy link
Contributor

A nice new feature would be to update the MODS datastreams for a list of PIDs by running them through a given self-transform XSL.

Use case: I've got a pull request open for Islandora XML Forms that provides a new self-transform XSLT, which I would like to apply against every object with a MODS datastream in the repository (~176,000 objects).

@mjordan
Copy link
Collaborator

mjordan commented May 17, 2019

This would be a useful feature... but maybe we can extend it to allow the use of an arbitrary XSLT stylesheet. If the user wanted to use the self transform configured in XML forms, they would pass a specific value to the drush command, but if they wanted to use an external XSLT stylesheet, they could pass the path to it.

@mjordan
Copy link
Collaborator

mjordan commented May 17, 2019

Of course, we don't want to make it specific to MODS, we'd need to allow the user to indicate the DSID that would be updated.

@bondjimbond
Copy link
Contributor Author

Sounds good to me!

@mjordan
Copy link
Collaborator

mjordan commented May 17, 2019

Over the weekend, I will test and merge (unless there are large issues) #74. Do you want me to take an initial stab at implementing this new feature as well?

@bondjimbond
Copy link
Contributor Author

Sure, no reason not to if you have the time!

@mjordan
Copy link
Collaborator

mjordan commented May 17, 2019

Just a note here, but this feature will need to take the behavior described in the "Updating DC datastreams by pushing other XML datastreams" section of the readme into account, which means we'll probably need an option to also update the DC or not.

@bondjimbond
Copy link
Contributor Author

Absolutely! And default behaviour should be to update DC if updating MODS.

@mjordan
Copy link
Collaborator

mjordan commented May 17, 2019

I was thinking the default behavior would be to do as it does now - prompt you for a yes or no. But I'll have to think about it a bit more.

@bondjimbond
Copy link
Contributor Author

Yes, that's sensible.

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

2 participants