W3C home > Mailing lists > Public > public-qa-dev@w3.org > November 2005

Re: CSS validator's soap output should give uri context for each message

From: Yves Lafon <ylafon@w3.org>
Date: Tue, 08 Nov 2005 07:45:49 +0000
To: olivier Thereaux <olivier@tokyoartbeat.com>
Cc: QA Dev <public-qa-dev@w3.org>
Message-ID: <Pine.GSO.4.63.0511080841570.3480@gnenaghyn.vaevn.se>




On Mon, 31 Oct 2005, olivier Thereaux wrote:

> Hi Yves, all.
>
> While coding in new functionalities into the perl lib that accesses the CSS 
> validator's SOAP output, I noticed that the uri context is apparently only 
> given at the start of the <m:warninglist> or <m:errorlist>. I suggest it 
> should be given for each message (error, warning), even if it makes the 
> output more verbose.

Well, there may be multiple <m:warninglist> and <m:errorlist>, each list 
gives the uri context in it, for all the item parts of this list.

If URIs are put in each warning or error, the list grouping makes less 
sense.
Both version (the current one and the  proposed one) have the same 
expressiveness, so it's just a matter of style and how people intend to 
use this output.

>
> Rationale: when validating a document linking to different stylesheets, the 
> errors and warnings may come from several sources. Knowing exactly which 
> errors come from which document/location is therefore very important.
>
> so instead of :
>
> <m:warninglist>
>  <m:uri>http://www.example.org/style.css</m:uri>
>    <m:warning>
>      <m:line>24</m:line>
>      <m:level>1</m:level>
>      <m:message>You have no background-color with your color</m:message>
>      <m:context>A:link</m:context>
>    </m:warning>
>    <m:warning>
>      <m:line>30</m:line>
>      <m:level>1</m:level>
>      <m:message>You have no background-color with your color</m:message>
>      <m:context>A:visited</m:context>
>    </m:warning>
> </m:warninglist>
>
> I suggest
> <m:warninglist>
>    <m:warning>
>      <m:uri>http://www.example.org/style.css</m:uri>
>      <m:line>24</m:line>
>      <m:level>1</m:level>
>      <m:message>You have no background-color with your color</m:message>
>      <m:context>A:link</m:context>
>    </m:warning>
>    <m:warning>
>      <m:uri>http://www.example.org/style.css</m:uri>
>      <m:line>30</m:line>
>      <m:level>1</m:level>
>      <m:message>You have no background-color with your color</m:message>
>      <m:context>A:visited</m:context>
>    </m:warning>
> </m:warninglist>
>
> If you think this makes sense, here's a patch (attached) which I will commit 
> unless you think it's a bad idea.
>
> Thank you.
>

-- 
Yves Lafon - W3C
"Baroula que barouleras, au tiéu toujou t'entourneras."
Received on Tuesday, 8 November 2005 08:31:02 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Thursday, 19 August 2010 18:12:45 GMT