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

Untranslatable string in Workshop module

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.0.4
    • Component/s: Workshop
    • Labels:
      None

      Description

      I've found an untranslatable string in the assessment form of the workshop module.

      Have searched a bit and I'm pretty sure that the wrong line is in

      form/numerrors/assessment_form.php:74: $mform->addElement('radio', 'grade_idx' . $i, 'Your assessment', $fields->

      {'grade0__idx_'.$i}

      , 0); // todo localize

      which clearly is marked as "todo localize"

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            mudrd8mz David Mudrák added a comment -

            Ooops, thanks for the report!

            Show
            mudrd8mz David Mudrák added a comment - Ooops, thanks for the report!
            Hide
            mudrd8mz David Mudrák added a comment -

            The submitted branch fixes the reported hardcoded string as well as some others spotted during a general code review.

            Show
            mudrd8mz David Mudrák added a comment - The submitted branch fixes the reported hardcoded string as well as some others spotted during a general code review.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, assuming that other strings like 'debug::indent::next square should be from group id ' were left untranslated by the developer and the target for this was, mainly 'Your assessment'.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, assuming that other strings like 'debug::indent::next square should be from group id ' were left untranslated by the developer and the target for this was, mainly 'Your assessment'. Ciao
            Hide
            mudrd8mz David Mudrák added a comment -

            Correct, the debugging strings are left in English intentionally. Thanks for the prompt integration!

            Show
            mudrd8mz David Mudrák added a comment - Correct, the debugging strings are left in English intentionally. Thanks for the prompt integration!
            Hide
            samhemelryk Sam Hemelryk added a comment -

            PASS

            Show
            samhemelryk Sam Hemelryk added a comment - PASS
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            All git & cvs servers have been updated with these cool changes, so closing, many thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - All git & cvs servers have been updated with these cool changes, so closing, many thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11