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

Drag and drop a section to general section causes JS error

    XMLWordPrintable

Details

    • MOODLE_23_STABLE, MOODLE_25_STABLE
    • MOODLE_24_STABLE, MOODLE_25_STABLE
    • MDL-36465-master
    • Hide
      • Open a course with multiple sections
      • Turn editing on
      • Attempt to drag section 1 to section 2
        • Confirm that all section titles were updated
        • Confirm that the section was moved by JS
      • Reload the page
        • Confirm that the section was still moved
      • Attempt to drag section 2 to section 1
        • Confirm that all section titles were updated
        • Confirm that the section was moved by JS
      • Reload the page
        • Confirm that the section was still moved
      • Attempt to drag a section to the last position
        • Confirm that all section titles were updated
        • Confirm that the section was moved by JS
      • Reload the page
        • Confirm that the section was still moved
      • Attempt to drag section 1 to section 0 (General)
        • Confirm that it was not possible to drop on 0
        • Confirm that dropping anyway didn't do anything
      Show
      Open a course with multiple sections Turn editing on Attempt to drag section 1 to section 2 Confirm that all section titles were updated Confirm that the section was moved by JS Reload the page Confirm that the section was still moved Attempt to drag section 2 to section 1 Confirm that all section titles were updated Confirm that the section was moved by JS Reload the page Confirm that the section was still moved Attempt to drag a section to the last position Confirm that all section titles were updated Confirm that the section was moved by JS Reload the page Confirm that the section was still moved Attempt to drag section 1 to section 0 (General) Confirm that it was not possible to drop on 0 Confirm that dropping anyway didn't do anything

    Description

      When trying to drag and drop a section, in place of the general section, a javascript error occurs and the lightbox containing the loading icon is never hidden.

      Replication steps:

      1. Log in as admin/password
      2. Navigate to a course with existing activities
      3. Turn editing on
      4. Wait for page to reload
      5. Drag a lower section over section 0 (the top section) and release it

      Expected result: Either the section should move to become section 0 or the move should not be possible.

      Actual result: The section is allowed to be dropped on section 0, but the move does not complete. The loading icon appears and the section is blurred. On reloading, the moved section is restored to its original location. The following JS error is given.

      TypeError: node is null
      http://michael.moodle.local/moodle_master_test_mysql/theme/yui_combo.php?moodle/1352347236/course/dragdrop/dragdrop.js
      Line 105
      

      Line 105 is...

      return Number(node.get('id').replace(/section-/i, ''));
      

      Attachments

        Issue Links

          Activity

            People

              dobedobedoh Andrew Lyons
              fred Frédéric Massart
              Marina Glancy Marina Glancy
              Sam Hemelryk Sam Hemelryk
              Michael de Raadt Michael de Raadt
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Sara Arjona (@sarjona), Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              7 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Nov/13