Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 364504

Summary:

ValidationDemo in org.eclipse.edt.rui.samples_0.7.0 doesn't display all errors

Product:

z_Archived

Reporter:

broy2

Component:

EDT

Assignee:

Brian Svihovec <svihovec>

Status:

CLOSED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

jqian, svihovec

Version:

unspecified

Target Milestone:

---

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
EDT	none
RBD	none
Screen shot	none

Description broy2

2011-11-22 13:49:50 EST

Created attachment 207378 [details]
EDT

Open gallery.validation.ValidationDemo in org.eclipse.edt.rui.samples_0.7.0.
Preview.
Click Submit.
There should be 8 errors, but there are only 2.
Screen shots of RBD and EDT attached.

Comment 1 broy2

2011-11-22 13:50:14 EST

Created attachment 207379 [details]
RBD

Comment 2 Jing Qian

2011-11-22 15:03:44 EST

we don't support dataitem in 070, also we don't support custom message properties.

So the sample should be modified

change the record definition
any field that is dataitem type should be changed to primitive and its validation annotation should be added for it. 

i.e. 



record accountRecord
	username string { MinimumInput=6, MinimumInputMsgKey = "usrMinInput", validationPropertiesLibrary=ValidationMessages };
	password string { MinimumInput = 8, MinimumInputMsgKey = "pwMinInput", validationPropertiesLibrary=ValidationMessages };
	email string;
	firstName String {inputRequired = yes};
	lastName String {inputRequired = yes};
	birthday date { dateFormat="MM/dd/yy", TypeChkMsgKey = "invalidDate", validationPropertiesLibrary=ValidationMessages };
	address String {inputRequired = yes};
	city String {inputRequired = yes};
	state String {inputRequired = yes, uppercase = yes };
	zip int{@isDecimalDigit, typeChkMsgKey = "validZip", validationPropertiesLibrary=ValidationMessages};
end

Comment 3 Brian Svihovec

2011-11-29 14:10:11 EST

Removed usage of the Data item for now.  I left in the data item for future use, and added a comment about why it is not being used in this release.

Note that the error messages are still the default messages, since validationPropertiesLibrary is not supported in the .7 release.  I have added this note in the code as well.

Comment 4 broy2

2011-11-29 15:51:32 EST

Verified, but looks really tacky.
Screen shot attached.

Comment 5 broy2

2011-11-29 15:57:54 EST

Created attachment 207690 [details]
Screen shot