librenms / librenms

includes/polling/availability.inc.php

<?php

use LibreNMS\Config;

This use statement conflicts with another class in this namespace, Config. Consider defining an alias.

use LibreNMS\RRD\RrdDefinition;

$os->enableGraph('availability');

$col = dbFetchColumn('SELECT duration FROM availability WHERE device_id = ?', [$device['device_id']]);
foreach (Config::get('graphing.availability') as $duration) {
    if (! in_array($duration, $col)) {
        $data = ['device_id' => $device['device_id'],
            'duration' => $duration, ];
        dbInsert($data, 'availability');
    }
}

echo 'Availability: ' . PHP_EOL;

foreach (dbFetchRows('SELECT * FROM availability WHERE device_id = ?', [$device['device_id']]) as $row) {
    //delete not more interested availabilities
    if (! in_array($row['duration'], Config::get('graphing.availability'))) {

It seems like LibreNMS\Config::get('graphing.availability') can also be of type null; however, parameter $haystack of in_array() does only seem to accept array, maybe add an additional type check?

        dbDelete('availability', 'availability_id=?', [$row['availability_id']]);
        continue;
    }

    $avail = \LibreNMS\Device\Availability::availability($device, $row['duration']);
    $human_time = \LibreNMS\Util\Time::humanTime($row['duration']);

    $rrd_name = ['availability', $row['duration']];
    $rrd_def = RrdDefinition::make()
        ->addDataset('availability', 'GAUGE', 0);

    $fields = [
        'availability' => $avail,
    ];

    $tags = ['name' => $row['duration'], 'rrd_def' => $rrd_def, 'rrd_name' => $rrd_name];
    data_update($device, 'availability', $tags, $fields);

    dbUpdate(['availability_perc' => $avail], 'availability', '`availability_id` = ?', [$row['availability_id']]);

    echo $human_time . ' : ' . $avail . '%' . PHP_EOL;
}
unset($duration);