Issue #2346433 by yched: Fixed iterators omit computed fields by default.

core/lib/Drupal/Core/Entity/ContentEntityBase.php

@@ -383,7 +383,7 @@ public function set($name, $value, $notify = TRUE) {
   /**
    * {@inheritdoc}
    */
-  public function getFields($include_computed = FALSE) {
+  public function getFields($include_computed = TRUE) {
     $fields = array();
     foreach ($this->getFieldDefinitions() as $name => $definition) {
       if ($include_computed || !$definition->isComputed()) {

core/lib/Drupal/Core/Entity/ContentEntityInterface.php

@@ -196,7 +196,7 @@ public function set($field_name, $value, $notify = TRUE);
    * @return \Drupal\Core\Field\FieldItemListInterface[]
    *   An array of field item lists implementing, keyed by field name.
    */
-  public function getFields($include_computed = FALSE);
+  public function getFields($include_computed = TRUE);
 
   /**
    * Reacts to changes to a field.

core/lib/Drupal/Core/Entity/ContentEntityStorageBase.php

@@ -180,7 +180,7 @@ protected function invokeTranslationHooks(ContentEntityInterface $entity) {
   protected function invokeFieldMethod($method, ContentEntityInterface $entity) {
     foreach (array_keys($entity->getTranslationLanguages()) as $langcode) {
       $translation = $entity->getTranslation($langcode);
-      foreach ($translation->getFields(TRUE) as $field) {
+      foreach ($translation->getFields() as $field) {
         $field->$method();
       }
     }