search

librenms-plugins / Weathermap

MIT License
64 stars 51 forks source link

Add #!/usr/bin/php to map-poller.php #92

Closed pepperoni-pi closed 1 year ago

pepperoni-pi commented 1 year ago

Adding this line seems to be the fix for getting the editor to start working again after my upgrade to PHP8.1. Without this line, edits never get applied and don't display in the WebUI.

pepperoni-pi commented 1 year ago

Explicitly calling php in the cron file also addresses the issue.

Before: /5 * librenms /opt/librenms/html/plugins/Weathermap/map-poller.php >> /dev/null 2>&1

After: /5 * librenms php /opt/librenms/html/plugins/Weathermap/map-poller.php >> /dev/null 2>&1

BarbarossaTM commented 1 year ago

This has already been included as part of https://github.com/librenms-plugins/Weathermap/commit/4ce6ad1aa1b6fadda25b42042a9603212a27c386