- Issue created by @Anna D
- Status changed to Needs review
about 2 years ago 9:10pm 4 April 2023 The last submitted patch, 2: 3352242-tmgmt-state-02.patch, failed testing. View results β
- Status changed to Needs work
about 2 years ago 9:55pm 4 April 2023 - π¨πSwitzerland berdir Switzerland
That changes is wrong, you can't mix !empty() with a ?:
The reason we use that is because that stores the chosen moderation state, if you get this as an error then maybe your source did not have a valid moderation state when a translation was requested?
Fine with changing to !empty() but you mist keep the ternary as it is now otherwise.
- Status changed to Needs review
almost 2 years ago 2:53pm 5 April 2023 The last submitted patch, 5: 3352242-tmgmt-state-03.patch, failed testing. View results β
- Status changed to Needs work
almost 2 years ago 3:44pm 5 April 2023 - πΊπ¦Ukraine Anna D
maybe your source did not have a valid moderation state when a translation was requested
Content moderation was added before tmgmt and creating nodes, so all nodes should had at least default state on requesting translation. I suppose that some exception happened during translation request to the node, because how to explain that only 1 from lets say 100 nodes is broken.
- πΊπ¦Ukraine Anna D
I'm wondering why test is failed, should I check if isset && !empty ?
- Status changed to RTBC
14 days ago 5:03pm 21 March 2025 - π΅πΉPortugal jrochate
Thanks.
I was getting this fatal error, and the #5 solved it.