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:2 by , 19 years ago
dependson: | → 836 |
---|
comment:3 by , 19 years ago
Milestone: | FUTURE → 4.4 release |
---|---|
Status: | new → assigned |
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 , 19 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Done. map->setSize() added to PHP MapScript.
Note:
See TracTickets
for help on using tickets.