Make public properties on ConfigEntityBase protected

Let's see what breaks.

No idea why we have that status handling. If a config entity wants status, they specify a key. They all currently use 'status', which defaults to TRUE.

Added this to ConfigEntityBase help debug tests:

+ function __get($name) {
+ throw new \Exception($name);
+ }

Okay, took this into a testing issue, this should pass (without the __get hack).

1. +++ b/core/lib/Drupal/Core/Entity/ContentEntityDatabaseStorage.php
   @@ -1589,7 +1589,7 @@ static protected function _generateFieldTableName(FieldConfigInterface $field, $
          $entity_type = substr($field->entity_type, 0, 34);
   -      $field_hash = substr(hash('sha256', $field->uuid), 0, 10);
   +      $field_hash = substr(hash('sha256', $field->uuid()), 0, 10);
          $table_name = $entity_type . $separator . $field_hash;
        }
   

   This overlaps with #2144263: Decouple entity field storage from configurable fields, which will introduce a new method for this (as it also needs to work for non-entity field definitions there)

2. +++ b/core/modules/block/src/BlockViewBuilder.php
   @@ -85,7 +85,7 @@ public function viewMultiple(array $entities = array(), $view_mode = 'full', $la
              $entity->id(),
   -          $entity->langcode,
   +          $entity->get('langcode'),
   

   This is usually done as language()->id, we need a langcode() method.. :)

Looks good to me.

Committed and pushed to 8.x. Thanks!