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

          Attachments

            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