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

      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.

        Gliffy Diagrams

          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: