Bug 705 – improve reporting for case of required attributes and elements that are missing

NOTE: The current preferred location for bug reports is the GitHub issue tracker.

Bug 705 - improve reporting for case of required attributes and elements that are missing


Summary:	improve reporting for case of required attributes and elements that are missing

Status:	RESOLVED FIXED

Product:	Validator.nu
Classification:	Unclassified
Component:	General
Version:	HEAD
Hardware:	All All

Importance:	P2 enhancement
Assigned To:	Nobody

URL:

Depends on:	748
Blocks:
	Show dependency tree / graph

Reported:	2010-01-04 22:54 CET by Michael[tm] Smith
Modified:	2010-11-29 14:26 CET (History)
CC List:	0 users

See Also:

Attachments
patch (9.39 KB, patch) 2010-01-04 22:54 CET, Michael[tm] Smith	Details
patch for MessageEmitterAdapter code (9.06 KB, patch) 2010-05-27 10:49 CEST, Michael[tm] Smith	Details
updated patch (9.91 KB, patch) 2010-05-30 17:20 CEST, Michael[tm] Smith	Details
Show Obsolete (2) Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Michael[tm] Smith 2010-01-04 22:54:13 CET

Created attachment 143 [details]
patch

The messages that jing emits for the case of required attributes that are missing are nearly useless. We can give users better messages by moving the required-attributes checking out of the RNC schema and into the assertions-checking code.

Comment 1 Michael[tm] Smith 2010-01-16 09:01:21 CET

build r89
http://whattf.browse.cvsdude.com/build?view=rev&revision=89

syntax r519
http://whattf.browse.cvsdude.com/syntax?view=rev&revision=519

validator r340
http://whattf.browse.cvsdude.com/validator?view=rev&revision=340

Comment 2 Michael[tm] Smith 2010-05-21 15:18:09 CEST

reverted in:

build r95
syntax 556
validato r358

Comment 3 Michael[tm] Smith 2010-05-21 15:18:59 CEST

will fix this properly by patching jing based on George Bina's patch

Comment 4 Michael[tm] Smith 2010-05-27 10:49:23 CEST

Created attachment 173 [details]
patch for MessageEmitterAdapter code

Comment 5 Michael[tm] Smith 2010-05-30 17:20:17 CEST

Created attachment 176 [details]
updated patch

Comment 6 Michael[tm] Smith 2010-11-29 14:26:49 CET

https://github.com/validator/validator/commit/d145be43efc159431729c6272e2b94334e449187