Hi,<div><br></div><div>I&#39;ve been struggling with a Magritte validation problem. I find that a condition I add (using #addCondition:labelled:) to a MABooleanDescription is only intermittently called.</div><meta charset="utf-8"><meta charset="utf-8"><div>
<br></div><div>I&#39;ve simplified the conditions to reproduce the problem to a component description comprising of two MABooleanDescription descriptions. The second description includes the condition which is intermittently called.</div>
<div><br></div><div>If the value of the first description is true (ie the checkbox is checked) the second description&#39;s condition will be executed, however if it is false it will never be executed.</div><div><br></div>
<div>I suspect the problems lies in the code in  MAValidatorVisitor&gt;&gt;#visitContainer: which calls MAGraphVisitor&gt;&gt;#use:during: which is as follows:</div><div><br></div><div><div>MAGraphVisitor &gt;&gt;use: anObject during: aBlock</div>
<div><span class="Apple-tab-span" style="white-space:pre">	</span>| previous |</div><div><span class="Apple-tab-span" style="white-space:pre">	</span>(seen includes: anObject)</div><div><span class="Apple-tab-span" style="white-space:pre">		</span>ifTrue: [ ^ self ].</div>
<div><span class="Apple-tab-span" style="white-space:pre">	</span>anObject isNil</div><div><span class="Apple-tab-span" style="white-space:pre">		</span>ifFalse: [ seen add: anObject ].</div><div><span class="Apple-tab-span" style="white-space:pre">	</span>previous := object. object := anObject.</div>
<div><span class="Apple-tab-span" style="white-space:pre">	</span>aBlock ensure: [ object := previous ]</div></div><div><br></div><div>If the first check box is false, and the second is also false then (seen includes: anObject) will return true and method returns early resulting in block associated with second description (self visit: description) never being executed. </div>
<div><br></div><div>I&#39;ve attached a file out of a simple component TestMagritteValidation illustrating the problem that I tested in the latest build from Hudson.</div><div><br></div><div>Thanks</div><div><br></div><div>
Nick </div><meta charset="utf-8">