Skip Menu |

This queue is for tickets about the Net-UPS CPAN distribution.

Report information
The Basics
Id: 20540
Status: resolved
Priority: 0/
Queue: Net-UPS

People
Owner: Nobody in particular
Requestors: scoob [...] mindinmotiontech.com
Cc:
AdminCc:

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



Subject: Bug report and fix for Net-UPS-0.04
Date: Tue, 18 Jul 2006 13:09:26 -0700
To: sherzodr [...] cpan.org, bug-Net-UPS [...] rt.cpan.org
From: Christian Marcotte <scoob [...] mindinmotiontech.com>
Hi there. A while back I gave you a patch for Net-UPS-0.04 to fix the routine is_oversize (found in Package.pm). I'm afraid there as a bug in the code. I was sorting the sides of the box using regular sort which is is NOT a numerical sort... Here is the fix. File: UPS/Package.pm Routine: is_oversized Original line: my @sides = sort {$a <=> $b} ($self->length, $self->width, self->height); CORRECT LINE: # sort numerically ascending my @sides = sort {$a <=> $b} ($self->length, $self->width, $self->height); Thanks again for a great module! -- Christian ------------------------------------------------------------------ Christian Marcotte Mind In Motion Technology Chief Technologist scoob@MindInMotionTech.com (831) 621-3788 By honoring others, you are not doing a favor to others but to yourself. - Swami Muktananda
patch applied