Unable to load php_MapGuideApi.dll when there are multiple PHP directories in the system PATH

[ronnielouie]

When configured with IIS or Apache, the system PATH is used to find the location of dependent DLLS for the php_MapGuideApi.dll. When there are multiple locations for PHP in the system PATH, the first PHP occurrence will be used which is not necessarily correct.

To resolve this issue, php_MapGuideApi.dll and its dependencies should be able to load without relying the PATH environment variable.

[tomfukushima]

I see that this is addressed by r3265. It creates a new dll called php_MapGuideHelper, but I'm not too keen on this somewhat generic name. How about php_MapGuideApiLoader? Any other suggestions or opinions?

[ronnielouie]

I think php_MapGuideApiLoader may be a little misleading because the extension doesn't load the php_MapGuideApi. Since its purpose is to set the system such that the MapGuideApi can successfully load, how about php_MapGuideApiConfig?

[tomfukushima]

Oh, OK, I thought it loaded the php_MapGuideApi dll. I like your suggestion.

[ronnielouie]

Renamed php_MapGuideHelper to phpMapGuideApiEnvConfig. See r3266.

Remaining tasks for this ticket are to update the installer for windows to include the new extension, update the php.ini file to load the extension, and to no longer add PHP to the system PATH.

Also the Building and Configuring MGOS on Windows document should be updated to add instructions for loading the new extension via php.ini and to omit the step for adding PHP to the PATH.

[ronnielouie]

Assigning to Tim for the installer tasks

[crispinatime]

Hi,

This also fixes issue 454...

​http://trac.osgeo.org/mapguide/ticket/454#comment:20

[jng]

The new installer configures php.ini to load this new extension.

Any adventurous soul wants to try the new installer to see if thi scenario is covered?

[jbirch]

I've tested this with my latest 2.1 build, running PHP 5.3 earlier in my path than MapGuide's PHP, and MapGuide amazingly continues to work. Closing; please re-open if you see different results.

[tomfukushima]

This was fixed by r3265 and r3266