- Issue created by @RoloDMonkey
- Merge request !6276Issue #3416304 - check length of array before trying to use first index → (Closed) created by RoloDMonkey
- Status changed to Needs review
11 months ago 5:57pm 22 January 2024 - Status changed to Needs work
11 months ago 2:19pm 25 January 2024 - 🇺🇸United States smustgrave
Have not tested but MR appears to have failures. Is the issue fixed in 🐛 contentTranslationDependentOptions behavior doesn't check if found element array is empty RTBC ? May need test coverage.
- 🇺🇸United States RoloDMonkey
As far as I can tell, this issue is not fixed by 🐛 contentTranslationDependentOptions behavior doesn't check if found element array is empty RTBC . That is a very similar issue in the same file, but not in the same place.
The merge request "failure" was just some suggested whitespace changes. I have made those changes and everything is passing now.
- Status changed to Needs review
11 months ago 3:12pm 25 January 2024 - Status changed to Needs work
11 months ago 8:18pm 29 January 2024 - 🇺🇸United States smustgrave
Since this is a very reproducible issue think it would be good to get a test case. Probably could extend an existing one with an additional assertion.
- First commit to issue fork.
- Status changed to Needs review
10 months ago 11:13pm 27 February 2024 - Status changed to RTBC
10 months ago 2:25pm 28 February 2024 - 🇺🇸United States smustgrave
Test coverage can be seen here https://git.drupalcode.org/issue/drupal-3416304/-/jobs/919261
Manually testing fixes the issue.
- Status changed to Fixed
10 months ago 8:09pm 29 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.