Bug #27345 for WWW-Facebook-API: bug in Base.pm

Wed May 30 12:16:22 2007 jianing [...] gmail.com - Ticket created

Subject:	bug in Base.pm
Date:	Wed, 30 May 2007 09:15:42 -0700
To:	bug-www-facebook-api [...] rt.cpan.org
From:	"Jianing Hu" <jianing [...] gmail.com>

at line 136, shouldn't "split /=/, $_" be "split /=/, $_, 2"? since the parameter may contain '=' itself, e.g., a URL.

Wed May 30 12:30:45 2007 unobe@cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #27345] bug in Base.pm
Date:	Wed, 30 May 2007 09:29:52 -0700
To:	Jianing Hu via RT <bug-WWW-Facebook-API [...] rt.cpan.org>
From:	David Romano <unobe [...] cpan.org>

Hi Jianing, Jianing Hu via RT wrote on Wed, May 30, 2007 at 09:16:23AM PDT: Show quoted text

> at line 136, shouldn't "split /=/, $_" be "split /=/, $_, 2"? since > the parameter may contain '=' itself, e.g., a URL.

Thanks for the feedback! You're right that split should have a limit. Someone else also brought this up, too, and I have provided a fix in the latest version available from CPAN (0.1.4). - David -- "Hope lies in dreams; in imagination and in the courage of those who dare to make dreams into reality." -- Jonas Salk

Wed May 30 12:30:48 2007 The RT System itself - Status changed from 'new' to 'open'

Wed May 30 12:34:13 2007 unobe [...] cpan.org - Correspondence added

Fixed in 0.1.4

Wed May 30 12:34:17 2007 unobe [...] cpan.org - Status changed from 'open' to 'resolved'

Wed May 30 12:36:20 2007 jianing [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #27345] bug in Base.pm
Date:	Wed, 30 May 2007 09:33:55 -0700
To:	bug-WWW-Facebook-API [...] rt.cpan.org
From:	"Jianing Hu" <jianing [...] gmail.com>

Thanks David. That was fast. You Rock! - Jianing On 5/30/07, unobe@cpan.org via RT <bug-WWW-Facebook-API@rt.cpan.org> wrote: Show quoted text

> > <URL: http://rt.cpan.org/Ticket/Display.html?id=27345 > > > Hi Jianing, > Jianing Hu via RT wrote on Wed, May 30, 2007 at 09:16:23AM PDT:

> > at line 136, shouldn't "split /=/, $_" be "split /=/, $_, 2"? since > > the parameter may contain '=' itself, e.g., a URL.

> Thanks for the feedback! You're right that split should have a limit. > Someone else also brought this up, too, and I have provided a fix in > the latest version available from CPAN (0.1.4). > > - David > > -- > "Hope lies in dreams; in imagination and in the courage of those who > dare to make dreams into reality." > -- Jonas Salk > > >

Wed May 30 12:36:27 2007 The RT System itself - Status changed from 'resolved' to 'open'

Wed May 30 19:39:49 2007 unobe [...] cpan.org - Status changed from 'open' to 'resolved'