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

Filemanager in User private files does not save changes

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.1.6, 2.2.3
    • Fix Version/s: 2.1.7, 2.2.4
    • Component/s: Files API
    • Labels:
    • Testing Instructions:
      Hide
      1. Log in as user
      2. Go to My private files (Navigation->My profile->My private files)
      3. click 'Manage my private files'
      4. Create a folder and add a file
      5. click 'Save changes'
      6. Make sure file and folder is saved.
      Show
      Log in as user Go to My private files (Navigation->My profile->My private files) click 'Manage my private files' Create a folder and add a file click 'Save changes' Make sure file and folder is saved.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      wip-mdl-32866

      Description

      Occured very recently, it was working before:

      Select: Navigation->My profile->My private files
      click button 'Manage my private files'
      add a file
      click button 'Save changes'

      • no changes were saved.

      This is a standard file manager, that continues to work for other file areas, for example when you edit files in the folder resource

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            dongsheng Dongsheng Cai added a comment -

            My branch which rebased at 27/04/2012 not affected, so it must be recent changes.

            Show
            dongsheng Dongsheng Cai added a comment - My branch which rebased at 27/04/2012 not affected, so it must be recent changes.
            Hide
            poltawski Dan Poltawski added a comment -

            Looks good, fast work Raj!

            Show
            poltawski Dan Poltawski added a comment - Looks good, fast work Raj!
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks Dan.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks Dan.
            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
            nebgor Aparup Banerjee added a comment -

            Thanks, thats been picked cleanly into 21, 22 and master.
            Please test in all branches.

            Show
            nebgor Aparup Banerjee added a comment - Thanks, thats been picked cleanly into 21, 22 and master. Please test in all branches.
            Hide
            andyjdavis Andrew Davis added a comment -

            Tested adding and deleting a folder and file in master, 2.2 and 2.1. Passing.

            Show
            andyjdavis Andrew Davis added a comment - Tested adding and deleting a folder and file in master, 2.2 and 2.1. Passing.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            U P S T R E A M I Z E D !

            Many thanks for the hard work, closing this as fixed.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - U P S T R E A M I Z E D ! Many thanks for the hard work, closing this as fixed. Ciao

              People

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

                Dates

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