Skip Menu |

This queue is for tickets about the Finance-Quote CPAN distribution.

Report information
The Basics
Id: 56401
Status: resolved
Priority: 0/
Queue: Finance-Quote

People
Owner: Nobody in particular
Requestors: MarkoSchuetz [...] web.de
Cc:
AdminCc:

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



Subject: VWD.pm needs update after structure change
Date: Thu, 08 Apr 2010 14:17:17 -0400
To: bug-Finance-Quote [...] rt.cpan.org
From: Marko Schütz <MarkoSchuetz [...] web.de>
Dear All, apparently vwd changed their web page structure recently. The <div class="wpKurs"> is no longer beneath an <h1>, but now beneath an <h2>. Line 126 in Finance::Quote::VWD.pm (of version 1.17) needs to be changed to my $head = $content->find("h2"); Best regards, Marko
Download (untitled)
application/pgp-signature 194b

Message body not shown because it is not plain text.

From: Martin Kompf
Attached is a patch against version 1.17 that fixes this issue. Additionally bug #45762 is fixed by this patch.
Subject: VWD.pm.diff
47c47 < $VERSION = '1.17'; --- > $VERSION = '1.17patched'; 125,126c125,126 < # <h1> contains price, time, name, and symbol < my $head = $content->find("h1"); --- > # <h2> contains price, time, name, and symbol > my $head = $content->find("h2"); 138,139c138,139 < if ($time) { < $info{$fund, "time"} = $time->as_trimmed_text; --- > if ($time->as_trimmed_text =~ /([012]\d:[0-5]\d:[0-5]\d)/) { > $info{$fund, "time"} = $1;
From: Thomas.Eschenbacher [...] gmx.de
just a note for the maintainer of this package: this report seems to be a duplicate of #62619
thanks for the work VWD is working through gihub pull -- Erik