shouldn't this be:
`$goal = empty( $goal ) ? $this->get_goal() : $goal;`

(if $toal is not empty then keep it?)