Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swig mapscript missing php/mapscript functions #1822

Closed
mapserver-bot opened this issue Apr 3, 2012 · 14 comments
Closed

Swig mapscript missing php/mapscript functions #1822

mapserver-bot opened this issue Apr 3, 2012 · 14 comments

Comments

@mapserver-bot
Copy link

Reporter: woodbri@swoodbridge.com
Date: 2006/07/08 - 15:15
Trac URL: http://trac.osgeo.org/mapserver/ticket/1822

Swig mapscript is missing following php/mapscript functions:

array getLayersIndexByGroup(string groupname)
array getAllGroupNames()
array getAllLayerNames() 

There may be others I have not had time to check the whole API.
@mapserver-bot
Copy link
Author

Author: sdlime
Date: 2006/07/10 - 22:18

We'll need to get the language specific folks involved with this if (as I fear) 
typemaps are necessary to go from char ** to language specific arrays. Perhaps 
this could be brought up on mapserver-dev briefly...

Steve

@mapserver-bot
Copy link
Author

Author: woodbri@swoodbridge.com
Date: 2006/08/10 - 20:11

Steve, can you try and get this fixed on 4.10? At least for Perl, but I think we
should try to get this done for all the languages if we can get the support
because it is really problematic to port mapscript between languages when there
are missing functions.

@mapserver-bot
Copy link
Author

Author: thomas.bonfort@gmail.com
Date: 2006/09/12 - 20:14

a few other missing features:
 - the constructor for a new mapobj is missing the second argument (path)
 - imageObj savewebimage function

the mapobj constructor quirk is quite important, as I haven't found out how to
circumvent this without patching the swig interface (the other missing functions
can be mimicked in mapscript) 

@mapserver-bot
Copy link
Author

Author: sdlime
Date: 2007/08/08 - 08:39
This will require some typemap work so I don't want to putz with this at this late date. Marking for 5.2...

Steve

@ghost ghost assigned sdlime Apr 5, 2012
@tbonfort
Copy link
Member

no updates, no need!

@jmckenna
Copy link
Member

jmckenna commented Feb 1, 2021

I think this would be a very useful addition for the MapServer 8.0 release: add important missing SWIG MapScript functions:

mapObj: array getLayersIndexByGroup(string groupname)
mapObj: array getAllGroupNames()
mapObj: array getAllLayerNames()

Reopening ticket.

@geographika
Copy link
Member

See also #5763 - once an array typemap is added for PHP several of these functions could be added.

@sdlime
Copy link
Member

sdlime commented Oct 3, 2022

@jmckenna and @geographika - is this still relevant?

@jmckenna
Copy link
Member

jmckenna commented Oct 3, 2022

still relevant, as mentioned through #5763

@sdlime
Copy link
Member

sdlime commented Oct 3, 2022

Do we need multiple open tickets or could we consolidate things? Just trying whittle the number of tickets down.

@geographika
Copy link
Member

The "missing" functions were never in the SWIG API, and are more convenience functions for PHP users. From the Python MapScript side I don't see a particular need for the functions mentioned in this ticket. I'd be in favour of closing this and maybe just leaving #5763 open.

@sdlime
Copy link
Member

sdlime commented Oct 4, 2022

Ok, I can live with that and call this old one a duplicate. The references back to this ticket will live on in #5763 anyway.

@sdlime sdlime closed this as completed Oct 4, 2022
@jmckenna
Copy link
Member

jmckenna commented Oct 4, 2022

This is unfortunate. Thanks guys.

@sdlime
Copy link
Member

sdlime commented Oct 4, 2022

This is unfortunate. Thanks guys.

Feel free to re-open if you feel that strongly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants