add type checking to generic_stats.inc.php for $vars['to'] and $vars['from']

[VVelox]

This resolves the issues with generic_stats.inc.php passing it from=-24hour/to=now instead of integers.

DO NOT DELETE THE UNDERLYING TEXT

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

• Have you followed our code guidelines?
• If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.
• If my Pull Request makes discovery/polling/yaml changes, I have added/updated test data.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

[murrant]

@VVelox this works, but GraphParameters should contain the actual timediff.

$timediff = $graph_params->to - $graph_params->from;

Try to use $graph_params over $vars when possible :)

[VVelox]

@VVelox this works, but GraphParameters should contain the actual timediff.

$timediff = $graph_params->to - $graph_params->from;

Try to use $graph_params over $vars when possible :)

Nice! Did not know about those. Thanks! <3

[librenms-bot]

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/23-4-0-changelog/21194/1