-
Notifications
You must be signed in to change notification settings - Fork 0
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
"Resplit PDF" button produces error #80
Comments
IIIF print issue. Related to notch8/iiif_print#294 |
I've only done some sleuthing so far; I don't have a branch with code changes. I was able to reproduce this in local Hyku. I noticed the where clauses need to be updated to valkyrie queries. Then the #pdf? is not a method error is because file_set.pdf? needs to be updated to file_set.original_file.pdf?. Same goes for #mime_type => file_set.original_file.mime_type. Or I suppose we could introduce a helper method. cc @laritakr handing it back over to you since you may already have had WIP. |
There were a few pieces of the pdf resplitting process that were not fully valkyrized. This commit valkyrizes the resplitting, and fixes a bug with removing the child works from the initial split. Ref notch8/palni_palci_knapsack#80
Bring in fixes for resplitting & deleting the child works along with the parent work. Refs notch8/palni_palci_knapsack#80
Bring in fixes for resplitting & deleting the child works along with the parent work. Refs notch8/palni_palci_knapsack#80
Bring in fixes for resplitting & deleting the child works along with the parent work. Refs notch8/palni_palci_knapsack#80 Co-authored-by: LaRita Robinson <[email protected]>
QA RESULTS: ❌ NEEDS REWORKtested on STAGING Before clicking re split led to a 500 error page. Now the user will get a success message that the job is enqueued: However, after the jobs completed I am not able to view the pages in the UV and the pages have lost their thumbnails. Additionally, if I try to download one of the pages it says it cant be found: cc @laritakr could you have a look and pull this back for rework if more needs to be done here? |
The remaining problem is that Hyrax's In BatchCreateJob below, the array of upload_ids ["1639", "1640"] should iterate and create two CreateWorkJobs, with attributes
The logic in Hyrax has not changed anytime recently, and it doesn't make sense why this would be happening:
|
Refs: notch8/palni_palci_knapsack#229 notch8/palni_palci_knapsack#80 Adds a new decorator to override the IiifPrint FileSetPresenterDecorator and prevent the resplit PDF button from appearing. This is a temporary fix until we can get the resplit button to work.
Additional notes to above: CreateWorkJob is submitted correctly with parameters. As it runs it receives an error |
Fixed by samvera/hyku#2445 and #231 |
The "Resplit this pdf" button on the Fileset page produces a "You are not authorized to access this page" for a superadmin user, example: https://nicdon.palni-palci-staging.notch8.cloud/concern/parent/f7a1fe4c-c553-420d-9205-0402d577df4a/file_sets/3e3e018f-096c-486d-8dc2-22b3cd665d7a
Acceptance Criteria
Screenshots or Video
Hyku Main:
![Image](https://private-user-images.githubusercontent.com/10081604/405808198-704bbb36-c96a-4bcd-9bed-cd7db2059f4d.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzMTYwOTYsIm5iZiI6MTczOTMxNTc5NiwicGF0aCI6Ii8xMDA4MTYwNC80MDU4MDgxOTgtNzA0YmJiMzYtYzk2YS00YmNkLTliZWQtY2Q3ZGIyMDU5ZjRkLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjExVDIzMTYzNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWI4M2E4MDI3NWY3OWQ2OGJhYTgyZTNhOTFhZTFiNzFlZWI2MDk0Y2Q1OTAwNjM0NWU4MTE1M2RlYjAzZDM3NmImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.bG88L2CJt8k8DJxhmBF_Ucy_0o6Sm5oosrpVoBZX6T4)
Testing Instructions
Create a work with a PDF (be sure feature flipper is set so the PDF splits into child works).
Go to the PDF file set page and click the option to re-split the pdf.
Notes
v3 no longer supports active fedora
The text was updated successfully, but these errors were encountered: