Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.2.3
    • Fix Version/s: 2.6
    • Component/s: Course
    • Labels:
    • Testing Instructions:
      Hide

      Test 1

      1. Make sure move_section() is not used in moodle core and is present in deprecatedlib.php
        • grep -Ri "move_section" .

      Test 2

      1. Log in as admin/teacher
      2. Enable ajax and go to course
      3. Turn editing on
      4. Drag-n-Drop section below another section
      5. Refresh page and make sure section retains it's position.
      6. Drag-n-drop section to top
      7. Turn editing off and refresh page
      8. Make sure section retains it's position

      Test 3

      1. Repeat Test 2 with AJAX disabled and clicking up/down icons.
      Show
      Test 1 Make sure move_section() is not used in moodle core and is present in deprecatedlib.php grep -Ri "move_section" . Test 2 Log in as admin/teacher Enable ajax and go to course Turn editing on Drag-n-Drop section below another section Refresh page and make sure section retains it's position. Drag-n-drop section to top Turn editing off and refresh page Make sure section retains it's position Test 3 Repeat Test 2 with AJAX disabled and clicking up/down icons.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull Master Branch:
      wip-mdl-33379

      Description

      The move_section function was deprecated and now needs to be removed.

      move_section and move_section_to are similar functions. It might be nice to remove move_section from course/lib.php

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              rajeshtaneja Rajesh Taneja
              Reporter:
              rajeshtaneja Rajesh Taneja
              Peer reviewer:
              Ankit Agarwal Ankit Agarwal
              Integrator:
              Sam Hemelryk Sam Hemelryk
              Tester:
              Michael de Raadt Michael de Raadt
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13