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
    • Rank:
      17604

      Description

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

        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: