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

The webdav-repository loads empty files

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.0.4
    • Component/s: Repositories
    • Labels:
      None
    • Testing Instructions:
      Hide

      You need a valid webdav-server and a related instance of the webdav-repository plugin.
      Insert at any context a file from this repository so the file will be empty.

      Show
      You need a valid webdav-server and a related instance of the webdav-repository plugin. Insert at any context a file from this repository so the file will be empty.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-27530_head

      Description

      The webdav-repository does not load files correctly. This results in empty files.
      The reason for this is that the url is overriden by the save path before the url is passed to the webdav-get() function.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys this has been integrated now.

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys this has been integrated now. Cheers Sam
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            And this is now part of Moodle upstream, many thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - And this is now part of Moodle upstream, many thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11