Bug #71949 for Shipment: Measurement system hardoced (LBS) on method _build

Wed Oct 26 05:41:52 2011 paul.vlasov [...] gmail.com - Ticket created

Subject:	Measurement system hardoced (LBS) on method _build_services Shipment::UPS 0.01112650
Date:	Wed, 26 Oct 2011 13:41:43 +0400
To:	bug-Shipment [...] rt.cpan.org
From:	Pavel Vlasov <paul.vlasov [...] gmail.com>

Good morning! When I try to get services between two cities in Europe (ex. Berlin and Zurich). 36 my $shipment = Shipment::UPS->new( ....); 51 foreach my $service ( $shipment->all_services ) { 52 print $service->id . " (" . $service->name . " | ".$service->cost . ")\n"; 53 } I had error "This measurement system is not valid for the selected country". I try to investigate this error and found that Shipment/UPS.pm consist on method "_build_services" line 221, hardoced Code => 'LBS'. So for European country it should be 'KGS'. Could you may to check it? My environment: Module: Shipment::UPS 0.01112650 Perl version: This is perl, v5.10.0 built for i386-linux-thread-multi OS: Linux 2.6.18-028stab091.2 #1 SMP Fri Jun 3 00:02:40 MSD 2011 i686 i686 i386 GNU/Linux Thanks you. -- Best regards, Pavel Vlasov Please consider the environment before printing this email.

Fri Dec 09 11:20:22 2011 andrew [...] pullingshots.ca - Taken

Fri Dec 09 11:20:35 2011 andrew [...] pullingshots.ca - Status changed from 'new' to 'open'

Fri Dec 09 13:08:49 2011 andrew [...] pullingshots.ca - Correspondence added

As of v0.01113430, the _build_services method of Shipment::UPS will respect what has been set in the weight_unit attribute. To override the default weight unit (lb), you can do: my $shipment = Shipment::UPS->new( ... weight_unit => 'kg', ); For details about the fix, please see https://github.com/pullingshots/Shipment/commit/eea224f84612fffaab6e75fb7d73a550d3292fba

Fri Dec 09 13:08:50 2011 andrew [...] pullingshots.ca - Status changed from 'open' to 'resolved'

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Fixed in 0.01113430 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01111360 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01111361 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01111450 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01111510 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01111730 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01112490 added

Fri Dec 09 17:45:33 2011 andrew [...] pullingshots.ca - Broken in 0.01112650 added

Sat Dec 10 02:03:44 2011 paul.vlasov [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #71949] Measurement system hardoced (LBS) on method _build_services Shipment::UPS 0.01112650
Date:	Sat, 10 Dec 2011 10:03:35 +0300
To:	bug-Shipment [...] rt.cpan.org
From:	Pavel Vlasov <paul.vlasov [...] gmail.com>

Good morning Andrew! Thank you! What do you think if I fork your project and add feature (that show soap request reply or responce for debugging) On Fri, Dec 9, 2011 at 9:08 PM, Andrew Baerg via RT < bug-Shipment@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=71949 > > > As of v0.01113430, the _build_services method of Shipment::UPS will > respect what has been set in the weight_unit attribute. > > To override the default weight unit (lb), you can do: > > my $shipment = Shipment::UPS->new( > > ... > > weight_unit => 'kg', > ); > > For details about the fix, please see > > https://github.com/pullingshots/Shipment/commit/eea224f84612fffaab6e75fb7d73a550d3292fba >

-- Best regards, Pavel Vlasov Please consider the environment before printing this email.

Sat Dec 10 02:03:45 2011 The RT System itself - Status changed from 'resolved' to 'open'

Sat Dec 10 12:34:12 2011 andrew.baerg [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #71949] Measurement system hardoced (LBS) on method _build_services Shipment::UPS 0.01112650
Date:	Sat, 10 Dec 2011 10:33:48 -0700
To:	bug-Shipment [...] rt.cpan.org
From:	Andrew Baerg <andrew.baerg [...] gmail.com>

Of Course! That is why it is published on Github... you can fork it and then do a pull request so that your features can be brought into the ditribution on cpan. If you have any questions, do not hesitate to ask. Thanks! Andrew On Sat, 10 Dec 2011, 00:03:45 MST, Pavel Vlasov via RT <bug-Shipment@rt.cpan.org> wrote: Show quoted text

> Queue: Shipment > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=71949 > > > Good morning Andrew! > > Thank you! > What do you think if I fork your project and add feature (that show soap > request reply or responce for debugging) > > On Fri, Dec 9, 2011 at 9:08 PM, Andrew Baerg via RT < > bug-Shipment@rt.cpan.org> wrote: >

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=71949 > > > > > As of v0.01113430, the _build_services method of Shipment::UPS will > > respect what has been set in the weight_unit attribute. > > > > To override the default weight unit (lb), you can do: > > > > my $shipment = Shipment::UPS->new( > > > > ... > > > > weight_unit => 'kg', > > ); > > > > For details about the fix, please see > > > > https://github.com/pullingshots/Shipment/commit/eea224f84612fffaab6e75fb7d73a550d3292fba > >

> > > > -- > Best regards, Pavel Vlasov > Please consider the environment before printing this email. >

Sat Dec 10 18:49:15 2011 andrew [...] pullingshots.ca - Status changed from 'open' to 'resolved'

Bug #71949 for Shipment: Measurement system hardoced (LBS) on method _build_services Shipment::UPS 0.01112650