Moodle

Wrong icon can appear next to a file resource

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.1.1
  • Fix Version/s: 2.0.6, 2.1.3
  • Component/s: Course, Resource
  • Labels:
  • Testing Instructions:
    Hide
    1. Enable editing of a course
    2. Add a resource that is a file
    3. In the uploader, upload a PDF and Word document
    4. Click on the 'Save and return to course' button.
    5. Click on the file link and you should be shown a link to the same file type as indicated by the icon
    Show
    Enable editing of a course Add a resource that is a file In the uploader, upload a PDF and Word document Click on the 'Save and return to course' button. Click on the file link and you should be shown a link to the same file type as indicated by the icon
  • Workaround:
    Hide

    Upload files into a folder instead.
    Don't upload two files into one resource.

    Show
    Upload files into a folder instead. Don't upload two files into one resource.
  • Affected Branches:
    MOODLE_21_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE, MOODLE_21_STABLE
  • Pull from Repository:
  • Pull Master Branch:
    wip-MDL-29231-master
  • Pull Master Diff URL:

Description

When adding two files to one file resource in a course, the wrong icon can be displayed.

This is cosmetic only.

Replication instructions:

  1. Enable editing of a course
  2. Add a resource that is a file
  3. In the uploader, upload a PDF and Word document
  4. Click on the 'Save and return to course' button.

You could now have the PDF icon, but when you click on it you get the word document. Or inversely you could get the Word document icon, but instead download the PDF.

You can invert the icons and the documents by changing the main file.

Issue Links

has been marked as being related by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-11476 Wrong icons appear alongside certain file types (e.g. .fla) and URLs in the "Resources" area

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-29944 in navigation block.doc file is not link

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael de Raadt added a comment -

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.

Show
Michael de Raadt added a comment - 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.
Hide
Permalink
Michael de Raadt added a comment -

The problem appears in the navigation also.

Show
Michael de Raadt added a comment - The problem appears in the navigation also.
Hide
Permalink
Jason Fowler added a comment -

Diff to patch the error

Show
Jason Fowler added a comment - Diff to patch the error
Hide
Permalink
Jason Fowler added a comment -

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

Show
Jason Fowler added a comment - Fixed by ensuring the last file loaded is linked to by default, as that is the one the icon is derived from
Hide
Permalink
Michael de Raadt added a comment -

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...

mod/resource/lib.php, line 260
$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.

Show
Michael de Raadt added a comment - 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... mod/resource/lib.php, line 260 $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.
Hide
Permalink
Ankit Agarwal added a comment -

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

Show
Ankit Agarwal added a comment - 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
Hide
Permalink
Sam Hemelryk added a comment -

Thanks guys - this has been integrated now

Show
Sam Hemelryk added a comment - Thanks guys - this has been integrated now
Hide
Permalink
Rossiani Wijaya added a comment -

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

Show
Rossiani Wijaya added a comment - I noticed that docs file doesn't contain any link in navigation blocks. Is this on purpose?
Hide
Permalink
Rossiani Wijaya added a comment -

Patch is looking good.

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

Show
Rossiani Wijaya added a comment - Patch is looking good. I will wait for Sam to comment on this issue before passing this issue.
Hide
Permalink
Sam Hemelryk added a comment -

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

Cheers
Sam

Show
Sam Hemelryk added a comment - Hi Rosie Can you explain what you are seeing and how to replicate it? I'll give it a shot this afternoon. Cheers Sam
Hide
Permalink
Rossiani Wijaya added a comment -

Hi Sam,

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

I'm passing this issue.

Test passed.

Show
Rossiani Wijaya added a comment - Hi Sam, I create a new issue to discuss the link in navigation blocks ( MDL-29944 ). I'm passing this issue. Test passed.
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

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

Closing as fixed, ciao

Show
Eloy Lafuente (stronk7) added a comment - Done, your delicious hacks have been sent upstream, many thanks! Closing as fixed, ciao

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: