Please use $Sender instead of &$Sender in event handlers.

When declaring an event handler there are a lot of instances where the $Sender parameter is declared as a reference. This is a bad practice and will break plugins in future versions of Vanilla.

So for example, do this:

public function Base_GetAppSettingsMenuItems_Handler($Sender) {
...
}

instead of this:

public function Base_GetAppSettingsMenuItems_Handler(&$Sender) {
...
}

This has been a Vanilla service announcement.

ddumont

Interesting... Will do.