If the learner attempt is ended and the status of the sco is "passed", but the sco did not fix the cmi.exit value to "normal" (it's the responsibility of the sco), then the "suspend icon" override the "passed icon" and the toc doesn't reflect the real status of the scoes.

I think the "suspend icon" can only be added next to the status icon when the sco is incomplete.

I've attached a patch for this,
Valérian

thanks Valerian - I've just pushed this into HEAD - could you please QA before I push into 1.9Stable?

Hi Dan,

I've reviewed this, it's ok for me,
I've noticed one difference with my patch : i add the suspend icon next to the status icon, but maybe you want to keep just one icon ?

my version :
$statusicon .= '<img src="'.$OUTPUT->pix_url('suspend', 'scorm').'" alt="'.$strsuspended.'" title="'.$strsuspended.'" />';

your version :
$statusicon = '<img src="'.$OUTPUT->pix_url('suspend', 'scorm').'" alt="'.$strstatus.' - '.$strsuspended.'" title="'.$strstatus.' - '.$strsuspended.'" />';

Valérian

Hi Valerian,

finally got round to looking at this again - I think I prefer to stick with the single icon - but I could probably be convinced to use 2 if a few more people voted on it and wanted it!

otherwise - closing this off as fixed (now in 1.9stable and HEAD)

thanks heaps!