Make JobItem::saveTranslation update the item state

Looks pretty fine.

1. +++ b/sources/tmgmt_config/src/Tests/ConfigSourceUiTest.php
   @@ -104,28 +104,22 @@ class ConfigSourceUiTest extends EntityTestBase {
   -    // Allow auto-accept.
   -    $default_translator = Translator::load('test_translator');
   -    $default_translator
   -      ->setSetting('auto_accept', TRUE)
   -      ->save();
   -
   

   Hm, it seems this test was pretty wrong... Indeed below there is no auto_accept behaviour expected.

2. +++ b/src/Entity/JobItem.php
   @@ -750,13 +750,16 @@ class JobItem extends ContentEntityBase implements JobItemInterface {
   +      // We don't know if the source plugin was able to save the translation
   +      // after this point. That means that the plugin has to set the 'accepted'
   +      // states on its own.
   

   Not true anymore. Make a proper statement.

+++ b/sources/tmgmt_config/src/Tests/ConfigSourceUiTest.php
@@ -229,16 +223,16 @@ class ConfigSourceUiTest extends EntityTestBase {
-        $this->assertEqual('Spanish' || 'German', (string) $value->td[1]);
+        $this->assertEqual('Spanish', (string) $value->td[1]);

Much nice 'Spanish' || 'German' ;-)
What confuses me is that still we have submitted two jobs to the translator, from English to both German and Spanish. As a result, i would expect both should be "needs review".

Yes the revert and the fixing of the button is fine.

But you stumbled accross this:

+++ b/sources/tmgmt_config/src/Tests/ConfigSourceUiTest.php
@@ -104,16 +104,16 @@ class ConfigSourceUiTest extends EntityTestBase {
+        $this->assertEqual('Spanish' || 'German', (string) $value->td[1]);

This means the assert doesn't do anything valuable at all. 'Spanish' || 'German' is simply TRUE.
Since you are fixing this test, you should make it test something useful with the "Needs review" links.

+++ b/src/Entity/JobItem.php
@@ -750,13 +750,15 @@ class JobItem extends ContentEntityBase implements JobItemInterface {
   public function acceptTranslation() {
-    if (!$this->isNeedsReview() || !$plugin = $this->getSourcePlugin()) {

I would prefer an early exit with !$plugin
Rest looks fine to me! :-)

I would have loved to commit this, but the test shows that we are missing the node_access schema in ContentEntitySourceUnitTest::testAcceptTranslation().