Skip Menu |

This queue is for tickets about the Games-WoW-Armory CPAN distribution.

Report information
The Basics
Id: 27340
Status: resolved
Priority: 0/
Queue: Games-WoW-Armory

People
Owner: Nobody in particular
Requestors: sabine.hurka [...] freeda.info
Cc:
AdminCc:

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



Subject: not really a bug...
Date: Wed, 30 May 2007 11:21:58 +0200
To: bug-games-wow-armory [...] rt.cpan.org
From: Sabine Hurka <sabine.hurka [...] freeda.info>
... but it would be nicer, if your check for the region (EU or US) wouldnot be case sensitive in sub fetch_data: my $base_url; if ( $$params{ country } eq "EU" ) { $base_url = $WOW_EUROPE; } elsif ( $$params{ country } eq "US" ) { $base_url = $WOW_US; } else { croak "Unknow region code, please choose US or EU"; } I use 0.0.3 and only took a look in http://cpan.uwinnipeg.ca/htdocs/Games-WoW-Armory/Games/WoW/Armory.pm.html to know why it didnot want to work with "eu". Greetings Sabine
From: FRANCKC [...] cpan.org
Hi Thanks, fixed this in the new 0.0.5 release. Show quoted text
> ... but it would be nicer, if your check for the region (EU or US) > wouldnot be case sensitive in sub fetch_data: > > my $base_url; > if ( $$params{ country } eq "EU" ) { > $base_url = $WOW_EUROPE; > } > elsif ( $$params{ country } eq "US" ) { > $base_url = $WOW_US; > } > else { > croak "Unknow region code, please choose US or EU"; > } > > I use 0.0.3 and only took a look in > http://cpan.uwinnipeg.ca/htdocs/Games-WoW-Armory/Games/WoW/Armory.pm.html > to know why it didnot want to work with "eu". > > Greetings > Sabine > >
Fixed in 0.0.5 release