Skip Menu |

This queue is for tickets about the MailClientYahoo CPAN distribution.

Report information
The Basics
Id: 18796
Status: open
Priority: 0/
Queue: MailClientYahoo

People
Owner: Nobody in particular
Requestors: spacedboy2k
Cc:
AdminCc:

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



Subject: doesn't work at all
hi, Method select_folder dies with: Can't call method "rows" on an undefined value at C:/Perl/site/lib/HTML/TableExtract.pm line 226. Since you can't do anything unless a folder is selected I guess this means this module won't work at all. It's quite a useful module so a fix would be nice. thanks, Jon. p.s. WinXP, ActiveState perl 5.8.4, MailClientYahoo v1.0.
From: PJS [...] cpan.org
The attached patch is a start on fixing this. It is all I need for the limited functionality I use in this module, so it almost certainly isn't enough. But it'll get you further along.
--- Yahoo.pm.old 2006-10-12 13:58:10.000000000 -0700 +++ Yahoo.pm 2006-10-12 14:32:26.000000000 -0700 @@ -156,7 +156,7 @@ ); $self->{step} = 'Redirection'; - while(my $redir = $mech->res->header('Location')) { + while(my ($redir) = $mech->content =~ /content="0; url=([^"]+)/) { $mech->get($redir); } if ( $mech->content =~ m#window.location.replace\("([^"]*?)"# ) { @@ -195,7 +195,8 @@ $self->{step} = 'Folder Stats Retrieval'; - $self->{mech}->get($URLs{folders}); + # $self->{mech}->get($URLs{folders}); + $self->{mech}->follow_link(text => 'Edit'); my @stats = keys %{$Patterns{folder_stats}}; my @cols = @{$Patterns{folder_stats}}{@stats}; @@ -213,6 +214,7 @@ $stats{unread} = $1 if $stats{unread} =~ $Patterns{folder_unread}; %{$self->{folder_list}{$stats{name}}} = %stats; } + $self->{mech}->back; }