This PR can be summarized in the following changelog entry:
Internal linking suggestions should show a warning within a content element with Language "All" (-1) instead of throwing an exception
Relevant technical choices:
Moved the getLanguageById(.... call into the try/catch block to catch the InvalidArgumentException, if the $locale is null the warning is rendered within the template
Test instructions
This PR can be tested by following these steps:
Pull branch
Add a content element with Language "All"
Check if the warning is shown within the Internal Linking Suggestions
Quality assurance
[X] I have tested this code to the best of my abilities
[ ] I have added unittests to verify the code works as intended
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
getLanguageById(....
call into thetry
/catch
block to catch theInvalidArgumentException
, if the$locale
isnull
the warning is rendered within the templateTest instructions
This PR can be tested by following these steps:
Quality assurance
Fixes #543