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

When passed a null value, number_format_i18n() can return a invalid number #5136

Merged
merged 2 commits into from Jan 6, 2023

Conversation

ddb4github
Copy link
Contributor

Fixed: functiom number_format_i18n should return a valid number '0' for NULL input

@TheWitness
Copy link
Member

Can you add a CHANGELOG entry?

@netniV
Copy link
Member

netniV commented Jan 5, 2023

We will also need to apply this to develop

@netniV netniV merged commit 1301ac4 into Cacti:1.2.x Jan 6, 2023
@netniV
Copy link
Member

netniV commented Jan 6, 2023

This commit should now be cherry picked to develop or committed as a new change there if it doesn't already apply.

@ddb4github ddb4github deleted the fix_number_format_i18n branch January 6, 2023 12:48
@netniV netniV changed the title Functiom number_format_i18n return a invalid number for NULL input When passed a null value, the number_format_i18n can return a invalid number Feb 15, 2023
@netniV netniV changed the title When passed a null value, the number_format_i18n can return a invalid number When passed a null value, number_format_i18n() can return a invalid number Feb 15, 2023
@netniV netniV added bug Undesired behaviour resolved A fixed issue confirmed Bug is confirm by dev team labels Feb 15, 2023
@netniV netniV added this to the v1.2.24 milestone Feb 15, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants