This project is archived and is in readonly mode.

#648 ✓invalid
Samori Gorse

Array#hexToRgb dependency bug in Core Builder

Reported by Samori Gorse | April 28th, 2009 @ 11:28 AM | in 2.0 (closed)

Hi all,

When building Core from the Core Builder, selecting the Array package does not select the String package while Array#hexToRgb depends on String#toInt.

Steps to reproduce :

-Go to http://www.mootools.net/core

-Select Array and download the file (Core is automatically added, String is not)

-Add the according script element to a test page

-Try to execute ['11','22','33'].hexToRgb(); (from the docs)

-Exception triggered : "TypeError: value.toInt is not a function"

Comments and changes to this ticket

  • Scott Kyle

    Scott Kyle April 28th, 2009 @ 06:35 PM

    I audited Array.js and that is the only dependency to String.js that I could find, and therefore I believe that it is justified to change that call to parseInt() to maintain modularity.

    However, the other problem is that in String.js, hexToRgb and rgbToHex depend on Array.js, so maybe they should just be cross-dependent. Or moved into their own module. Or moved into Color.js in More...

  • Christoph Pojer

    Christoph Pojer April 30th, 2009 @ 05:58 PM

    • State changed from “new” to “invalid”

    MooTools 2.0 will have a completely new and more awesome Color.js and this will be resolved then

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

People watching this ticket

Pages