Before I raise an MDL.... In M3.1, 3.2 and 3.3 '/blocks/navigation/block_navigation.php', method 'get_content' there is the code:
$arguments = array(
'id' => $this->instance->id,
'instance' => $this->instance->id,
'candock' => $this->instance_can_be_docked(),
'courselimit' => $limit,
'expansionlimit' => $expansionlimit
Which appears unused. Can anybody confirm this please? I know its not much but would save a bit of CPU time if it was removed.
Note: This also means that the value of '$CFG->navcourselimit' is not used / taken into account.