Moodle
  1. Moodle
  2. MDL-32474 META: Repositories need to return more information about files
  3. MDL-32497

WebDAV repository needs to return more information and work correctly with subdirs

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Repositories
    • Labels:
    • Testing Instructions:
      Hide

      1. Setup WebDav server (see http://samiux.wordpress.com/2009/06/27/howto-webdav-on-ubuntu-9-04-server/)
      2. Create several folders with subfolders and files on this server. Use special characters in folders and file names (spaces, etc.)
      3. Enable WebDav repository and create two instances, one with path "/" and another with non-empty path
      4. Access WebDav from Filepicker and make sure you can navigate correctly within the folders and back and select files

      Show
      1. Setup WebDav server (see http://samiux.wordpress.com/2009/06/27/howto-webdav-on-ubuntu-9-04-server/ ) 2. Create several folders with subfolders and files on this server. Use special characters in folders and file names (spaces, etc.) 3. Enable WebDav repository and create two instances, one with path "/" and another with non-empty path 4. Access WebDav from Filepicker and make sure you can navigate correctly within the folders and back and select files
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      wip-MDL-32497-master
    • Rank:
      39389

      Description

      WebDAV repository needs to return as much information about files as possible (see parent task).
      The current webdav library returns the information about date last modified and size of the files. We need to pass this to filepicker.

      Also, the WebDAV repository needs to navigate correctly inside subdirectories and back, especially when it is configured with a non-empty root path.

        Issue Links

          Activity

          Hide
          Dan Poltawski 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
          Dan Poltawski 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
          Dan Poltawski added a comment -

          Thanks Marina, this has been integrated now

          Show
          Dan Poltawski added a comment - Thanks Marina, this has been integrated now
          Hide
          Adrian Greeve added a comment -

          Test passed, though it does have some issues with strict error reporting. I'll create a bug for that (stuff for Petr to fix).
          Thanks.

          Show
          Adrian Greeve added a comment - Test passed, though it does have some issues with strict error reporting. I'll create a bug for that (stuff for Petr to fix). Thanks.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This has been near becoming rejected, because it's not the best code you are able to produce.

          But, luckily, at the end, it has landed and has been spread to all repos out there.

          Many thanks and, don't forget it, keep improving your skills, you can!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - This has been near becoming rejected, because it's not the best code you are able to produce. But, luckily, at the end, it has landed and has been spread to all repos out there. Many thanks and, don't forget it, keep improving your skills, you can! Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: