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

Parsing of Empty Tags #631

Open
khkiley opened this issue Jan 2, 2024 · 4 comments
Open

Parsing of Empty Tags #631

khkiley opened this issue Jan 2, 2024 · 4 comments

Comments

@khkiley
Copy link

khkiley commented Jan 2, 2024

  • [Y] Are you running the latest version?
  • [Y] Have you included sample input, output, error, and expected output?
  • [Y] Have you checked if you are using correct configuration?
  • [N ] Did you try online tool?

Description

I am looking for a way to provide a default value for empty tags, to override the default of empty string. Similar to issue #630

ETA: #619 may also be related.

Empty tags are not processed through the tagValueProcessor (which makes sense as there is no value to process) but then the value is always an empty string, and there is no way I am aware of, to transpose the value.

The use case is for a data set that sends flags down as an empty tag. ie.

<thisIsAFlag /> 

// would be parsed as 
{ thisIsAFlag: true }

// instead of 
{ thisIsAFlag:'' }

Bookmark this repository for further updates. Visit SoloThought to know about recent features.

Copy link

github-actions bot commented Jan 2, 2024

We're glad you find this project helpful. We'll try to address this issue ASAP. You can vist https://solothought.com to know recent features. Don't forget to star this repo.

@amitguptagwl
Copy link
Member

There are multiple possibilities. Hence, I've come up with a common solution in v5. Development of parser was completed few months back. But I'm not getting time to finish UTs and complete the changes in Builder. So please wait for the next release

@khkiley
Copy link
Author

khkiley commented Jan 4, 2024

Thank you, Amit, for your quick reply and all your hard work supporting this package.

I will keep an eye out for V5

Kurt

@amitguptagwl
Copy link
Member

I had updated the documents for v5. Please check it with latest version.

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

No branches or pull requests

2 participants