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

Course management page hits a Firefox JS max recursion error.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.5.1
    • Fix Version/s: None
    • Component/s: JavaScript
    • Labels:
    • Affected Branches:
      MOODLE_25_STABLE

      Description

      The course management page when viewed in Firefox (23.0) breaks due to a max recursion error that I tracked down to coming from the JS formautosumbit module.

      too much recursion
      http://localhost/integration/lib/yuilib/3.9.1/build/simpleyui/simpleyui.js
      Line 11497
      

      I've experienced this on a site with 200 course categories, which are all displayed at once on the course/manage.php
      Thats 200 autosubmit select boxes associated with moving categories, each with 200 options.
      I find that the select boxes at the top work as expected, however the ones at the bottom submit as soon as you click on them and when stepping through the JS I find that the required data attributes are not set.

      I don't think this issue should focus on fixing the management page, Marina's tweaked it nicely with MDL-41127 and with MDL-31830 I'll be replacing it completely.
      I think this issue should improve the autosubmit module so that it doesn't need to recurse all select boxes during initialisation.
      It'd also be ideal if all autosubmits were handled with a single delegation.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              samhemelryk Sam Hemelryk
              Participants:
              Component watchers:
              Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated: