-
Notifications
You must be signed in to change notification settings - Fork 7
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
Error: Call to a member function getPath() on null #221
Comments
Thank's for reporting this. I also saw these messages on my dev machine but I thought they are related to some test data I created earlier. Will try to dig deeper into that ASAP. |
@robsmy98 I think I found a fix for that. Would you mind testing the current patch from https://github.com/R0Wi-DEV/workflow_ocr/suites/15285716874/artifacts/872905928 ? |
I could try it in one week, thank you in advance :)
Robin Windey ***@***.***> schrieb am So. 20. Aug. 2023 um
22:17:
… @robsmy98 <https://github.com/robsmy98> I think I found a fix for that.
Would you mind testing the current patch from
https://github.com/R0Wi-DEV/workflow_ocr/suites/15285716874/artifacts/872905928
?
—
Reply to this email directly, view it on GitHub
<#221 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AWF3UIMTY6GRNXREWD6WO2DXWJWHDANCNFSM6AAAAAA3PNDKDU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
That seemed to work for me. |
* Fix Notifications not being processed #221 * Add Migration to remove Notification + additional tests Signed-off-by: Robin Windey <[email protected]> --------- Signed-off-by: Robin Windey <[email protected]>
* Fix Notifications not being processed #221 * Add Migration to remove Notification + additional tests Signed-off-by: Robin Windey <[email protected]> --------- Signed-off-by: Robin Windey <[email protected]>
Thanks for your feedback @farhills! Even though I could not reproduce this behaviour after the patch, I added a migration step which should remove the pending Notifications when upgrading to the next app version. Fix will be shipped with |
* Fix Notifications not being processed #221 * Add Migration to remove Notification + additional tests --------- Signed-off-by: Robin Windey <[email protected]>
After the Update from 27.0.1 to 27.0.2 I get this message 10 times in 1 minute in my protocol:
System
ocrmypdf
version: 14.0.1+dfsg1Server log
The text was updated successfully, but these errors were encountered: