Skip Menu |
 

This queue is for tickets about the HTML-FormatText-WithLinks-AndTables CPAN distribution.

Report information
The Basics
Id: 55919
Status: open
Priority: 0/
Queue: HTML-FormatText-WithLinks-AndTables

People
Owner: Nobody in particular
Requestors: jeff_hammock [...] fulfillmentcorp.com
Cc:
AdminCc:

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



Subject: HTML::FormatText::WithLinks::AndTables
Date: Wed, 24 Mar 2010 16:50:55 -0700
To: <bug-html-formattext-withlinks-andtables [...] rt.cpan.org>
From: "Jeff Hammock" <jeff_hammock [...] fulfillmentcorp.com>
Download (untitled) / with headers
text/plain 1.5k
Shaun First off thanks for the module! When I first tried to use the module I kept getting the following error: Can't call method "content" on an undefined value at C:/Perl/site/lib/HTML/FormatText/WithLinks/AndTables.pm line 215. I tracked the problem down to the first row in my HTML table that contained only <TH> tags. In looking in the .pm I noticed the comment # no support for <th>. sorry. on 2 lines (137 & 170). After playing around for a while I ended up changing both lines from $tr->look_down(_tag=>'td') to $tr->look_down(_tag=>qr/(td|th)/). Now I am no longer getting the error and the contents of both my <TD> and <TH> tags do show up in the plain text. However I do not know what I might be breaking with this modification. The module will still die with the above error if an HTML table contains a Table row without any <TR> or <TH> tags, i.e. "<TR></TR>". Again thanks for a great little module. Thanks Jeff Hammock Senior Programmer Analyst Fulfillment Corporation of America - Corporate 11065 SW 11th St. # 300 • Beaverton OR 97005 p: 503.906.4101 • d: 503.906.4128 • f: 503.906.4102 OREGON - CALIFORNIA - COLORADO - HAWAII This e-mail is for the sole use of the intended recipient and contains information belonging to sender, which is confidential. This e-mail does not include an electronic signature and is not intended to constitute a legally effective offer or to establish a contract. If you have received this e-mail in error, please immediately notify the sender by reply e-mail and destroy all copies of the original message.
From: jeffery.hammock [...] gmail.com
Download (untitled) / with headers
text/plain 371b
Sorry, forgot to mention I'm using Version 0.01. I have also added an 'If Ref' condition around line 215 to resolve the 'Can't call method "content" on...' error. Again I do not know if anything is getting broken with this tweak. if (ref($cols[0])) { $cols[0]->content->[0] = "__TOKEN__${table_count}__${row_count}__"; # place a token into the row at col 0 } Jeff


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.