If "when to show scores to students" is set to "never", the student's attempts listings page should not show the "review allowed from" date

[christianp]

There's a missing case in the template for the attempts listing table that the student sees.

If the resource is set to never show scores to students, then they can't re-enter completed attempts in review mode. At the moment, if the "allow students to review attempts from" date is set, then this date is shown in the table, even if it's passed, which is confusing.

I think the template should check attempt.should_show_scores, and only show the date if it's True.