That procedure would have to be a function and not a sub if I'm thinking along the same lines. However, I don't see what's wrong with that code line, but "doesn't work" doesn't help much. If all of the controls are bound, what would you need the current event for? That's a missing puzzle piece? As for that code being simpler
If [FailedVisual] Or [FailedNDI] Or [PartMissing] Or [Retired] Then [PartNeeded] = True
The more we hear silence, the more we begin to think about our value in this universe.
Paraphrase of Professor Brian Cox.