>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #1968118. --> Reported by: [soapboxcicero](https://www.drupal.org/user/2111168) >>> <h2>Description</h2> <p>Rules Collections adds the ability to loop in Rules Conditions. </p> <p>There are two kinds of loops: Any and All.</p> <p>Any evaluates to true if any item in the list causes its loop body to evaluate to true.</p> <p>All evaluates to true if every item in the list causes its loop body to evaluate to true.</p> <h2 id="motivation">Motivation</h2> <p><a href="https://drupal.org/project/commerce">Commerce</a> uses <a href="https://drupal.org/project/rules">Rules</a> to control the majority of its functions. For all events involving an Order entity it is often necessary to alter behavior depending on the kinds of LineItem entities are in the order. For example, only enable shipping if <strong>any</strong> line item in the order meets some criteria for 'being shippable'. This can be awkward or can even involve dropping down into PHP within the rule. But it is simple enough to encode the store's criteria for being shippable as a set of conditions on the line item. The only thing missing is a way to loop over a list in a condition. </p> <p>This module closes that gap in a generic way that is not dependent on Commerce, only Rules itself. </p> <h2 id="shortcomings">Known shortcomings</h2> <ul> <li><del>The "+Add any" and "+Add all" links in the condition pane come before the usual "+Add condition" link as they appear to always be displayed in alphabetical order</del> (fixed, thanks to mitchell)</li> <li><del>The "+Add any" and "+Add all" links appear in the links for any and all loops though Rules does not support lists of lists</del> (rules does support lists of lists, my mistake)</li> </ul> <h2 id="patch">Why not make this a patch to Rules</h2> <p>Integrating this directly into Rules would be tricky. To do it properly there would be some refactoring necessary in Rules' core. For example, the UI plugin for the Any and All loops is derived from RulesLoopUI which is a RulesActionContainerUI not a RulesConditionContainerUI and then the functionality of RulesConditionContainerUI (a negation checkbox) is bolted on. Similarly, the code in the base class for Any and All is genetically related to both RulesLoop and RulesConditionContainer. </p> <p>This isn't a problem in practice, despite its inelegance, and works quite well. It does however point to the need for some changes in Rules' core in order to support the features provided by this module without any code duplication or any such infelicities. I do not see a way to do this without breaking the API of Rules at this time, though I am by no mean an expert in Rules' codebase and found most of this out trying to get my module to work :). </p> <p>Providing the functionality as a separate module does three things:</p> <ol> <li>It let's those who require this functionality now to get it.</li> <li>It will act as a barometer for the need of this functionality (ratio of installs of this module to installs of the rules module). If it remains small, then there's little point in making such sweeping changes to appease a small group who can simply install this module.</li> <li>It demonstrates the areas that need to be refactored in Rules' core in order to support such functionality, should it be deemed worth the effort.</li> </ol> <h2 id="requirements">Requirements</h2> <ul> <li>Rules: <a href="https://drupal.org/project/rules">https://drupal.org/project/rules</a></li> </ul> <h2 id>Resources</h2> <ul> <li>Project page: <a href="https://drupal.org/sandbox/soapboxcicero/1966034">https://drupal.org/sandbox/soapboxcicero/1966034</a></li> <li>Issue queue: <a href="https://drupal.org/project/issues/1966034">https://drupal.org/project/issues/1966034</a></li> <li>Git repo:<br> <pre>git clone --branch 7.x-1.x http://git.drupal.org/sandbox/soapboxcicero/1966034.git rules_collections<br>cd rules_collections</pre></li> <li>Parview Online Review Link: <a href="http://ventral.org/pareview/httpgitdrupalorgsandboxsoapboxcicero1966034git">http://ventral.org/pareview/httpgitdrupalorgsandboxsoapboxcicero1966034git</a></li> </ul> <p>(Note that the documentation errors from parview are related to it not recognizing {@inheritdoc} which I read was the standard here: <a href="https://drupal.org/node/1354#classes">https://drupal.org/node/1354#classes</a> and the only other error is about the name of a Rules method I am overloading and have no control over)</p>