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

A hard coded string "Loading" in file manager element

    Details

    • Testing Instructions:
      Hide

      [Its not possible to really discover this improvement without problems with javascript allowing the existing to be displayed, so mostly we should test for regressions when loading the repository repository browser and ensure that the repository picker loads without problems

      Show
      [Its not possible to really discover this improvement without problems with javascript allowing the existing to be displayed, so mostly we should test for regressions when loading the repository repository browser and ensure that the repository picker loads without problems
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      MDL-29523-master

      Description

      This is not an obvious problem. I found it just because occasionally javascript cache problem.

      Details in the diff link

        Gliffy Diagrams

          Activity

          Hide
          salvetore Michael de Raadt added a comment -

          Just to add some detail: the term Loading is hard-coded in the repository code rather than being gathered from a language file.

          Show
          salvetore Michael de Raadt added a comment - Just to add some detail: the term Loading is hard-coded in the repository code rather than being gathered from a language file.
          Hide
          salvetore Michael de Raadt added a comment -

          Thanks for providing a solution Sunner.

          Show
          salvetore Michael de Raadt added a comment - Thanks for providing a solution Sunner.
          Hide
          poltawski Dan Poltawski added a comment -

          I have reviewed this and it looks good, so submitting for integration. Thanks Sunner!

          Show
          poltawski Dan Poltawski added a comment - I have reviewed this and it looks good, so submitting for integration. Thanks Sunner!
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks! (21, 22 & master)

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks! (21, 22 & master)
          Hide
          phalacee Jason Fowler added a comment -

          File repos doesn't seem to have any issues

          Show
          phalacee Jason Fowler added a comment - File repos doesn't seem to have any issues
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Well,

          I wish I said it every time
          you do the things you do.
          You always lend a helping hand,
          and I'm filled with gratitude.

          You are strong and generous
          for each and everyone one of us.
          I am eternally grateful,
          I cannot say thanks enough.

          Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Well, I wish I said it every time you do the things you do. You always lend a helping hand, and I'm filled with gratitude. You are strong and generous for each and everyone one of us. I am eternally grateful, I cannot say thanks enough. Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Mar/12