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

New AJAX in weekly format has a few serious but probably easy-to-fix bugs

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.7, 1.8
    • Component/s: Course, JavaScript
    • Labels:
      None
    • Affected Branches:
      MOODLE_17_STABLE
    • Fixed Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE

      Description

      Hi, Ed

      Can you look at this urgently and help complete the AJAX implementation?

      Locations:

      course/rest.php - the old commands.php
      lib/ajax/ajaxcourse.js - the old JS from the topics format
      lib/ajax/section_classes.js - renamed back from php recently

      To try AJAX in the weekly course format:

      1) Set the course format to "weekly" in the settings.
      2) Turn on AJAX in the user profile preferences
      3) Admin > Server > Debugging .... set the level to DEBUG_DEVELOPER
      4) Go to the course and "Turn editing on".

      It nearly works but I think it's not detecting sections properly, because

      • moving activities (resources) will always end up in the top section, because beforeId is always 0.
      • moving blocks will send a bad block id and lose them

      Help!

        Attachments

        1. ajax_activities_fix.patch
          1 kB
          Dan Poltawski
        2. ajax objects fail in week zero.jpg
          257 kB
          Dennis Daniels
        3. ajax objects fail in week zero.jpg
          257 kB
          Dennis Daniels
        4. ajax objects fail in week zero.jpg
          257 kB
          Dennis Daniels

          Issue Links

            Activity

              People

              Assignee:
              cap2501 Edward Coyne
              Reporter:
              dougiamas Martin Dougiamas
              Tester:
              Vy-Shane Sin Fat
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                7/Nov/06