Bug 840

Summary: [Chameleon - Widgets] Some nav tool based widgets are double calling constructor
Product: Chameleon Reporter: William Bronsema <wbronsema@dmsolutions.ca>
Component: WidgetAssignee: assefa yewondwossen <assefay@gmail.com>
Status: RESOLVED FIXED    
Severity: normal CC: assefay@gmail.com, chameleon-dev@lists.maptools.org
Priority: P2    
Version: 1.99   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description From 2004-11-11 13:45:41 
Just for the record, I have found a number widgets that inadvertantly call their
parent's constructor twice.

i.e.
parent::NavTool();
$this->NavTool();

This probably happened when the code was updated to call the parent contructor
properly using the parent::NavTool method.  The old code was just not removed.

I have fixed this in the ZoomIn, ZoomOut, Extract, PanMap, MONPhotoSearch,
MONQuery, Query, ROIMode, Recenter, Ruler, ZoomAllLayers widgets.  Please fix
any other widgets that you find this problem in.
------- Comment #1 From 2004-11-11 13:46:09 ------- 
Marking as fixed.
------- Comment #2 From 2004-11-11 13:51:34 ------- 
Just realized that I included fixes for a few of Kevin Flanders widgets. 
Assefa, can you make these changes to the MONPhotoSearch and MONQuery widgets?

Thanks.
------- Comment #3 From 2004-11-12 11:50:48 ------- 
Corrected in MONPhotoSearch and MONQuery.