Moodle
  1. Moodle
  2. MDL-29966

404 for i/ajaxloader image - wrong variable order in function call

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.5, 2.1.2, 2.2
    • Fix Version/s: 2.0.6, 2.1.3
    • Component/s: Groups
    • Labels:
      None
    • Testing Instructions:
      Hide

      1/ go to groups page
      2/ click on group in left column
      3/ there should be a spinning thing above the right column with users (you may have to slow somehow your browser, I hacked in one js alert to make it pause a bit when testing it)

      Show
      1/ go to groups page 2/ click on group in left column 3/ there should be a spinning thing above the right column with users (you may have to slow somehow your browser, I hacked in one js alert to make it pause a bit when testing it)
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w43_MDL-29966_m22_grouploading

      Description

      In /group/clientlib.js, line 186
      replace
      loadingImg.setAttribute("src", M.util.image_url('moodle', '/i/ajaxloader'));
      with
      loadingImg.setAttribute("src", M.util.image_url('/i/ajaxloader', 'moodle'));

      Variables are supplied to the function in wrong order, the result is that the image isn't found (404 in apache log)

        Gliffy Diagrams

          Activity

          Hide
          Petr Skoda added a comment -

          thanks for the report and patch!

          Show
          Petr Skoda added a comment - thanks for the report and patch!
          Hide
          Aparup Banerjee added a comment -

          awesome, thats been integrated.

          Show
          Aparup Banerjee added a comment - awesome, thats been integrated.
          Hide
          Ankit Agarwal added a comment -

          I can see the spinning sun!
          Test passed!
          Thanks

          Show
          Ankit Agarwal added a comment - I can see the spinning sun! Test passed! Thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And this has been sent upstream (already available @ git and cvs repos). Many, many thanks!

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - And this has been sent upstream (already available @ git and cvs repos). Many, many thanks! Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: