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

add missing thumbnail check to verification #295

Open
lsat12357 opened this issue Aug 24, 2021 · 2 comments
Open

add missing thumbnail check to verification #295

lsat12357 opened this issue Aug 24, 2021 · 2 comments

Comments

@lsat12357
Copy link
Contributor

at the moment, the missing thumbnails are actually only missing from solr, can fix by reindexing.
strictly speaking, this is not a migration fail, but meh, close enough.
this should be a task that queries to make sure that the thumbnail field is populated in solr, and if not, initiates reindex

@lsat12357
Copy link
Contributor Author

so afaik this is one line of code to check, and then one more line to fix.
since accessing the solr doc is available via HyraxCore, something like
Hyrax::Migrator::HyraxCore::Asset.solr_record(asset.pid)['thumbnail_path_ss'].blank?
and then to set the thumbnail path
asset.update_index
seems overkill to add this to the verification process, also upon reflection, verification should not be fixing stuff.

@lsat12357
Copy link
Contributor Author

could add this to the derivatives check.

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

1 participant