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

Numerical Units grading select element does not show the correct option.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.0.2
    • Component/s: Questions
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      Numerical Units grading select element does not show the correct option.
      The texts should be reversed

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            ppichet Pierre Pichet added a comment -

            Primarly
            numerical/questiontype.php
            line 1047 $unitgradingtypes = array('1' => get_string('decfractionofquestiongrade', 'qtype_numerical'), '2' => get_string('decfractionofresponsegrade', 'qtype_numerical'));
            should be change to

            $unitgradingtypes = array('1' => get_string('decfractionofresponsegrade', 'qtype_numerical'), '2' => get_string('decfractionofquestiongrade', 'qtype_numerical'));

            Will look if there are not other unit grading settings to change.

            and use this as a first git ...

            Show
            ppichet Pierre Pichet added a comment - Primarly numerical/questiontype.php line 1047 $unitgradingtypes = array('1' => get_string('decfractionofquestiongrade', 'qtype_numerical'), '2' => get_string('decfractionofresponsegrade', 'qtype_numerical')); should be change to $unitgradingtypes = array('1' => get_string('decfractionofresponsegrade', 'qtype_numerical'), '2' => get_string('decfractionofquestiongrade', 'qtype_numerical')); Will look if there are not other unit grading settings to change. and use this as a first git ...
            Hide
            ppichet Pierre Pichet added a comment -

            Tim
            Here a first GIT trial.
            https://github.com/ppichet/moodle/commit/ff6b824e6da0e48fdd1266612f0dc9c0ceea04bc

            Fixing GIT in Windows is somehow searching the LOST ARK...
            Finally I found the Z: directory containing the SSH keys in another server...

            Show
            ppichet Pierre Pichet added a comment - Tim Here a first GIT trial. https://github.com/ppichet/moodle/commit/ff6b824e6da0e48fdd1266612f0dc9c0ceea04bc Fixing GIT in Windows is somehow searching the LOST ARK... Finally I found the Z: directory containing the SSH keys in another server...
            Hide
            timhunt Tim Hunt added a comment -

            Thanks Pierre. That fix looks good to me, so I have made pull-request PULL-270 for it, so that should get into next week's weekly build.

            Just one thing I would change in future, your commit comment:

            MDL-26356:Correcting the texts of Numerical Units grading select element

            Does not quite match the standard format. You can see three good examples here: https://github.com/stronk7/moodle/compare/master...MDL-26229_restore_attempts_wip

            MDL-26229 backup & restore - fix quiz & lesson conflicts
            MDL-26229 restore - prevent tag-conflicting structures to be created
            MDL-26229 restore - parser improvements (repetitions and skipped tags)

            Just something to note for future bug-fixes. Thanks, Tim.

            Show
            timhunt Tim Hunt added a comment - Thanks Pierre. That fix looks good to me, so I have made pull-request PULL-270 for it, so that should get into next week's weekly build. Just one thing I would change in future, your commit comment: MDL-26356 :Correcting the texts of Numerical Units grading select element Does not quite match the standard format. You can see three good examples here: https://github.com/stronk7/moodle/compare/master...MDL-26229_restore_attempts_wip MDL-26229 backup & restore - fix quiz & lesson conflicts MDL-26229 restore - prevent tag-conflicting structures to be created MDL-26229 restore - parser improvements (repetitions and skipped tags) Just something to note for future bug-fixes. Thanks, Tim.
            Hide
            brittb Britt added a comment -

            This might be the same thing I wanted to report, but not sure. The bug I see is that if one sets up a quiz question with "Only the NUMERICAL ANSWER will be graded, Unit1 will be shown" then enters the units and saves, all is well. However, when re-entering that question to edit it, the Units Handling radio button is not set correctly. My proofreader/editor helper then cannot figure this out so her only option is to cancel the changes.

            Show
            brittb Britt added a comment - This might be the same thing I wanted to report, but not sure. The bug I see is that if one sets up a quiz question with "Only the NUMERICAL ANSWER will be graded, Unit1 will be shown" then enters the units and saves, all is well. However, when re-entering that question to edit it, the Units Handling radio button is not set correctly. My proofreader/editor helper then cannot figure this out so her only option is to cancel the changes.
            Hide
            ppichet Pierre Pichet added a comment - - edited

            This is not the same bug but MDL-26299 which was recently resolved.
            On testing with the proposed new MDL-26299 code, the probleem disappears.
            Your site needs updating.

            Show
            ppichet Pierre Pichet added a comment - - edited This is not the same bug but MDL-26299 which was recently resolved. On testing with the proposed new MDL-26299 code, the probleem disappears. Your site needs updating.
            Hide
            tsala Helen Foster added a comment -

            Pierre, thanks for reporting and fixing this issue.

            Show
            tsala Helen Foster added a comment - Pierre, thanks for reporting and fixing this issue.
            Hide
            tsala Helen Foster added a comment -

            Fix included in latest 2.0.1+ weekly.

            Show
            tsala Helen Foster added a comment - Fix included in latest 2.0.1+ weekly.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  21/Feb/11