Checks in the tool_task incorrectly set id and action_link in a pointless way

This is just a bug due a change in the api for checks and these couple checks are using the wrong way which doesn't actually work.

The correct way to set check metadata is viab methods:

get_id(), get_name(), get_action_link()

The incorrect way to do this was via properties in a version of the api which never landed:

$this->id = 'foo', $this->name = 'blah'.

Split from MDL-77336