Skip Menu |

This queue is for tickets about the Graphics-ColorPicker CPAN distribution.

Report information
The Basics
Id: 59193
Status: resolved
Priority: 0/
Queue: Graphics-ColorPicker
People
Owner: Nobody in particular
Requestors: spoon.reloaded [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: 0.11
Fixed in: (no value)

History Show all quoted text
  Thu Jul 08 01:37:29 2010 https://launchpad.net/~newacct - Ticket created
Subject: eval() should be changed to parseInt()
rgb[0] = eval('0x' + s.charAt(0) + s.charAt(1)); // r rgb[1] = eval('0x' + s.charAt(2) + s.charAt(3)); // g rgb[2] = eval('0x' + s.charAt(4) + s.charAt(5)); // b should be changed to rgb[0] = parseInt(s.slice(0,2), 16); // r rgb[1] = parseInt(s.slice(2,4), 16); // g rgb[2] = parseInt(s.slice(4,6), 16); // b
  Thu Jul 08 16:39:23 2010 michael [...] bizsystems.com - Correspondence added
parseInt is not a function in any version of perl that I have through 12.1 Even if it exists in a newer version it is not backwards compatible what exists in the field.
  Thu Jul 08 16:39:24 2010 The RT System itself - Status changed from 'new' to 'open'
  Thu Jul 08 16:39:24 2010 michael [...] bizsystems.com - Status changed from 'open' to 'rejected'
  Fri Jul 09 12:05:35 2010 https://launchpad.net/~newacct - Correspondence added
From: spoon.reloaded [...] gmail.com
On Thu Jul 08 16:39:23 2010, MIKER wrote: Show quoted text
> parseInt is not a function in any version of perl that I have through 12.1 > > Even if it exists in a newer version it is not backwards compatible what > exists in the field.
The code in question is Javascript (which is printed from perl), not perl
  Fri Jul 09 12:05:36 2010 The RT System itself - Status changed from 'rejected' to 'open'
  Wed May 18 18:21:21 2011 michael [...] bizsystems.com - Correspondence added
Patch applied Graphics-ColorPicker-0.12.tar.gz uploaded to CPAN Thanks to spoon.reloaded@gmail.com for the code improvement On Fri Jul 09 12:05:35 2010, https://launchpad.net/~newacct wrote: Show quoted text
> On Thu Jul 08 16:39:23 2010, MIKER wrote:
> > parseInt is not a function in any version of perl that I have
through 12.1 Show quoted text
> > > > Even if it exists in a newer version it is not backwards compatible what > > exists in the field.
> > The code in question is Javascript (which is printed from perl), not perl
  Wed May 18 18:21:21 2011 michael [...] bizsystems.com - Status changed from 'open' to 'resolved'