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

URL downloader repository appears in non-image environment

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.2.4, 2.3.1
    • Component/s: Repositories
    • Labels:
      None
    • Testing Instructions:
      Hide

      1. Enable URL repository
      2. Open any page with texteditor and make sure that URL repository appears in the repositories list when you try to insert image but does not appear when you try to insert media

      Show
      1. Enable URL repository 2. Open any page with texteditor and make sure that URL repository appears in the repositories list when you try to insert image but does not appear when you try to insert media
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:
      wip-MDL-34047-master

      Description

      Repository 'URL downloader' does not specify that it returns only images and therefore it appears in filepicker when other file types are expected.

      Raised from discussion MDLSITE-1818, the moodle.org/plugins filemanager expects .zip file and 'URL downloader' is confusing in the repositories list there

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              marina Marina Glancy added a comment -

              TO INTEGRATORS: please cherry-pick to 23_STABLE

              Show
              marina Marina Glancy added a comment - TO INTEGRATORS: please cherry-pick to 23_STABLE
              Hide
              poltawski Dan Poltawski added a comment -

              As discussed, Marina could you create a branch for 22_STABLE and add testing instructions.

              Show
              poltawski Dan Poltawski added a comment - As discussed, Marina could you create a branch for 22_STABLE and add testing instructions.
              Hide
              marina Marina Glancy added a comment -

              Added branch for 2.2

              Show
              marina Marina Glancy added a comment - Added branch for 2.2
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks Marina, i've integrated that now

              Show
              poltawski Dan Poltawski added a comment - Thanks Marina, i've integrated that now
              Hide
              fred Frédéric Massart added a comment -

              Passed on 2.2, 2.3 and master. Cheers!

              Show
              fred Frédéric Massart added a comment - Passed on 2.2, 2.3 and master. Cheers!
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Congratulations your code is upstream - gold star for you!

              This issue + 79 others made it in in time for the minor releases.
              Thank you everyone involved for your exuberant efforts.

              Show
              samhemelryk Sam Hemelryk added a comment - Congratulations your code is upstream - gold star for you! This issue + 79 others made it in in time for the minor releases. Thank you everyone involved for your exuberant efforts.

                People

                • Votes:
                  1 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    9/Jul/12