<div dir="ltr">I'm concerned that that does too much work, because the Function and in the Instruction will be checked again elsewhere. Also, the error message would be bad, since the problem of which value is used outside the module is actually somewhere else.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 23, 2015 at 8:33 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 23 December 2015 at 13:18, Keno Fischer <<a href="mailto:kfischer@college.harvard.edu">kfischer@college.harvard.edu</a>> wrote:<br>
> Somewhere we have to have the logic as to which users we can handle (those<br>
> which are embedded in a module). We can have that in the callback with a<br>
> boolean flag as to whether we could handle it, or we could have that logic<br>
> in the foreach function. The latter seemed cleaner to me, but as I said, no<br>
> strong opinions here.<br>
><br>
<br>
</span>What I am suggesting is that the helper should be a fully generic<br>
"walk every user". What do you think of the attached patch?<br>
<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>