[Xenon]

"No idea what any of it does, but it looks logical"
Very good statement

as i see you sometimes change a 0 to 1:
if it is an Ex-Employe then empactive=0 you said in the code empactive=1...

so change

PHP Code:

 if ($empactive) {

        $sqlconds .= iif($sqlconds=="","WHERE","AND")." empactive = $empactive ";

    } 


to

PHP Code:

if (!$empactive) {

        $sqlconds .= iif($sqlconds=="","WHERE","AND")." empactive = $empactive ";

    } 


also this

PHP Code:

makeinputcode("Is Ex-Employee (Inactive)","empactive",iif($empactive,$empactive,"1")); 


should be this i think:

PHP Code:

makeyesnocode("Is Employee (Aactive)","empactive",iif($empactive,$empactive,"0"));