id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
2014	[patch] Error messages of RequiredIfOtherField* validators are not userfriendly	Steven Armstrong	Jacob	"When working with RequiredIfOtherFieldEquals and RequiredIfOtherFieldDoesNotEqual validators the error messages can be quite user unfriendly.

Assuming you have a select-box named 'action' where the currently selected option has a value of 2 and a label of 'Forward Message to' you get error messages like:[[BR]]
''This field must be given if action is 2''

That sort of let's the user sit there feeling dumb. 2, where does that 2 come from and what does it mean ???

With the attached patch applied you can have error messages like:[[BR]]
''This field must be given if action is Forward Message to''

The label is passed as a third, optional, argument to the validators constructor.

e.g.:[[BR]]
RequiredIfOtherFieldEquals('action', 2)

becomes:[[BR]]
RequiredIfOtherFieldEquals('action', 2, 'Forward Message to')

This is a backwards compatible change.
"	enhancement	closed	Validators	dev	normal	fixed			Ready for checkin	1	0	0	0	0	0