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

Document how to replace a media #1208

Closed
mjordan opened this issue Jul 5, 2019 · 10 comments
Closed

Document how to replace a media #1208

mjordan opened this issue Jul 5, 2019 · 10 comments
Assignees

Comments

@mjordan
Copy link
Contributor

mjordan commented Jul 5, 2019

In Islandora 7.x we have the ability to replace a datastream:

replace_datastream

In Islandora 8, you must "Remove" the file and replace it with another:

media

This should be documented at https://islandora-claw.github.io/CLAW/user-documentation/datastreams/.

Related issue: #1035.

@dannylamb
Copy link
Contributor

You should link out to https://islandora-claw.github.io/CLAW/technical-documentation/rest-create/#files-and-media as well to point out that there's ways to do this headlessly.

@mjordan
Copy link
Contributor Author

mjordan commented Jul 19, 2019

PR merged.

@mjordan mjordan closed this as completed Jul 19, 2019
@rosiel
Copy link
Member

rosiel commented Jul 20, 2019

I don't know if this is in scope. But does this process still leave the original file and its metadata floating around in Fedora? @dannylamb

@mjordan
Copy link
Contributor Author

mjordan commented Jul 17, 2020

Reopening this, since the text added by the above PR does not appear in https://islandora.github.io/documentation/user-documentation/media/ (maybe it was removed subsequently?), so that documentation still does not explain how to replace a media or file.

@rosiel to answer your question ^^, if I replace the file as described in the PR at https://github.com/Islandora/documentation/pull/1221/files, the file and its metadata remain in Fedora.

Also, if I delete the media at http://localhost:8000/media/X/delete, the binary and its metadata persist in Fedora. I've just verified this by adding a media, visiting it Fedora, then deleting the media, and revisiting it and its metadata in Fedora. Still there.

The only way I've found of verifiably deleting a file and its metadata is to run the "Delete media and file(s) action" in the general media list (http://localhost:8000/admin/content/media). Revisiting the file and its metadata in Fedora returns 404s.

This is all very confusing. I'd love to see a way to:

  1. replace a file, along with the option to completely purge the one you think you're deleting or to create a new version of the file and accompanying metadata (i.e., keep the old file too by not make it the current revision)
  2. delete (i.e. completely nuke from "Islandora") a file and its accompanying media and metadata

I wonder how we can wire up the code in the "Delete media and file(s) action" so that the user doesn't have to go to http://localhost:8000/admin/content/media to actually delete a file and its media but instead can do it closer to the node the file is associated with.

@mjordan mjordan reopened this Jul 17, 2020
@mjordan
Copy link
Contributor Author

mjordan commented Jul 18, 2020

Thinking about some UI options for this. These options are not "closer to the node the file is associated with" but they are closer to the existing UI for managing the specific media.

if I replace the file as described in the PR at https://github.com/Islandora/documentation/pull/1221/files, the file and its metadata remain in Fedora.

We could allow the user to choose whether the file and its metadata remain in Fedora (current behavior) or do not (expected behavior):

remove_media

if I delete the media at http://localhost:8000/media/X/delete, the binary and its metadata persist in Fedora.

We could allow the user to choose whether the media remains in Fedora (current behavior) or does not (expected behavior):

delete_media

Also, these options should only be present if Fedora is being used.

@rosiel
Copy link
Member

rosiel commented Jul 20, 2020

Thank you for validating that, @mjordan ! Would it be appropriate to document the current actual behaviour in the documentation? I would like to make a PR.

@mjordan
Copy link
Contributor Author

mjordan commented Jul 20, 2020

Sounds good to me, until we can come to a landing on how to implement the desired behavior. @manez any thoughts on the docs?

@mjordan
Copy link
Contributor Author

mjordan commented Jul 20, 2020

@rosiel 's improvements to the doc have been merged, but should I close this issue and open a new one for the UI mods described above?

@rosiel
Copy link
Member

rosiel commented Jul 20, 2020

Sounds good, as this is a "document" ticket.

I like your suggestions, by the way.

@mjordan
Copy link
Contributor Author

mjordan commented Jul 20, 2020

Thanks! Will close this one and open a new one.

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