Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 291394

Decouple Drupal\Core\Condition from Drupal\Core\Executable

$
0
0

Follow up from #1743686: Condition Plugin System. It doesn't make sense to me that Condition extends Executable (in any sense: interface, base class, or manager). Evaluating a condition is not conceptually similar to executing something.

Furthermore, it's a WTF that $condition->execute() returns the exact opposite of $condition->evaluate() when the 'negate' configuration is true. $condition->evaluate() should take all its configuration into account, including 'negate'.


Viewing all articles
Browse latest Browse all 291394

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>