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

ID number of assignment grade item changes

    Details

    • Testing Instructions:
      Hide
      1. Create/edit and assignment and set the ID number to "A".
      2. Edit the grade item of the assignment (in Grades -> Categories and items -> Simple view). The ID number is "A".
      3. Edit a submission as a student or grade a submission as a teacher.
      4. Edit the assignment settings. Verify the ID number is still 'A'.
      5. Edit the grade item of the assignment. Verify the ID number is still 'A'.
      Show
      Create/edit and assignment and set the ID number to "A". Edit the grade item of the assignment (in Grades -> Categories and items -> Simple view). The ID number is "A". Edit a submission as a student or grade a submission as a teacher. Edit the assignment settings. Verify the ID number is still 'A'. Edit the grade item of the assignment. Verify the ID number is still 'A'.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-39487_master

      Description

      The ID number of assignment grades items is overwritten every time a student edits a submission or a teacher grades a submission. Only the ID number of the grade item changes, the ID number of the assignment does not.

      Steps to reproduce:

      1. Create/edit and assignment and set the ID number to "A".
      2. Edit the grade item of the assignment (in Grades -> Categories and items -> Simple view). The ID number is "A".
      3. Edit a submission as a student or grade a submission as a teacher.
      4. Edit the assignment settings. The ID number is still 'A'.
      5. Edit the grade item of the assignment. The ID number is set to a number.

      I've traced the bug to function gradebook_item_update of mod/assign/locallib.php. It updates the grade item, using the course module id as id number.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            damyon Damyon Wiese added a comment -

            This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue.

            For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

            Show
            damyon Damyon Wiese added a comment - This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue. For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment
            Hide
            damyon Damyon Wiese added a comment -

            Thanks Albert,

            This change looks perfect to me. Sending for integration.

            Show
            damyon Damyon Wiese added a comment - Thanks Albert, This change looks perfect to me. Sending for integration.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (23, 24, 25 - by cherrypick - and master), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (23, 24, 25 - by cherrypick - and master), thanks!
            Hide
            andyjdavis Andrew Davis added a comment -

            Works as described. Passing.

            Show
            andyjdavis Andrew Davis added a comment - Works as described. Passing.
            Hide
            poltawski Dan Poltawski added a comment -

            Feature: Thanks to our superb contributors
              In order to make Moodle better
              As an integrator
              I need to thank all our contributors
             
              Scenario: Dan thanks you all
                Given I log in as "dan"
                And I see "lots of fixed issues"
                When I follow "Close integrated issues"
                Then I should see "Lots of thanks to all our contributors"
            

            Your changes are upstream

            Show
            poltawski Dan Poltawski added a comment - Feature: Thanks to our superb contributors In order to make Moodle better As an integrator I need to thank all our contributors   Scenario: Dan thanks you all Given I log in as "dan" And I see "lots of fixed issues" When I follow "Close integrated issues" Then I should see "Lots of thanks to all our contributors" Your changes are upstream

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  8/Jul/13