Skip Menu |

This queue is for tickets about the WebService-Validator-HTML-W3C CPAN distribution.

Report information
The Basics
Id: 60554
Status: resolved
Priority: 0/
Queue: WebService-Validator-HTML-W3C

People
Owner: Nobody in particular
Requestors: jjnapiork [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)



Subject: Feature request: add source to Error
Hi, I was wondering if you'd mind adding the source to the Error object. This is pretty useful since it gives the report more context. Should just be something like: source => $xp->find( './m:source', $msg )->get_node(1)->getChildNode(1)->getValue, added to the error parsing and adding a source attribute. If you point me at a repo I'd be happy to give you a real patch. Thanks!
Subject: Re: [rt.cpan.org #60554] Feature request: add source to Error
Date: Sun, 22 Aug 2010 13:23:31 +0100
To: bug-WebService-Validator-HTML-W3C [...] rt.cpan.org
From: Struan Donald <struan [...] exo.org.uk>
On 20 Aug 2010, at 20 15:21, John Napiorkowski via RT wrote: Show quoted text
> Fri Aug 20 10:21:54 2010: Request 60554 was acted upon. > Transaction: Ticket created by JJNAPIORK > Queue: WebService-Validator-HTML-W3C > Subject: Feature request: add source to Error > Broken in: (no value) > Severity: Wishlist > Owner: Nobody > Requestors: jjnapiork@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=60554 > > > > Hi, > > I was wondering if you'd mind adding the source to the Error object. > This is pretty useful since it gives the report more context. Should > just be something like: > > source => $xp->find( './m:source', $msg > )->get_node(1)->getChildNode(1)->getValue, > > added to the error parsing and adding a source attribute. If you point > me at a repo I'd be happy to give you a real patch.
Hi, This sounds like a good thing. If you're willing to do the patch thing then there's a repo at: http://github.com/struan/webservice-validator-html-w3c Cheers and thanks Struan
On Fri Aug 20 10:21:54 2010, JJNAPIORK wrote: Show quoted text
> Hi, > > I was wondering if you'd mind adding the source to the Error object. > This is pretty useful since it gives the report more context. Should > just be something like: > > source => $xp->find( './m:source', $msg > )->get_node(1)->getChildNode(1)->getValue, > > added to the error parsing and adding a source attribute. If you point > me at a repo I'd be happy to give you a real patch.
And ages later this is now part of version 0.27 which should be on the CPAN shortly Struan