Force all edits to use command

[qqmyers]

Conflicts:
src/main/java/edu/harvard/iq/dataverse/EditDatafilesPage.java

New Contributors

Welcome! New contributors should at least glance at CONTRIBUTING.md, especially the section on pull requests where we encourage you to reach out to other developers before you start coding. Also, please note that we measure code coverage and prefer you write unit tests. Pull requests can still be reviewed without tests or completion of the checklist outlined below. Note that we use the "closes" syntax below to trigger Github's automation to close the corresponding issue once the pull request is merged.

Thanks for your contribution to Dataverse!

Related Issues

• closes File metadata updates not reindexed #6092 File metadata updates not reindexed

Pull Request Checklist

• Unit tests completed
• Integration tests: None
• Deployment requirements, SQL updates, Solr updates, etc.: None
• Documentation completed
• Merged latest from "develop" branch and resolved conflicts

[dataversebot]

Can one of the admins verify this patch?

[coveralls]

Coverage increased (+0.02%) to 19.476% when pulling 5f7155e on QualitativeDataRepository:IQSS/alt6092 into df8dcf6 on IQSS:develop.

[qqmyers]

This is an alternative to #6094, per the discussion there, that removes the optimization logic that would route some file edits outside of any commands. I've tested making file-level metadata,tag,and provenance edits and think it is working as before. I did a quick time test and thing this version is slightly slower when editing one file in a dataset with 1000 files. (Not clear that's a fair comparison since I think #6094 includes indexing whereas this one now uses a command that does index after the call returns.) In any case, I think one of these two should get integrated since they fix an active bug (e.g. if you add a tag to a file, etc., the facets for sorting don't update)

[sekmiller]

Looks good. Can you get the latest from the Develop branch and resolve any conflicts?
Thanks!