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

Fix #36 - 'Attribute redefined' problem #50

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ciforusalterra
Copy link

I update this libs to fix Attribute Redefined problem on "xmlns:SOAP-ENV". We use different style configuration on our editor, so sorry for a lot of line arrangement. Thanks.

@moszeed
Copy link
Owner

moszeed commented Aug 14, 2019

Can you please fix those style quirks, there is a .eslint file you can use for this!

@ciforusalterra
Copy link
Author

Can you please fix those style quirks, there is a .eslint file you can use for this!

I've already update it bro. Please check.

@moszeed
Copy link
Owner

moszeed commented Aug 14, 2019

thanks..

hmm, but there seems something wrong, the tests are failing:
https://travis-ci.org/moszeed/easysoap/jobs/571747519

can you please check why !?

@ciforusalterra
Copy link
Author

thanks..

hmm, but there seems something wrong, the tests are failing:
https://travis-ci.org/moszeed/easysoap/jobs/571747519

can you please check why !?

It failed since I run test on your original source code. I don't have any idea for now.

@ciforusalterra
Copy link
Author

ciforusalterra commented Aug 14, 2019

thanks..

hmm, but there seems something wrong, the tests are failing:
https://travis-ci.org/moszeed/easysoap/jobs/571747519

can you please check why !?

I've fix some of your test just to make it pass. I don't know if that was the correct behaviour.

@ciforusalterra
Copy link
Author

Hi @moszeed, could you help me to merge this PR if there is no more issue? Our team need to use this libs as soon as possible. Thanks.

@3mp3ri0r
Copy link

Hi @moszeed. Are you still maintain this libs? How can we help you to maintain this repo?

@moszeed
Copy link
Owner

moszeed commented Aug 21, 2019

Hi, yes i still maintain this repo..
but in my spare time, so if you want to help, send small pull request, this one here has so many small changes (mostly style) that i have to check every thing...

so keep the styling as is ... 🙏

@3mp3ri0r
Copy link

We're already use your eslint config and that was the result. Any idea?

@ikonduktor
Copy link

ikonduktor commented May 23, 2022

You have error in your code. In file request.js

const envelope = await getRequestEnvelopeParams(combinedParams, opts); - returns xmlns:SOAP-ENV="${envelope.soap_env}"

and then you add this string one more time to

const $soapEnvelope = `<SOAP-ENV:Envelope
            xmlns:SOAP-ENV="${envelope.soap_env}"
            ${$namespacesAsString}>
            ${$head}
            ${$body}
        </SOAP-ENV:Envelope>`;

After that there is an error "Attribute redefined"

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

Successfully merging this pull request may close these issues.

None yet

4 participants