- 🇧🇪Belgium swentel
The translated entity should probably be loaded in the checkAccess method too, although it's really an edge case we've experienced in combination with the Content Language Access module ( https://www.drupal.org/project/content_language_access → ).
If the original entity is passed, the language of that entity might not be equal to the current content language.
Again, very edge case, but good to know if other people run into that, although it's probably related with #2976381: Downloaded PDFs for translation display data in original language → too.
- First commit to issue fork.
- 🇮🇳India shalini_jha
I have faced one more issue with the translated node View PDF link is also showing access denied for the translated node.
- 🇮🇳India shalini_jha
I have tested Patch #2, and the file name generation is based on the current language. As mentioned in #4, these translated nodes need to load in the checkAccess method as well. I encountered an issue with the download PDF link; it was showing "access denied" for translated nodes because the current code was checking in the default language, thus causing permission denied errors. To resolve this, I have added the same logic to the checkAccess method and tested it, confirming that it now works for translated nodes.
I have updated the Patch #2.
- Status changed to RTBC
9 months ago 2:38pm 15 November 2024 - Status changed to Needs work
about 1 month ago 4:56am 15 July 2025 - 🇦🇺Australia jannakha Brisbane!
thank you for your contribution.
please provide a merge request for easier testing and review for the latest version - First commit to issue fork.
- 🇦🇺Australia jannakha Brisbane!
@solariel is this still an issue for you?
can you please test/review MR82?