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

filepicker.js has superfluous comma which prevents the filepicker loading in IE

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.3.3
    • Fix Version/s: 2.3.4
    • Component/s: Filepicker
    • Labels:
    • Testing Instructions:
      Hide

      In Moodle 2.3:

      1. Force IE into compatibility mode (IE7 mode worse case, but I found IE8 mode also caused problems)
      2. Load a page with the filepicker
      3. No errors should be seen and the filepicker should load)

      In Moodle 2.4 it shouldn't be possible for IE to go into compatibility mode and this change was actually done elsewhere, so shouldn't need testing.

      Show
      In Moodle 2.3: Force IE into compatibility mode (IE7 mode worse case, but I found IE8 mode also caused problems) Load a page with the filepicker No errors should be seen and the filepicker should load) In Moodle 2.4 it shouldn't be possible for IE to go into compatibility mode and this change was actually done elsewhere, so shouldn't need testing.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-36696-master

      Description

      The file picker will not load in Internet Explorer (tested in IE 8 and 9) due to a superfluous comma on line 1027 in filepicker.js

      1027: repo_id: this.active_repo.id,

      Need to be,

      1027: repo_id: this.active_repo.id

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    14/Jan/13