Opened 19 years ago

Closed 19 years ago

#1066 closed defect (fixed)

Add map->setSize() to PHP MapScript

Reported by: dmorissette Owned by: mapserverbugs
Priority: high Milestone: 4.4 release
Component: MapScript-PHP Version: 4.3
Severity: normal Keywords:
Cc:

Description

In bug 836, issues were reported with mapscript code setting map->width and
map->height directly without calling msMapComputeGeoTransform(). A new
mapObj.setSize() method was added to address that in SWIG MapScript and we
should add the same to PHP MapScript to bring them in sync.

However I'm not sure this method was really needed since AFAIK all other calls
in MapScript that use the width/height end up indirectly calling
msMapComputeGeoTransform().

Let's wait to hear feedback on bug 836 comment 8 before adding this to PHP
MapScript.

Change History (4)

comment:1 by dmorissette, 19 years ago

Milestone: FUTURE
Setting FUTURE target milestone.

comment:2 by dmorissette, 19 years ago

dependson: 836

comment:3 by dmorissette, 19 years ago

Milestone: FUTURE4.4 release
Status: newassigned
Frank and Sean have convinced me that it's a good thing to switch to using a
setSize() method instead of setting width/height directly.

Setting width/height directly will continue to work, but using setSize() instead
will be recommended in the docs.

comment:4 by dmorissette, 19 years ago

Resolution: fixed
Status: assignedclosed
Done. map->setSize() added to PHP MapScript.
Note: See TracTickets for help on using tickets.