- Issue created by @andreastkdf
- 🇬🇧United Kingdom andreastkdf
Added MR targeting 10.x: ensures we skip a task before rendering if necessary properties are missing.
- 🇺🇸United States smustgrave
Fixes should be against 11.x
Also will need test coverage
That said may need backtrace more vs an empty check.
- Status changed to Needs work
3 months ago 3:05pm 14 January 2025 - 🇪🇸Spain plopesc Valladolid
Hello @andreastkdf
Tried to reproduce the issue installing Gin and Navigation in a Umami vanilla site where content translation is enabled. The issue is not happening there.
Could you please provide further information to have clear steps to reproduce the issue? Tried installing devel module, but I was unable to reproduce the issue.
MR looks good but that information would be helpful to determine the root cause of the issue and how to define the tests.
Thank you.
- Status changed to Closed: works as designed
1 day ago 8:44am 8 April 2025 - 🇬🇧United Kingdom scott_euser
Actually I think this bug is related to the code in Page to PDF module, raised an issue here that stops this fatal error 🐛 Compatibility with navigation module Active . @andreastkdf feel free to reopen if not, but I believe this should be closed here.
- 🇬🇧United Kingdom scott_euser
Well I suppose the issue summary is suggesting to add this check just in case some other contrib modules end up doing the same and incorrectly structuring local tasks, so up to maintainers if you wish to add this to make the code more protective about its expectations of structure. In any case it solves it (or masks the problem):
- Enable translation eg on article
- Install Page to PDF module → 1.0.2 or earlier
- Enable page to pdf for article
- Attempt to translate an existing article