I am pretty sure you could get rid of the dynamic sql at the very least. Isn't this this same thing for Check 1?

/* check 1 */

IF @Check1 is null

insert into Discrepancies

(

FormName,

IDNumber,

VisitDate,

ErrorDetail,

ErrVarName1,

Status

)

select @FormName,

@PatientID,

@VisitDate,

'enr_first cannot be null',

'enr_first',

'Open'

where not exists

(

select *

from Discrepancies

where FormName = @FormName

and IDNumber = @PatientID

and VisitDate = @VisitDate

and ErrorDetail = @ErrorDetail

)

It is hard to figure out exactly what your code is doing but I think the loop would be fairly easy to get rid of too.