Skip Menu |

This queue is for tickets about the WebService-Pushover CPAN distribution.

Report information
The Basics
Id: 84653
Status: resolved
Priority: 0/
Queue: WebService-Pushover

People
Owner: shuff [...] cpan.org
Requestors: alec [...] edworthyuk.co.uk
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: v0.1.0
Fixed in: v0.1.1



Subject: Unable to locate push method
Date: Tue, 16 Apr 2013 07:34:09 +0100
To: bug-webservice-pushover [...] rt.cpan.org
From: Alec Edworthy <alec [...] edworthyuk.co.uk>
Hi, Sorry, I just realised that my previous email didn't go to the bugs address but to you personally. As such I've re-sent it below. Since the 0.1.0 upgrade you mentioned in your previous message I can't send push notifications anymore, *** alec@aeserv:~/tmp$ ./test.pl Can't locate object method "push" via package "WebService::Pushover" at ./ test.pl line 15. alec@aeserv:~/tmp$ cat test.pl #!/usr/bin/perl -w use WebService::Pushover; my $push = WebService::Pushover->new(debug => 1) or die( "Unable to instantiate WebService::Pushover.\n" ); my %params = ( token => 'xxx', user => 'xxx', title => '', sound => 'none', ); my $status = $push->push( %params, message => 'Perl modules rock!' ); foreach my $key (keys %{$status}) { print "\$status->{$key} = " . $status->{$key} . "\n"; } alec@aeserv:~/tmp$ *** Am I doing something wrong? Thanks, Alec -- Alec Edworthy alec@edworthyuk.co.uk <javascript:_e({}, 'cvml', 'alec@edworthyuk.co.uk');> -- Alec Edworthy alec@edworthyuk.co.uk
Subject: Re: [rt.cpan.org #84653] AutoReply: Unable to locate push method
Date: Tue, 16 Apr 2013 08:35:48 +0100
To: bug-WebService-Pushover [...] rt.cpan.org
From: Alec Edworthy <alec [...] edworthyuk.co.uk>
OK, I think I'm getting somewhere. It appears the method for sending a message has changed from push to message, as in for a correctly instantiated WebService::Pushover object referenced by $push with a correctly populated %params variable, $status = $push->push( %params ); becomes $status = $push->message( %params ); The installation of WebService::Pushover also has an unresolved dependency on Net::HTTP::Spore::Middleware::Header, Can't locate Net/HTTP/Spore/Middleware/Header.pm in @INC (@INC contains: /usr/local/lib64/perl5 /usr/local/share/perl5 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at ./test.pl line 16. After installing that module and updating my script to call message rather than push I get success! Alec
Subject: Re: [rt.cpan.org #84653] AutoReply: Unable to locate push method
Date: Tue, 16 Apr 2013 08:41:55 +0100
To: bug-WebService-Pushover [...] rt.cpan.org
From: Alec Edworthy <alec [...] edworthyuk.co.uk>
As an addendum it appears you need to set debug to either 0 or 1, leaving it out complete results in a failure. See attached output where WebService::Pushover was instantiated by, my $push = WebService::Pushover->new or die( "Unable to instantiate WebService::Pushover.\n" ); Setting debug to 0, as in, my $push = WebService::Pushover->new(debug => 0) or die( "Unable to instantiate WebService::Pushover.\n" ); results in successful and quiet operation, setting it to 1 results in lots of useful debug output and success once again. Alec

Message body is not shown because sender requested not to inline it.

yeah, this is documentation-fail on my part, plus a couple of additional fails. look for 0.1.1 soon! thanks again for reporting. -shuff -- Steve Huff (SHUFF) * shuff@cpan.org
Subject: Re: [rt.cpan.org #84653] Unable to locate push method
Date: Fri, 19 Apr 2013 14:52:25 +0100
To: bug-WebService-Pushover [...] rt.cpan.org
From: Alec Edworthy <alec [...] edworthyuk.co.uk>
On 19 April 2013 14:51, Steve Huff via RT <bug-WebService-Pushover@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=84653 > > > yeah, this is documentation-fail on my part, plus a couple of additional fails. look for 0.1.1 soon! > > thanks again for reporting.
No problem, thanks for fixing :-) A -- Alec Edworthy alec@edworthyuk.co.uk
On Fri Apr 19 09:51:43 2013, SHUFF wrote: Show quoted text
> yeah, this is documentation-fail on my part, plus a couple of > additional fails. look for 0.1.1 soon!
ok, 0.1.1 pushed to GitHub. i'm uploading to CPAN now; please let me know if this fixes the issues you see. -shuff -- Steve Huff (SHUFF) * shuff@cpan.org
Subject: Re: [rt.cpan.org #84653] Unable to locate push method
Date: Fri, 19 Apr 2013 17:00:29 +0100
To: bug-WebService-Pushover [...] rt.cpan.org
From: Alec Edworthy <alec [...] edworthyuk.co.uk>
On 19 April 2013 15:35, Steve Huff via RT <bug-WebService-Pushover@rt.cpan.org> wrote: Show quoted text
> ok, 0.1.1 pushed to GitHub. i'm uploading to CPAN now; please let me know if this fixes the issues you see.
The CPAN index available via the shell is a few hours out of date. I'll try it again later and confirm that it works for me. Thanks for your help, Alec -- Alec Edworthy alec@edworthyuk.co.uk
Subject: Re: [rt.cpan.org #84653] Unable to locate push method
Date: Fri, 19 Apr 2013 23:09:23 +0100
To: bug-WebService-Pushover [...] rt.cpan.org
From: Alec Edworthy <alec [...] edworthyuk.co.uk>
On 19 April 2013 17:00, Alec Edworthy <alec@edworthyuk.co.uk> wrote: Show quoted text
> On 19 April 2013 15:35, Steve Huff via RT > <bug-WebService-Pushover@rt.cpan.org> wrote:
>> ok, 0.1.1 pushed to GitHub. i'm uploading to CPAN now; please let me know if this fixes the issues you see.
> > The CPAN index available via the shell is a few hours out of date. > I'll try it again later and confirm that it works for me.
0.1.1 looks good to me. No need to explicitly declare debug anymore. Speaking as someone who's written a few scripts which make use of WebService::Pushover it's a shame that the push method had to be replaced by message as it means anyone who used previous versions of the module will need to re-code their scripts to call the new method. Otherwise all is good and thank you for writing the module and being so responsive to bug reports :-) Alec -- Alec Edworthy alec@edworthyuk.co.uk