Refactor: scheduler: best practices when checking action definition

Rename check_action_definition() to pcmk__check_action_config() to meet current
guidelines, add a doxygen block, drop a redundant argument, use bool instead of
gboolean, and improve formatting, comments, sanity checks, variable names, and
log messages.