Invoke field methods first on the current entity translation

+++ b/core/lib/Drupal/Core/Entity/ContentEntityStorageBase.php
@@ -436,6 +436,14 @@ protected function invokeFieldMethod($method, ContentEntityInterface $entity) {
+      $key = array_search($current_entity_langcode, $langcodes);
+      unset($langcodes[$key]);

We can do it in one line. $langcodes = array_diff($langcodes, [$current_entity_langcode]);

I think this patch makes sense. I don't think it is a backwards compatibility problem to better define an order that was not defined before. Nonetheless I asked Berdir and plach to look at this, if either of them are available :)

Anyway, no need to block this on them I think :)

Random fail.

fail: [Browser] Line 248 of core/modules/system/src/Tests/Update/UpdatePathTestBase.php:
GET http://localhost/checkout/update.php/start?id=2&op=do_nojs returned 0 (0 bytes).

fail: [Other] Line 264 of core/modules/system/src/Tests/Update/UpdatePathTestBase.php:
Schema key block.block.bartik_account_menu:settings.cache failed with: missing schema

fail: [Other] Line 264 of core/modules/system/src/Tests/Update/UpdatePathTestBase.php:
Schema key block.block.bartik_breadcrumbs:settings.cache failed with: missing schema
...

Git checkout error on testbot.

Yet another random fail.

This looks pretty solid. Looking at ContentEntityStorageBase the other similar invocation by language is ::invokeTranslationHooks() - should we make it behave similarly? I.e starting with the current entity's language? Maybe that doesn't make much sense because this is about firing the translation_insert and translation_delete hooks.

In discussion with @effulgentsia, @catch, @xjm and @cottser we decided that this patch should probably only go in 8.2.x as changing the order hooks are fired in is a behaviour change and this is a task not a bug.

Committed e0af755 and pushed to 8.2.x. Thanks!

Removing from sprint, thanks all again!