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

PW-1462: Parse refactor to support Any inner content in xsys messages #97

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ptorres-prowide
Copy link
Contributor

…-parse-xsys002

return Optional.ofNullable(parsedDocument);
}

private static String getDocumentXml(String xml, MxReadParams params) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Este metodo no se usa por lo que veo. Es el intento por lo de Any? En general hay que evitar usar DOM, es muy lento, por eso usamos Sax

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La llamada a este método está comentada. Habíamos quedado en que lo subía así y vos revisabas si no encontrabas una solución mejor. Este branch fue una 1era idea que no te convencía.

@zubri zubri changed the title CU-85ztm247h_PW-1462--PMH-Bottomline-GTX-extension-test-case-fails-to… PW-1462: Parse refactor to support Any inner content in xsys messages Aug 22, 2023
@ptorres-prowide ptorres-prowide force-pushed the CU-85ztm247h_PW-1462--PMH-Bottomline-GTX-extension-test-case-fails-to-parse-xsys002 branch from b90d8f8 to 6c7b659 Compare November 21, 2023 23:03
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

2 participants