Yes, it did crash on that form and any form that uses that section of code but only when running in Access 2013, runs fine in Access 2010.
Arrhhh thank you, thank you.
I'm not the author of the code. I will have a play and see if I can work it out based on what I need it to do. Maybe I can comment block it and see what the results are.
As far as writing an alternative to these parts, It's way above my head. Generally, I can work out what is happening in the code but have trouble working out why. I do want to learn though.
The forms are intended to be used as a check list for the different components as named. Not all items in the list may be applicable which accounts for the result Not Inspected. The results are recorded back into the tables for later reference.
I comment blocked the looping section in the On Load event. I think that is where the problem is.
This part of the code brings the results back into the form to allow me to change Failed to Passed results hopefully, on the next visit to the property.