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

Notice: Undefined variable: prefix #26

Open
zawaruddin opened this issue Mar 14, 2018 · 6 comments
Open

Notice: Undefined variable: prefix #26

zawaruddin opened this issue Mar 14, 2018 · 6 comments

Comments

@zawaruddin
Copy link
Contributor

in the line 5048 if (preg_match('/:/', $name)) {
variables $prefix and $namespace has no default value before it, so it showing error

Notice: Undefined variable: prefix in ..\vendor\econea\nusoap\src\nusoap.php on line 5095

Notice: Undefined variable: namespace in ..\vendor\econea\nusoap\src\nusoap.php on line 5142

Notice: Undefined variable: namespace in ..\vendor\econea\nusoap\src\nusoap.php on line 5143
@f3l1x
Copy link
Member

f3l1x commented Mar 14, 2018

Hi, thanks. Could you please send a PR?

@zawaruddin
Copy link
Contributor Author

OK

@zawaruddin
Copy link
Contributor Author

Well, You have made lot of changes in this repo....

@f3l1x
Copy link
Member

f3l1x commented Mar 15, 2018

@zawaruddin Yep, but in develop branch. There's also a master branch where you can fix it.

@f3l1x
Copy link
Member

f3l1x commented Mar 15, 2018

It's fixed in master. Could you please test it econea/nusoap: dev-master?

@f3l1x
Copy link
Member

f3l1x commented Apr 16, 2018

Could you @zawaruddin test it on dev-master? Thanks.

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

No branches or pull requests

2 participants