Thanks for reporting this. I can confirm the problem.

I've put it on our backlog and we'll try to get to it as soon as we can.

In the meantime feel free to help us work on this issue. If you do add a solution, please add a 'patch' label.

The problem appears in the navigation also.

Diff to patch the error

Fixed by ensuring the last file loaded is linked to by default, as that is the one the icon is derived from

I thought I should follow up on this to avoid confusion in future.

There doesn't appear to be a problem with view.php. It is delivering the correct file.

The choice of mainfile in mod/resource/lib.php seems to be choosing the incorrect file and so the icons are appearing incorrectly. To fix this, the choice of mainfile should be the same as the choice in view.php. This can be achieved by changing...

$mainfile = array_pop($files);

...to...

$mainfile = reset($files);

...which will return a reference to the first file object rather than the object itself (I believe).

This fix should be checked.

Hi Jason,
Looks good!

Unrelated Question to discuss :why do we allow multiple files to be uploaded in such a situation, if there is no way to access those files?
Thanks

Thanks guys - this has been integrated now

I noticed that docs file doesn't contain any link in navigation blocks. Is this on purpose?

Patch is looking good.

I will wait for Sam to comment on this issue before passing this issue.

Hi Rosie
Can you explain what you are seeing and how to replicate it?
I'll give it a shot this afternoon.

Cheers
Sam

Hi Sam,

I create a new issue to discuss the link in navigation blocks (MDL-29944).

I'm passing this issue.

Test passed.

Done, your delicious hacks have been sent upstream, many thanks!

Closing as fixed, ciao