Move BlockStorageController::loadByProperties() into ConfigStorageController

Some overlap with: #1846454: Add Entity query to Config entities

I opened a related issue recently as well: #1887058: Remove usages of entity_load_multiple_by_properties() and EntityStorageController::loadByProperties(). I was toying with the idea of removing it altogether... entity_load_multiple_by_properties() as well as StorageController::loadByProperties(). Once that EFQ patch lands they will be mere wrappers for invoking EFQ. And we really should stop causing endless trails of function / method calls / forwards for no good reason.

There was a test in taxonomy vocabularies for this method, it was commented/removed in #1552396-71: Convert vocabularies into configuration

+1 to RTBC
Patch just restores removed tests as pointed in #4

#6: 1893442-load-6.patch queued for re-testing.

Chasing HEAD

+++ b/core/lib/Drupal/Core/Config/Entity/ConfigStorageController.phpundefined
@@ -169,7 +169,13 @@ public function deleteRevision($revision_id) {
+    $entities = $this->load();
+    foreach ($values as $key => $value) {
+      $entities = array_filter($entities, function($entity) use ($key, $value) {
+        return $value === $entity->get($key);
+      });
+    }
+    return $entities;

Don't we have EQ for config entities as well? Sure it's doing basically the same, but I guess we could use it here.

Scope of the issue to move implementation to parent class

@dawehner for #9 I filed follow-up #1981516: Refactor ConfigStorageController to use entity query

I don't think it's a problem that we have both tests for vocabulary and config entity in general.

#8: 1893442-load-8.patch queued for re-testing.

+++ b/core/modules/config/lib/Drupal/config/Tests/ConfigEntityUnitTest.phpundefined
@@ -43,6 +43,30 @@ public function testStorageControllerMethods() {
+    $style = $this->randomName();
...
+      'style' => $this->randomName(),

It is possible for this test to generate a random fail as randomName() could return the same string. We have to ensure that the style value is different... i.e. set different lengths.

Here is a small fix for that, but just by looking at the test I wouldn't assume that you actually need the ID here.

Nice catch

Needs a re-roll...

curl https://drupal.org/files/drupal-1893442-14.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4784  100  4784    0     0   3326      0  0:00:01  0:00:01 --:--:--  6336
error: patch failed: core/modules/block/lib/Drupal/block/BlockStorageController.php:40
error: core/modules/block/lib/Drupal/block/BlockStorageController.php: patch does not apply

re-roll

Committed 80da26f and pushed to 8.x. Thanks!