Skip to content

When upgrading database at command line, some PHP errors may be seen#5012

Merged
TheWitness merged 2 commits intoCacti:1.2.xfrom
ddb4github:php8
Nov 20, 2022
Merged

When upgrading database at command line, some PHP errors may be seen#5012
TheWitness merged 2 commits intoCacti:1.2.xfrom
ddb4github:php8

Conversation

@ddb4github
Copy link
Copy Markdown
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
Copy Markdown
Member

Looks good @ddb4github !

@TheWitness
Copy link
Copy Markdown
Member

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

@TheWitness
Copy link
Copy Markdown
Member

Note the one minor change for PHP5.4 @ddb4github

@TheWitness
Copy link
Copy Markdown
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.

2 participants