Bug 840 – [Chameleon - Widgets] Some nav tool based widgets are double calling constructor

Bug 840 - [Chameleon - Widgets] Some nav tool based widgets are double calling constructor

Summary:

[Chameleon - Widgets] Some nav tool based widgets are double calling constructor

Status:	RESOLVED FIXED

Product:	Chameleon
Component:	Widget
Version:	1.99
Platform:	PC Windows XP

Importance:	P2 normal
Target Milestone:	---
Assigned To:	assefa yewondwossen

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2004-11-11 13:45 by William Bronsema
Modified:	2004-11-12 11:50 (History)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From William Bronsema 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 William Bronsema 2004-11-11 13:46:09 -------

Marking as fixed.

------- Comment #2 From William Bronsema 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 assefa yewondwossen 2004-11-12 11:50:48 -------

Corrected in MONPhotoSearch and MONQuery.