> IF !EMPTY(hospital.phonecall) OR ; > !EMPTY(hospital.typecall) OR ; > ALLT(hospital.newstatus)<>"OPEN" OR ; > hospital.phonepic = "Y" > THIS.lblCallNow.Visible = .T. > *Red phone > FOR i = 1 to 14 > pictures="THIS.ImgPhone"+(i)+".Picture" > heights="THIS.ImgPhone"+(i)+".Height=14" > widths="THIS.ImgPhone"+(i)+".Width=16" > IF recno()=i > * Phone > &pictures = (PhoneBMP) > &heights=14 > &widths=16 > ELSE > *No phone > &pictures = (NoPhoneBMP) > &heights=14 > &widths=16 > ENDIF > ENDFOR > ENDIF >In the pictures/Heights/Widths lines you are concatinating a string and a numeric [i]. Below you don't need the = number since you have created them in the strings.
IF !EMPTY(hospital.phonecall) OR ; !EMPTY(hospital.typecall) OR ; ALLT(hospital.newstatus)<>"OPEN" OR ; hospital.phonepic = "Y" THIS.lblCallNow.Visible = .T. *Red phone FOR i = 1 to 14 pictures="THIS.ImgPhone"+alltrim(str(i))+".Picture" heights="THIS.ImgPhone"+alltrim(str(i))+".Height=14" widths="THIS.ImgPhone"+alltrim(str(i))+".Width=16" IF recno()=i * Phone &pictures = PhoneBMP &heights &widths ELSE *No phone &pictures = NoPhoneBMP &heights &widths ENDIF ENDFOR ENDIFBest of luck