Используйте блок else
.
if ( $current_user->s_managed == 'Yes' ) { //make this a string not a constant
return '<div align="right"><img src="abc.com/onswitch20.jpg"> </div>' ;
}else{
return '<div align="right"><img src="abc.com/offswitch20.jpg"></div>' ;
}
Поскольку это логическое значение, мы можем просто использовать else
, вы хотите использовать No
в другом, потому что мы можем использовать 0
или No
или Null
Обычно все, что falsy
. Это также будет похоже на планочное время (наименьшая возможная единица времени) быстрее, поскольку необходимо проверить только одно условие.
Другой, еще лучший способ написать это так:
function ajax_13xx()
return '<div align="right"><img src="abc.com/'.(wp_get_current_user()->s_managed == 'Yes' ? 'on' : 'off').'switch20.jpg"> </div>' ;
}
Тогда вы сохраните свой код немного более СУХИМ (не повторяйте себя)