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 node generation when a cdata property is null #701

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

Kelgors
Copy link
Contributor

@Kelgors Kelgors commented Dec 18, 2024

Purpose / Goal

During XML building, CDATA with null value is not handled properly. When a value is null, the cdata property is ignored and treated as a normal node.

This PR aims to resolve this issue (#700) by handling the cdata property when it has a null value.

Benchmark

Before Changes

Running Suite: XML Builder benchmark
fxp : 73911.77177576702 requests/second
fxp - preserve order : 96821055.83555794 requests/second
xml2js  : 23993.824078178754 requests/second

After Changes

Running Suite: XML Builder benchmark
fxp : 71796.8171188612 requests/second
fxp - preserve order : 95102276.51741566 requests/second
xml2js  : 22884.60165181542 requests/second

Type

Please mention the type of PR

  • Bug Fix
  • Refactoring / Technology upgrade
  • New Feature

Note : Please ensure that you've read contribution guidelines before raising this PR. If your PR is in progress, please prepend [WIP] in PR title. Your PR will be reviewed when [WIP] will be removed from the PR title.

Bookmark this repository for further updates.

@coveralls
Copy link

Coverage Status

coverage: 98.217% (+0.005%) from 98.212%
when pulling 60fe977 on Kelgors:fix/empty-cdata
into e132656 on NaturalIntelligence:master.

@amitguptagwl amitguptagwl merged commit 280cd63 into NaturalIntelligence:master Dec 20, 2024
9 checks passed
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.

3 participants