MDL-55166 admin: Fix display of misleading debug warning on manage block
authorDavid Mudrák <david@moodle.com>
Wed, 27 Jul 2016 20:49:49 +0000 (22:49 +0200)
committerDavid Mudrák <david@moodle.com>
Wed, 27 Jul 2016 21:15:48 +0000 (23:15 +0200)
commit0d7c4171756d05b4f3519b280e2beda5bd157f86
tree437aa80d61b211e75207276bc2cbbe6365c63b7f
parent90a8bdbfc0082651d0a60a4396947ce9cf5d43e5
MDL-55166 admin: Fix display of misleading debug warning on manage block

Blocks can declare has_config() as true without actually using the
default node in the admin tree. Typical use case is when the block
injects its settings to other parts of the admin tree and it assigns
null to its $setting node in its settings.php file.

As Janek L.B. correctly spotted, this led to false debugging message
on admin/blocks.php as the code interpreted it as missing settings.php
file.

The patch adds explicit file existence test for this rare case.
admin/blocks.php