Changes to support PHP7 and Cacti 1.x #9
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Changing script filename to ss_cisco_catalyst_sfp.php instead of ss_65xx_sfp.php as it is working for all IOS Catalyst switches
Change filename ss_cisco_65xx_sfp.php from to ss_cisco_catalyst_sfp.php Updated ereg_replace functions to preg_replace functions as it has been deprecated since PHP 5.3 and actually REMOVED since PHP 7.0 This script is working in a cacti 1.x environment
Created new template file cacti_data_query_cisco_catalyst_sfp_statistics.xml Changed query variable to get query_interface instead of broken query_device
Updated to fit the new changes (support of cacti 1.x )
Changing script filename
Create new template file
Create ss_cisco_catalyst_sfp.php
Update README.md
Delete ss_65xx_sfp.php
Delete cacti_data_query_cisco_65xx_sfp_statistics.xml
Delete cacti_graph_template_cisco_-_sfp_statistics_dbm.xml
This reverts commit 11ae623.
Update CHANGELOG
netniV
approved these changes
Dec 12, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Various changes to fit recent changes within Cacti and PHP 7
Tested in a production environment using cacti 1.2.21 (last release) and PHP 7.3.31-1