Support %nspace for PHP

[jdeniau]

Hello,

It would be quite nice if Swig could manager PHP namespaces.
Il is quite similar to the java namespacing.

Thank you !

[ojwb]

This has already been raised in the SWIG SF feature request tracker: http://sourceforge.net/p/swig/feature-requests/81/

At this point, it mostly just needs somebody to implement it...

[ojwb]

I've just merged Nihal's GSOC 2017 branch which has some commented out code for namespace support:

Source/Modules/php.cxx:    //if (nameSpace != NULL)
Source/Modules/php.cxx:      //Printf(s_oinit, "INIT_CLASS_ENTRY(internal_ce, \"%s\\\\%s\", class_%s_functions);\n", nameSpace, class_name, class_name);
Source/Modules/php.cxx-    //else
Source/Modules/php.cxx-    Printf(s_oinit, "  INIT_CLASS_ENTRY(internal_ce, \"%s%s\", class_%s_functions);\n", prefix, class_name, class_name);

It seems just specifying the class name including the namespace prefix works, as I've used that to name the classes we now use to wrap non-class pointer types with a SWIG\ namespace prefix.

So now this probably "just" needs hooking up to SWIG's %nspace mechanism.

[jdeniau]

To be honest I dont remember why I used swig, but thank you for the follow up on this issue ! 😲

[mnguyenpdftron]

@ojwb I'm also interested in using namespace. Now that php target file is gone when using swig 4.1, I have no clue how to have a namespace for our PHP wrapper.

[ojwb]

It's probably not something I'll find time to work on for a while I'm afraid. If you want to try to implement it yourself, looks like getNSpace() is useful:

  /* Return the namespace for the class/enum - the nspace feature */
  String *getNSpace() const;

[ojwb]

I don't think closing as "not planned" is appropriate. There's not a timescale for implementing it, but it's definitely something we ought to support (the first part is true for the majority of open tickets here, the second true for quite a lot of them).

If you aren't interested in notifications, just unsubscribe from the issue.

[jdeniau]

@ojwb sorry I was closing my issues of unmaintained package, I didn't checked out that swig was still under active development. Sorry 🙏