Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-30460

In wiki editor for Creole and NWiki pages, image dropdown does not include files with upper case suffixes.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.6, 2.1.3, 2.2
    • Fix Version/s: 2.0.7, 2.1.4
    • Component/s: Wiki (2.x)
    • Labels:

      Description

      Found this while testing MDLQA-1395 (A student can add images to a wiki). I uploaded an image file in the wiki Files tab. Then I went to edit a Creole-formatted wiki page, and the image file would not appear in the drop down for inserting an image. After some investigation, I found that the cause was that the file extension of the image file was in upper case. I then tried an image file with a lower-case file extension and that worked.

      This looks very similar to MDL-30165 except that it affects a different area. The problem appears to be in mod/wiki/edit_form.php with this condition in the definition method:

      if (preg_match('#'.$ext.'$#', $filename)) {

      The fix for MDL-30165 fixed a similar issue in repository/lib.php by change the preg_match pattern to '#'.$ext.'$#i' (Note the addition of 'i'.)

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for spotting this and providing a solution.

            Show
            salvetore Michael de Raadt added a comment - Thanks for spotting this and providing a solution.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Simple fix, sending through integration
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - Simple fix, sending through integration Thanks
            Hide
            nebgor Aparup Banerjee added a comment -

            Thanks, this has been integrated and tested for master and stables too.

            Show
            nebgor Aparup Banerjee added a comment - Thanks, this has been integrated and tested for master and stables too.
            Hide
            nebgor Aparup Banerjee added a comment -

            passing - simple and works fine.

            Show
            nebgor Aparup Banerjee added a comment - passing - simple and works fine.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days.

            Thanks for the hard work! Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days. Thanks for the hard work! Closing, ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Jan/12