- 🇷🇸Serbia ratvas Novi Sad
I highly recommend to avoid solution #3 (or #26) as removing constraints is not a solution, but workaround which can cause other issues.
Solution still needs to be found, or at least more precise log message in order to point developer in a right direction.
In my case, I was going through the /admin/config/regional/content-language list and check one by one entity and field so that brought me the result. I had some iframe field checked for translation and unchecking that field allowed me to translate content without issues. - Status changed to Needs review
over 1 year ago 1:05pm 11 August 2023 - last update
over 1 year ago Custom Commands Failed - 🇺🇦Ukraine bvakulin
This is raw decision for solution #a) (from description of this problem). Needs review this patch.
- last update
over 1 year ago 29,463 pass, 2 fail The last submitted patch, 67: 2955321_content_moderation_content_translation_02.patch, failed testing. View results →
- Status changed to Needs work
over 1 year ago 8:08am 14 August 2023 - First commit to issue fork.
- Merge request !6301Issue #2955321 by niknak: Unable to change non-translatable field value on... → (Open) created by jurgenhaas
- 🇩🇪Germany jurgenhaas Gottmadingen
I've turned this into an MR and fixed some code style issues from the patch.
However, the PHPUnit tests are failing since the new code requires a service which is not available in the test context. I guess, this needs dependency injection instead of loading the service with
\Drupal::service('...')
. - Status changed to Needs review
12 months ago 11:08am 24 January 2024 - 🇩🇪Germany jurgenhaas Gottmadingen
Phew, a couple of iterations and I got all the test green. This is now ready for review.
- Status changed to Needs work
12 months ago 6:58pm 28 January 2024 - 🇺🇸United States smustgrave
As a bug will need test coverage to make sure it doesn't break in the future.
- 🇩🇪Germany tobiasb Berlin
Fyi: The patch file is the history of all commits. Better is to use just the diff for composer-patches.
- 🇮🇳India basavarajhavaler
The patch is solving the issue but I have noticed a new Issue.
Steps to reproduce :
- Add multilingual capability to site with at least 2 languages.
- Have moderation enabled.
- Create a node & publish in default language.
- Go to node translate tab, Click on another language to add translation in NEW tab.
- Try to add content - draft -> needs review -> Publish
- But It wont allow to publish It throws error. - The submitted value needs_approval in the Change to element is not allowed.
- 🇮🇳India basavarajhavaler
The patch is solving the issue but I have noticed a new Issue.
Steps to reproduce :
- Add multilingual capability to site with at least 2 languages.
- Have moderation enabled.
- Create a node & publish in default language.
- Go to node translate tab, Click on another language to add translation in NEW tab.
- Try to add content - draft -> needs review -> Publish
- But It wont allow to publish It throws error. - The submitted value needs_approval in the Change to element is not allowed.
- Assigned to basavarajhavaler
- 🇮🇳India basavarajhavaler
The patch is solving the issue but I have noticed a new Issue.
Steps to reproduce :
- Add multilingual capability to site with at least 2 languages.
- Have moderation enabled.
- Create a node & publish in default language.
- Go to node translate tab, Click on another language to add translation in NEW tab.
- Try to add content - draft -> needs review -> Publish
- But It wont allow to publish It throws error. - The submitted value needs_approval in the Change to element is not allowed.
- Issue was unassigned.
- 🇧🇪Belgium stefvanlooveren
#77 is giving me problems on a multilingual website with content moderation and paragraphs: when created the translation, paragraphs are not accessible anymore both on the default language as on the translation.
As per #2, I believe this is expected behaviour, and opting out of it for content moderated content probably isn't advisable, and would cause more issues.
I'd recommend that all development efforts are placed into 🐛 New non translatable field on translatable content throws error Needs work since that addresses the actual issue at hand (stopping it from applying data from non-translatable fields when they're inaccessible).
Depending on how the module works, using this patch can also result in loss of data if the widget returns an empty value when it detects that translations are unsupported as per my smart_date patch (which is probably why Paragraphs are a problem as per @stefvanlooveren - see #3194515-4: Smart Date Recurring doesn't work in non translatable field → )