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

Introduce an alternate map based parsed form #78

Closed
wants to merge 2 commits into from

Conversation

vanjabucic
Copy link

While SimpleForm can be useful, I find that a map based parser can be easier to use for naive/simplistic workloads.
Almost all xml parsers out there have a to_map operator (with standard caveats).
Also, it can be used as another example on how to build a custom event handler.

@qcam
Copy link
Owner

qcam commented Oct 17, 2020

Hi @vanjabucic, thanks for such a wonderful PR (I really like how the decisions well documented and explained). While I'm pretty excited with the idea, I recently learned of https://github.com/bennyhat/xml_json which implements some conventional approaches to convert XML to JSON/map. Would that cover the use case in your mind?

@vanjabucic
Copy link
Author

Thanks for pointing out the other project that deals with different map representations, I was not aware of it.
It does provide nice examples on how to approach XML parsing into custom payloads.
Maybe it deserves a mention in the docs somewhere. Newcomers to the language (especially the FP aspect) would find SAX processing a daunting task. I know I always appreciated sample implementations.
Thanks again for your work on Saxy.

@qcam
Copy link
Owner

qcam commented Oct 17, 2020

👍 Definitely I should mention xml_json in README. Also would you mind if I turn your work in the PR to a guide in https://github.com/qcam/saxy/tree/master/guides?

@vanjabucic
Copy link
Author

Please do use it in any way you see fit, it was meant as an example/guide anyhow.

@qcam
Copy link
Owner

qcam commented Jun 5, 2021

Mentioned at 5b427b9

@qcam qcam closed this Jun 5, 2021
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.

2 participants