Moodle
  1. Moodle
  2. MDL-27962

Assignment - Unable to hide a column on submissions page

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1
    • Component/s: Assignment (2.2)
    • Labels:
    • Testing Instructions:
      Hide

      1) Create an online assignment (any type should work the same)
      2) Add in a couple of student submissions
      3) Go to the submissions page [quick grading off]
      4) Click on the [-] button in the column heading to hide a column (like comments)
      The comment column is no longer disappearing

      Show
      1) Create an online assignment (any type should work the same) 2) Add in a couple of student submissions 3) Go to the submissions page [quick grading off] 4) Click on the [-] button in the column heading to hide a column (like comments) The comment column is no longer disappearing
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull Master Branch:
      wip-MDL-27962-master

      Description

      Unable to remove columns when viewing the submissions page - for example, the comments column.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Anthony Borrow added a comment -

            This is another one of those frustrating ones that is technically not related to the functionality of the test IMO but according to the test instructions it requires me to fail MDLQA-1013. Peace - Anthony

            Show
            Anthony Borrow added a comment - This is another one of those frustrating ones that is technically not related to the functionality of the test IMO but according to the test instructions it requires me to fail MDLQA-1013 . Peace - Anthony
            Hide
            Sam Hemelryk added a comment -

            Good spotting Anthony - this ended up being a typo in within the flexible table code that was introduced in master only.

            Show
            Sam Hemelryk added a comment - Good spotting Anthony - this ended up being a typo in within the flexible table code that was introduced in master only.
            Hide
            Michael de Raadt added a comment -

            Promoting this issue as it relates to a QA test.

            Show
            Michael de Raadt added a comment - Promoting this issue as it relates to a QA test.
            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.

            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.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Passing test without further action. Will be tested by MDLQA-1013 once this meets upstream.

            Show
            Eloy Lafuente (stronk7) added a comment - Passing test without further action. Will be tested by MDLQA-1013 once this meets upstream.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Your awesome code is now part of the next Moodle release, congrats!

            Show
            Eloy Lafuente (stronk7) added a comment - Your awesome code is now part of the next Moodle release, congrats!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: