nusoap icon indicating copy to clipboard operation
nusoap copied to clipboard

Fixing bug with _transient gobal variable

Open seismicmike opened this issue 9 years ago • 2 comments

Hi!

I've been using your nusoap package from Packagist for a while now and it's worked pretty well for me. Today, I came across a weird issue. I was getting an error about an undefined index "_transient". I googled and came up with http://stackoverflow.com/questions/14607718/multiple-nusoap-clients-causes-undefined-index-transient-error

Apparently somebody called Scardetto had already patched this in their fork of nusoap but rather than trying to somehow trace that back to something that I could use with composer, I just decided to fork and patch your code.

So anyway, I figured I'd send this back upstream in case you're intreested. Here's the patch. You can pretty much ignore the changes to .gitignore, README, composer.json, etc.The material change is in lib/class.nusoap_base.php

Let me know if you have any questions. Thanks

seismicmike avatar Jan 26 '16 15:01 seismicmike

I've been coming across this error when using PHPUnit with the backupGlobals configuration value set to TRUE. Disabling this configuration setting resolves the error being generated. Not sure why this isn't simply a static property to avoid this issue.

williamorazi avatar Apr 20 '16 21:04 williamorazi

Interesting. I don't recall having any problems with this outside the PHPUnit context, so that's probably the culprit. I'll test with that disabled sometime and see if it makes a difference.

Thanks for the feedback!

seismicmike avatar May 04 '16 16:05 seismicmike