Moodle
  1. Moodle
  2. MDL-29490 META: WebDAV repository improvement
  3. MDL-29348

Breadcrumb in File Picker is broken for the WebDAV Repository plugin

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1.1, 2.2.4
    • Fix Version/s: 2.2.6
    • Component/s: Repositories
    • Labels:
      None
    • Testing Instructions:
      Hide

      Test pre-requisites

      • Enable WebDAV repositories
      • Set up an instance of WebDAV

      Test steps

      1. Go to your private files
      2. Click on add and navigate through the WebDAV instance
      3. Make sure you can navigate through directories
      4. Make sure the breadcrumb displays accurate information
      5. Make sure the breadcrumb links are correct
      6. Make sure you can download files from any folder
      Show
      Test pre-requisites Enable WebDAV repositories Set up an instance of WebDAV Test steps Go to your private files Click on add and navigate through the WebDAV instance Make sure you can navigate through directories Make sure the breadcrumb displays accurate information Make sure the breadcrumb links are correct Make sure you can download files from any folder
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Rank:
      18973

      Description

      The breadcrumb only shows the top-level folder 'WebDAV/'. It does not update with folder navigations. Equivalent to issue for Alfresco Repository Plugin in #MDL-27022

        Activity

        Hide
        Frédéric Massart added a comment -

        @Integators: Moodle 2.2 integration ONLY.

        Show
        Frédéric Massart added a comment - @Integators: Moodle 2.2 integration ONLY.
        Hide
        Marina Glancy added a comment -

        looks ok for me. There are a couple of possible improvements I discussed with Fred but for 2.2 and this particular Path issue this fix is enough

        Show
        Marina Glancy added a comment - looks ok for me. There are a couple of possible improvements I discussed with Fred but for 2.2 and this particular Path issue this fix is enough
        Hide
        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
        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
        Eloy Lafuente (stronk7) added a comment -

        The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday.

        This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody.

        This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P

        Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday. This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody. This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao
        Hide
        Eloy Lafuente (stronk7) added a comment -

        I did not understand a word nor tried to decipher it, lol.

        Integrated (22 only), thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - I did not understand a word nor tried to decipher it, lol. Integrated (22 only), thanks!
        Hide
        Ankit Agarwal added a comment -

        This is working as expected

        Thanks

        Show
        Ankit Agarwal added a comment - This is working as expected Thanks
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Gutta cavat lapidem, non vi sed saepe cadendo - Ovidio

        This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

        Thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Gutta cavat lapidem, non vi sed saepe cadendo - Ovidio This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

          People

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

            Dates

            • Created:
              Updated:
              Resolved: