I want to change color of my tank property "tankWarningcolor" based on the tags coming from PLC using scripting on the tankWarningColor

ALARM1 =[default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"HI_LEVEL_ALARM"
	ALARM2 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"Low_Level_Alarm"
	ALARM3 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"LEVEL ALARM"
	ALARM4 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"LOWER_HI_TEMP_ALARM"
	ALARM5 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"LOWER_LO_TEMP_ALARM"
	ALARM6 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"STRATIFICATION ALARM"
	ALARM7 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"TEMP_ALARM"
	ALARM8 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"UPPER_LO_TEMP_ALARM"
	ALARM9 = [default]Tanks/"+self.view.params.Title+"/Tank_"+str(self.view.params.TankNumber)+"testing tag"
	
		
	if (ALARM1==1 or ALARM2==1 or  ALARM==3 or ALARM4==1 or ALARM5==1 or ALARM6==1 or ALARM7==1 or ALARM8==1 or ALARM9 ==1):
		self.props.tankWarningColor = color('DE2121')
	else:
		self.props.tankWarningColor = color('FFFFFF')
	return 0

You have multiple issues. If your doing this as a script like you said then I’m sure your erroring out on the first line. For building a string everything has to be in quotes. You can probably do similar to what you have if the quotes where right but what you have shown is missing them at the start of each string. I’d just substitute into the string as I’m showing below. After you do this, you still have to read the value to use it in an if statement the way you are. If this is in python then top section should be closer to:

ALARM1 = "[default]Tanks/%s/Tank_%dHI_LEVEL_ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM2 = "[default]Tanks/%s/Tank_%dLow_Level_Alarm" % (self.view.params.Title,self.view.params.TankNumber)
ALARM3 = "[default]Tanks/%s/Tank_%dLEVEL ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM4 = "[default]Tanks/%s/Tank_%dLOWER_HI_TEMP_ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM5 = "[default]Tanks/%s/Tank_%dLOWER_LO_TEMP_ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM6 = "[default]Tanks/%s/Tank_%dSTRATIFICATION ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM7 = "[default]Tanks/%s/Tank_%dTEMP_ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM8 = "[default]Tanks/%s/Tank_%dUPPER_LO_TEMP_ALARM" % (self.view.params.Title,self.view.params.TankNumber)
ALARM9 = "[default]Tanks/%s/Tank_%dtesting tag" % (self.view.params.Title,self.view.params.TankNumber)

tags = [ALARM1,ALARM2,ALARM3,ALARM4,ALARM5,ALARM6,ALARM7,ALARM8,ALARM9]

ALARMS = system.tag.readBlocking(tags)

ALARM1 = ALARMS[0].value
ALARM2 = ALARMS[1].value
ALARM3 = ALARMS[2].value
ALARM4 = ALARMS[3].value
ALARM5 = ALARMS[4].value
ALARM6 = ALARMS[5].value
ALARM7 = ALARMS[6].value
ALARM8 = ALARMS[7].value
ALARM9 = ALARMS[8].value

For the bottom section that you used the formatting on here, I’m assuming “ALARM==3” should be “ALARM3==1”.

I’m also not sure what the return is for. Unless your calling a function I don’t believe it is needed. Where this is writing direct to a prop, I’d assume it isn’t needed.