Summary
In lib/poller-common.php, weathermap_repair_maps() has a copy-paste error in the ICON relocation block:
if ($dirgood && file_exists($mydir . $objfile)) {
// ...
if (is_writable($mydir . $bgfile)) { // BUG: $bgfile is from the BACKGROUND block
$bgfile is the loop variable from the outer BACKGROUND processing section, not the ICON being relocated. If the last BACKGROUND file processed was not writable, every ICON relocation silently fails regardless of the actual ICON file's permissions.
Fix
Change $bgfile to $objfile in the ICON writability check.
Priority
Silent failure. Queue for Friday.
Summary
In
lib/poller-common.php,weathermap_repair_maps()has a copy-paste error in the ICON relocation block:$bgfileis the loop variable from the outer BACKGROUND processing section, not the ICON being relocated. If the last BACKGROUND file processed was not writable, every ICON relocation silently fails regardless of the actual ICON file's permissions.Fix
Change
$bgfileto$objfilein the ICON writability check.Priority
Silent failure. Queue for Friday.