Files
cacti/cli/repair_templates.php
Thomas VincentandTheWitness fcbcbdd1d7 refactor: replace strpos() boolean checks with str_contains/str_starts_with (#6645)
* refactor: replace strpos() boolean checks with str_contains/str_starts_with

Completes the PHP 8.x string function migration started in lib/.
Converts remaining legacy patterns across 81 files:

- strpos($x, $y) !== false → str_contains($x, $y)
- strpos($x, $y) === false → !str_contains($x, $y)
- strpos($x, $y) != false  → str_contains($x, $y)
- bare if (strpos($x, $y)) → if (str_contains($x, $y))
- substr($x, 0, strlen($y)) == $y → str_starts_with($x, $y)

Intentionally preserves strpos() calls that use the return value as
an integer offset:
- user_admin.php / user_group_admin.php (4 calls, ternary for position)
- auth_profile.php (2 calls, offset for substr); guards fixed from
  bare if ($pos) to if ($pos !== false)

Behavior fix: ss_esxi_vhosts.php:55 bare strpos() missed position-0
matches; str_contains() now correctly detects 'not installed' at any
position. Adjacent line already used str_contains().

Ref: https://github.com/Cacti/cacti/issues/6641

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>

* fix: correct dead-code elseif in plugin_display_compat() version comparison

The if/elseif chain in plugin_display_compat() checked >=, >=, >, <
— the second >= was unreachable dead code. Corrected to <= to complete
the operator set (>=, <=, >, <).

Without this fix, <= constraints (e.g. "<=1.2") fell through to the <
branch and were rendered as "< =1.2" instead of "<= 1.2".

This pre-existing bug was exposed when converting strpos() to
str_contains(), which made PHPStan detect the always-false condition.

Ref: https://github.com/Cacti/cacti/issues/6641

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>

* fix: add limit to explode('=') in CLI argument parsers

17 CLI scripts used explode('=', $parameter) without a limit of 2,
silently truncating values containing = (e.g. --note=a=b parsed as
$value='a' instead of 'a=b'). The other 40 CLI scripts already had
the correct limit.

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>

---------

Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
Co-authored-by: TheWitness <thewitness@cacti.net>
2026-02-25 11:38:59 -05:00

176 lines
5.7 KiB
PHP
Executable File

#!/usr/bin/env php
<?php
/*
+-------------------------------------------------------------------------+
| Copyright (C) 2004-2026 The Cacti Group |
| |
| This program is free software; you can redistribute it and/or |
| modify it under the terms of the GNU General Public License |
| as published by the Free Software Foundation; either version 2 |
| of the License, or (at your option) any later version. |
| |
| This program is distributed in the hope that it will be useful, |
| but WITHOUT ANY WARRANTY; without even the implied warranty of |
| MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| GNU General Public License for more details. |
+-------------------------------------------------------------------------+
| Cacti: The Complete RRDtool-based Graphing Solution |
+-------------------------------------------------------------------------+
| This code is designed, written, and maintained by the Cacti Group. See |
| about.php and/or the AUTHORS file for specific developer information. |
+-------------------------------------------------------------------------+
| http://www.cacti.net/ |
+-------------------------------------------------------------------------+
*/
require(__DIR__ . '/../include/cli_check.php');
require_once(CACTI_PATH_LIBRARY . '/poller.php');
require_once(CACTI_PATH_LIBRARY . '/utility.php');
require_once(CACTI_PATH_LIBRARY . '/template.php');
// switch to main database for cli's
if (POLLER_ID > 1) {
db_switch_remote_to_main();
}
// process calling arguments
$parms = $_SERVER['argv'];
array_shift($parms);
$execute = false;
if (cacti_sizeof($parms)) {
foreach ($parms as $parameter) {
if (str_contains($parameter, '=')) {
[$arg, $value] = explode('=', $parameter, 2);
} else {
$arg = $parameter;
$value = '';
}
switch ($arg) {
case '--execute':
$execute = true;
break;
case '--version':
case '-V':
case '-v':
display_version();
exit(0);
case '--help':
case '-H':
case '-h':
display_help();
exit(0);
default:
print 'ERROR: Invalid Parameter ' . $parameter . PHP_EOL . PHP_EOL;
display_help();
exit(1);
}
}
}
if ($execute) {
print 'NOTE: Repairing All Duplicated Templates' . PHP_EOL;
} else {
print 'NOTE: Performing Check of Templates' . PHP_EOL;
}
// repair data templates first
if ($execute) {
print 'NOTE: Repairing Data Templates' . PHP_EOL;
} else {
print 'NOTE: Performing Check of Data Templates' . PHP_EOL;
}
$damaged_template_ids = db_fetch_assoc("SELECT DISTINCT data_template_id FROM data_template_rrd WHERE hash='' AND local_data_id=0");
if (cacti_sizeof($damaged_template_ids)) {
foreach ($damaged_template_ids as $id) {
$template_name = db_fetch_cell('SELECT name FROM data_template WHERE id=' . $id['data_template_id']);
print "NOTE: Data Template '$template_name' is Damaged and can be repaired" . PHP_EOL;
}
$damaged_templates = db_fetch_assoc("SELECT * FROM data_template_rrd WHERE hash='' AND local_data_id=0");
if (cacti_sizeof($damaged_templates)) {
print "NOTE: -- Damaged Data Templates Objects Found is '" . cacti_sizeof($damaged_templates) . "'" . PHP_EOL;
if ($execute) {
foreach ($damaged_templates as $template) {
$hash = get_hash_data_template($template['local_data_template_rrd_id'], 'data_template_item');
db_execute("UPDATE data_template_rrd SET hash='$hash' WHERE id=" . $template['id']);
}
}
}
} else {
print 'NOTE: No Damaged Data Templates Found' . PHP_EOL;
}
// reset the array
$damaged_templates = [];
// repair graph templates
if ($execute) {
print 'NOTE: Repairing Graph Templates' . PHP_EOL;
} else {
print 'NOTE: Performing Check of Graph Templates' . PHP_EOL;
}
$damaged_template_ids = db_fetch_assoc("SELECT DISTINCT graph_template_id FROM graph_template_input WHERE hash=''");
if (cacti_sizeof($damaged_template_ids)) {
foreach ($damaged_template_ids as $id) {
$template_name = db_fetch_cell('SELECT name FROM graph_templates WHERE id=' . $id['graph_template_id']);
print "NOTE: Graph Template '$template_name' is Damaged and can be repaired" . PHP_EOL;
}
$damaged_templates = db_fetch_assoc("SELECT * FROM graph_template_input WHERE hash=''");
if (cacti_sizeof($damaged_templates)) {
print "NOTE: -- Damaged Graph Templates Objects Found is '" . cacti_sizeof($damaged_templates) . "'" . PHP_EOL;
if ($execute) {
foreach ($damaged_templates as $template) {
$hash = get_hash_graph_template(0, 'graph_template_input');
db_execute("UPDATE graph_template_input SET hash='$hash' WHERE id=" . $template['id']);
}
}
}
} else {
print 'NOTE: No Damaged Graph Templates Found' . PHP_EOL;
}
/**
* display_version - displays version information
*
* @return void
*/
function display_version() : void {
$version = get_cacti_cli_version();
print "Cacti Database Template Repair Utility, Version $version, " . COPYRIGHT_YEARS . PHP_EOL;
}
/**
* display_help - displays the usage of the function
*
* @return void
*/
function display_help() : void {
display_version();
print PHP_EOL;
print 'usage: repair_templates.php [--execute]' . PHP_EOL . PHP_EOL;
print 'A utility designed to repair any damaged Cacti Graph and Data Templates. That lacked a hash.' . PHP_EOL;
print 'This utility should not need to be used in any modern Cacti install.' . PHP_EOL . PHP_EOL;
print 'Optional:' . PHP_EOL;
print ' --execute - Perform the repair. Otherwise check for errors.' . PHP_EOL . PHP_EOL;
}