[COCOON-1771] cocoon.ajax.Fader runtime error when style uses abreviated form #ccc in IE6.0sp2

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1.8
• Component/s: Blocks: Ajax
• Urgency: Normal
• Other Info: Patch available
• Assignee: Antonio Gallardo Rivera
• Reporter: Eric Meyer
• Created: 08/Feb/06 11:41
• Updated: 09/Feb/06 02:41
• Resolved: 09/Feb/06 02:41

Description

In cocoon.ajax.Fader

  this.toColor = cocoon.ajax.Fader.colorToRgb(cocoon.ajax.Fader.getBgColor(this.element));

getBgColor will return '#fff'

/** Converts a "#RRGGBB" color as an array of 3 ints */
cocoon.ajax.Fader.colorToRgb = function(hex) {
    return [
        parseInt(hex.substr(1,2),16),
        parseInt(hex.substr(3,2),16),
        parseInt(hex.substr(5,2),16) ];
}

Assumes that hex starts with a '#' and has 6 additional hex characters.

The corrected implementation is

/** Converts a "#RRGGBB" color as an array of 3 ints */
cocoon.ajax.Fader.colorToRgb = function(hex) {
var r = 255; // defaults if no match
var g = 255;
var b = 255;
var i=-1;
var colors = hex.match(/^#(\d{2})(\d{2})(\d{2})$/);
if (colors) {
r = parseInt(colors[++i]);
g = parseInt(colors[++i]);
b = parseInt(colors[++i]);
} else if (colors = hex.match(/^#(\d)(\d)(\d)$/)) {
r = parseInt(colors[++i] + colors[i]);
g = parseInt(colors[++i] + colors[i]);
b = parseInt(colors[++i] + colors[i]);
}
    return [r,g,b];
}

Patch attached.

Regards,
Eric Meyer, VP, Quoin, Inc.