Hi folks!<br><br>Today, at work, a friend ask me about a method in TestResource because he thought it was a bug. We look at it, and for us, it could be a bug. So, I ask to you.<br><br>See TestResource class side isAvailable<br>
<br>------------------------<br>isAvailable<br>��� ^self current notNil and: [self current isAvailable]<br>------------------------ <br><br>the first part (self current notNil) ALWAYS return true because of the implementation� of self current:<br>
<br>------------------------<br>current<br><br>��� current isNil<br>��� ��� ifTrue: [current := self new].<br><br>��� ^current<br>-------------------------<br><br><br>The second part too. Because the implementation of self available (object side method):<br>
<br>-------------------------<br>isAvailable<br>��� &quot;override to provide information on the<br>��� readiness of the resource&quot;<br>��� <br>��� ^true<br>-------------------------<br><br>Ok, you can override and change this method in subclasses, however, the first part is ALWAYS true. <br>
<br>Is this correct ? or isAvailable should be:<br><br>isAvailable<br>
��� ^ self current isAvailable<br><br><br>Thanks!<br><br>Mariano<br>��� ��� ��� <br>