diff -u b/core/lib/Drupal/Core/Config/Entity/Query/Condition.php b/core/lib/Drupal/Core/Config/Entity/Query/Condition.php --- b/core/lib/Drupal/Core/Config/Entity/Query/Condition.php +++ b/core/lib/Drupal/Core/Config/Entity/Query/Condition.php @@ -10,6 +10,11 @@ use Drupal\Core\Entity\Query\ConditionBase; use Drupal\Core\Entity\Query\ConditionInterface; +/** + * Defines the condition class for the config entity query. + * + * @see \Drupal\Core\Config\Entity\Query\Query + */ class Condition extends ConditionBase { /** @@ -95,8 +100,9 @@ * and the * wildcard. * @param array $parents * The current list of parents. + * * @return bool - * TRUE when matched. + * TRUE when the condition matched to the data else FALSE. */ protected function matchArray($condition, array $data, array $needs_matching, array $parents = array()) { $parent = array_shift($needs_matching); @@ -132,7 +138,7 @@ * @return bool * TRUE when matches. */ - function match($condition, $value) { + protected function match($condition, $value) { if (isset($value)) { switch ($condition['operator']) { case '=': diff -u b/core/modules/system/lib/Drupal/system/Tests/Entity/ConfigEntityQueryTest.php b/core/modules/system/lib/Drupal/system/Tests/Entity/ConfigEntityQueryTest.php --- b/core/modules/system/lib/Drupal/system/Tests/Entity/ConfigEntityQueryTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Entity/ConfigEntityQueryTest.php @@ -407,7 +407,7 @@ /** * Test dotted path matching. */ - function testDotted() { + protected function testDotted() { $this->queryResults = $this->factory->get('config_query_test') ->condition('array.level1.*', 1) ->execute(); @@ -432,11 +432,10 @@ * @param array $expected * Array of expected entity IDs. */ - function assertResults($expected) { + protected function assertResults($expected) { $this->assertIdentical(count($this->queryResults), count($expected)); foreach ($expected as $value) { - // This also tests whether $this->queryResults[$value] is even set at - // all. + // This also tests whether $this->queryResults[$value] is even set at all. $this->assertIdentical($this->queryResults[$value], $value); } }