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 upgrading database at command line, some PHP errors may be seen #5012

Merged
merged 2 commits into from Nov 20, 2022

Conversation

ddb4github
Copy link
Contributor

  • Fixed: get_rrdtool_version report str_replace error if rrdtool_version does not exist
  • Fixed: move global static $phperrors before other error-able function call
  • Fixed: file_exists report error if path_spine is not configured

…n does not exist

Fixed: move global static $phperrors before other error-able function call
Fixed: file_exists report error if path_spine is not configured
@TheWitness
Copy link
Member

Looks good @ddb4github !

@TheWitness
Copy link
Member

Can you amend a CHANGELOG for this. Just use the pull request #.

@TheWitness
Copy link
Member

Note the one minor change for PHP5.4 @ddb4github

@TheWitness
Copy link
Member

Just going to merge. I'll get the CHANGELOG updated

@TheWitness TheWitness merged commit 546d002 into Cacti:1.2.x Nov 20, 2022
TheWitness added a commit that referenced this pull request Nov 20, 2022
@ddb4github ddb4github deleted the php8 branch November 20, 2022 22:40
@netniV netniV changed the title Fixed three error for PHP 8 when execute cli/upgrade_database.php When upgrading database at command line, some PHP errors may be seen Dec 31, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants