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

Coding error when changing locked state of a assignment with blind marking set

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a assignment with blind marking
      2. Go to gradebook and lock the column for this assignment
      3. Submit an assignment as a student
      4. Go to gradebook and unlock the column for the assignment.
      5. Verify you do not receive this error: Coding error detected, it must be fixed by a programmer: The property "context" is not set.
      Show
      Create a assignment with blind marking Go to gradebook and lock the column for this assignment Submit an assignment as a student Go to gradebook and unlock the column for the assignment. Verify you do not receive this error: Coding error detected, it must be fixed by a programmer: The property "context" is not set.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:

      Description

      assign_get_user_grades() is not correctly loading the assignment context. This results in errors when the gradebook uses this callback to mod_assign.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              corleone Raymond Antonio added a comment -

              Hi Damyon and Ankit,

              This is the patch for the issue and it is ready for testing. The patch's details are as follows:
              1. Repo
              git://github.com/raymondAntonio/moodle.git
              2. The branch
              MDL-35389
              And This is the diff :
              https://github.com/raymondAntonio/moodle/commit/f5ed5949841a56d3bd3ce21b7e59ed08d1613a67

              Cheers

              Show
              corleone Raymond Antonio added a comment - Hi Damyon and Ankit, This is the patch for the issue and it is ready for testing. The patch's details are as follows: 1. Repo git://github.com/raymondAntonio/moodle.git 2. The branch MDL-35389 And This is the diff : https://github.com/raymondAntonio/moodle/commit/f5ed5949841a56d3bd3ce21b7e59ed08d1613a67 Cheers
              Hide
              salvetore Michael de Raadt added a comment - - edited

              Hi, Raymond.

              Thanks for your work on this issue and others.

              I've added you to the jira-developers group in Tracker. That allows you to edit issues, assign them to yourself and push issues for peer review. You can't submit to integration, but Damyon or a peer reviewer at HQ can do that after peer review.

              Feel free to assign this issue to yourself and put your Git details in the appropriate slots. This would need some testing instructions before being sent to peer review.

              Show
              salvetore Michael de Raadt added a comment - - edited Hi, Raymond. Thanks for your work on this issue and others. I've added you to the jira-developers group in Tracker. That allows you to edit issues, assign them to yourself and push issues for peer review. You can't submit to integration, but Damyon or a peer reviewer at HQ can do that after peer review. Feel free to assign this issue to yourself and put your Git details in the appropriate slots. This would need some testing instructions before being sent to peer review.
              Hide
              damyon Damyon Wiese added a comment -

              This fix looks correct to me. Thanks Raymond.

              Show
              damyon Damyon Wiese added a comment - This fix looks correct to me. Thanks Raymond.
              Hide
              fred Frédéric Massart added a comment -

              I could reproduce this working on MDL-34931 with an assignment (default settings) with groupings set.

              Show
              fred Frédéric Massart added a comment - I could reproduce this working on MDL-34931 with an assignment (default settings) with groupings set.
              Hide
              poltawski Dan Poltawski 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
              poltawski Dan Poltawski 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
              corleone Raymond Antonio added a comment -

              Hi Dan
              Rebase done. Cheers

              Show
              corleone Raymond Antonio added a comment - Hi Dan Rebase done. Cheers
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Hi Raymond,

              Changes look good thanks, I'll integrate this when git.moodle.org is available again.
              Again just to confirm is this issue 2.4 only?

              Cheers
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Hi Raymond, Changes look good thanks, I'll integrate this when git.moodle.org is available again. Again just to confirm is this issue 2.4 only? Cheers Sam
              Hide
              damyon Damyon Wiese added a comment -

              Hi Sam,

              Yep 2.4 only

              Cheers
              Damyon

              Show
              damyon Damyon Wiese added a comment - Hi Sam, Yep 2.4 only Cheers Damyon
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Integrated now thanks

              Show
              samhemelryk Sam Hemelryk added a comment - Integrated now thanks
              Hide
              markn Mark Nelson added a comment - - edited

              The assignment was unlocked by default. I changed this to locked and back to unlocked and did not receive any errors. Passed.

              Show
              markn Mark Nelson added a comment - - edited The assignment was unlocked by default. I changed this to locked and back to unlocked and did not receive any errors. Passed.
              Hide
              poltawski Dan Poltawski added a comment -

              Congratulations, you've done it!

              Nf n erjneq sbe fhpprfshy vagrtengvba vagb guvf jrrxf eryrnfr, V pna abj qvfpybfr gb lbh gur rkvfgnapr bs shapgvba fge_ebg13(), gb tb va lbhe gbbyxvg nybat jvgu uggc://cuc.arg/znahny/ra/shapgvba.tmtrgff.cuc

              Cyrnfr qb abg nyybj guvf vasbezngvba gb cnff shegure.

              Show
              poltawski Dan Poltawski added a comment - Congratulations, you've done it! Nf n erjneq sbe fhpprfshy vagrtengvba vagb guvf jrrxf eryrnfr, V pna abj qvfpybfr gb lbh gur rkvfgnapr bs shapgvba fge_ebg13(), gb tb va lbhe gbbyxvg nybat jvgu uggc://cuc.arg/znahny/ra/shapgvba.tmtrgff.cuc Cyrnfr qb abg nyybj guvf vasbezngvba gb cnff shegure.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    3/Dec/12