Opened 20 years ago
Closed 20 years ago
#562 closed enhancement (wontfix)
Provide access to inline features through layerObj::getShape()
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | high | Milestone: | |
Component: | MapScript | Version: | 4.1 |
Severity: | minor | Keywords: | |
Cc: |
Description
Christoph Spoerri has provided tested code that allows MapScript access to inline features that will be incorporated into SWIG MapScript. One thing that bugs me about getShape is that the order of arguments is tileindex, shapeindex. This is exactly opposite of what it _should_ be. Since not every call needs a tileindex, it should have been the second argument so that we could make it optional.
Change History (5)
comment:2 by , 20 years ago
layerObj.getShape now allows access to inline features through the msINLINELayerGetShape function in maplayer.c. layerObj.getNumFeatures uses msLayerGetNumFeatures(). Removing features from the linked list needs a little work yet.
comment:3 by , 20 years ago
Actually, when we implemented the PHP version of getShape() we made it return a shapeObj: shapeObj getShape(int tileindex, int shapeindex)
comment:4 by , 20 years ago
That was the right way. I don't think we should change SWIG getShape now, but should wait till the next major version change. As I said in an email to Cristoph yesterday, I decided against implementing a method to remove inline features. It's no good trying to treat a linked list as if it is an array. Maybe if we ever refactor layer->features to be an array, but I don't think that is worth the effort.
comment:5 by , 20 years ago
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
Closing this bug. Just a note to mention that I am resolving the issue of an improved getShape method in bug 586. http://mapserver.gis.umn.edu/bugs/show_bug.cgi?id=586
Note:
See TracTickets
for help on using tickets.