[Chameleon-dev] [Bug 1610] New: Keypress before drawing polygon ROI causes error

[bugzilla-daemon at bugzilla.maptools.org]

http://bugzilla.maptools.org/show_bug.cgi?id=1610

           Summary: Keypress before drawing polygon ROI causes error
           Product: Chameleon
           Version: 2.4
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Widget
        AssignedTo: chameleon-dev at lists.maptools.org
        ReportedBy: jacob.delfos at maunsell.com


When the ROI Polygon tool is active, any keypress before a ROI is drawn causes a
javascript error (which in IE pops up to annoy the user). This is because any
keypress other than escape causes an attempt to close the polygon, which can't
happen if no nodes exist yet.

In ROIPolygonTool.widget.php, on line 591, it says:

                ROIPolygonToolClosePolygon();

Which I suggest be changed to:

                if (goROIPolygonObj.aNodeCoords.length > 0)
                {
                ROIPolygonToolClosePolygon();
                }

That seems to fix it.

Jacob



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


Please do NOT reply to this email, use the link above instead to 
login to bugzilla and submit your comment. Any email reply to this
address will be lost.