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

In workshop, teacher can't provide additional assessment during the grading evaluation phase

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.4
    • Component/s: Workshop
    • Labels:
    • Testing Instructions:
      Hide

      Testing difficulty: trivial

      1. Prepare a workshop with at least two students and one submission
      2. During the assessment phase, let students do their peer-assessments.
      3. Switch the workshop into the grading evaluation phase.
      4. TEST: Make sure that the teacher is able to assess submissions now.
      5. TEST: Make sire that students can't re-assess submissions allocated to them.

      Show
      Testing difficulty: trivial 1. Prepare a workshop with at least two students and one submission 2. During the assessment phase, let students do their peer-assessments. 3. Switch the workshop into the grading evaluation phase. 4. TEST: Make sure that the teacher is able to assess submissions now. 5. TEST: Make sire that students can't re-assess submissions allocated to them.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-36209-workshop-assess-while-evaluating

      Description

      While evaluating grades, the teacher may often want to provide additional assessment. The common case is when peer-assessments vary a lot or the workshop itself is unable to decide the "best" assessment - such a situation is described at http://docs.moodle.org/23/en/Using_Workshop#Grade_for_assessment

      To do so, the teacher has to switch to the assessment phase, provide the assessment and then switch back. This is not only annoying for the teacher. It can be also confusing for students and under certain situations, may even lead to an unwanted situation that a student gets an opportunity to assess something after the expected date/time.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            mudrd8mz David Mudrák added a comment -

            Submitting a patch for integration.

            Show
            mudrd8mz David Mudrák added a comment - Submitting a patch for integration.
            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
            poltawski Dan Poltawski added a comment -

            Hi David,

            Should this be backported?

            Show
            poltawski Dan Poltawski added a comment - Hi David, Should this be backported?
            Hide
            poltawski Dan Poltawski added a comment -

            Sorry David, i'm going to need to reopening this as i'm unsure of how to proceed.

            Show
            poltawski Dan Poltawski added a comment - Sorry David, i'm going to need to reopening this as i'm unsure of how to proceed.
            Hide
            cibot CiBoT added a comment -

            Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

            Show
            cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
            Hide
            poltawski Dan Poltawski added a comment -

            Thanks David for clarifying in the chat. I've integrated this now.

            Show
            poltawski Dan Poltawski added a comment - Thanks David for clarifying in the chat. I've integrated this now.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            After step 4 "Teacher is not able to reassess the grades of a student who has already been assessed by another student, is that expected behaviour?
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - After step 4 "Teacher is not able to reassess the grades of a student who has already been assessed by another student, is that expected behaviour? Thanks
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            As discussed in dev chat, it is expected.
            Passing
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - As discussed in dev chat, it is expected. Passing Thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Amazed. Inspired. Grateful. That’s how your generosity makes me feel.

            (not really)

            Closing, thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Amazed. Inspired. Grateful. That’s how your generosity makes me feel. (not really) Closing, thanks!

              People

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

                Dates

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