don't let scorm_grade_user_attempt return a non-number

too many other bits of code expect it to return a number, so abandon the idea of letting it return "incomplete" for now.

Going to get just scorm_get_attempt_status to check completion state, since that's what students see.