Opened 13 years ago

Closed 13 years ago

#1427 closed defect (fixed)

Fix for array param case in dojo.gfx.color.hsv2rgb

Reported by: mde@… Owned by: Tom Trenka
Priority: high Milestone:
Component: General Version: 0.3
Keywords: Cc:
Blocked By: Blocking:

Description

Param names are reused as variables in the function -- in the case of an array as the first param, the current code overwrites the optional options object passed in as a second parameter.

Doing the check for options first fixes the problem.

Attachments (1)

dojo_gfx_color_hsv-mde-20060908.patch​ (456 bytes) - added by mde@… 13 years ago.
Fix for array param case in dojo.gfx.color.hsv2rgb

Download all attachments as: .zip

Change History (2)

Changed 13 years ago by mde@…

Attachment: dojo_gfx_color_hsv-mde-20060908.patch​ added

Fix for array param case in dojo.gfx.color.hsv2rgb

comment:1 Changed 13 years ago by Tom Trenka

Resolution: → fixed
Status: new → closed

(In [5645]) Fixes #1427.

Note: See TracTickets for help on using tickets.