Moodle
  1. Moodle
  2. MDL-33860

It´s not possible translate the "All" option on the Assignments per page configuration

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Install any language pack e.g. Africaans
      2. Change language to new language
      3. Go to an instance of the assignment and view the grading table
      4. View the options in the assignments per page select list
      5. Verify the first option (All) has been customised to the current language (Alle)
      Show
      Install any language pack e.g. Africaans Change language to new language Go to an instance of the assignment and view the grading table View the options in the assignments per page select list Verify the first option (All) has been customised to the current language (Alle)
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
    • Rank:
      41961

      Description

      The "All" option on the Assignments per page configuration is not translated on assign/gradingoptionsform.php line 50.

      Replication instructions:

      1. Change the moodle language to Portuguese
      2. Enter an assignment
      3. Click View/grade all submissions

      Result: The "All" option is not translated. Every Occurrence of "All" is translated on the language files so there is a problem with this translation.

        Activity

        Hide
        Damyon Wiese added a comment -

        Thanks for reporting this Antonio. This patch just uses the string from the core language pack so there won't be anything else to update in the assignment strings.

        Show
        Damyon Wiese added a comment - Thanks for reporting this Antonio. This patch just uses the string from the core language pack so there won't be anything else to update in the assignment strings.
        Hide
        Michael de Raadt added a comment -

        Looks good to me.

        Show
        Michael de Raadt added a comment - Looks good to me.
        Hide
        Dan Poltawski added a comment -

        Hi Damyon,

        This is conflicting on integration - is there any chance you could fix this up? (I know its fairly simple just don't want to make a mistake)

        thanks,
        dan

        Show
        Dan Poltawski added a comment - Hi Damyon, This is conflicting on integration - is there any chance you could fix this up? (I know its fairly simple just don't want to make a mistake) thanks, dan
        Hide
        Damyon Wiese added a comment -

        Rebased with integration.

        Show
        Damyon Wiese added a comment - Rebased with integration.
        Hide
        Dan Poltawski added a comment -

        Integrated, thanks

        Show
        Dan Poltawski added a comment - Integrated, thanks
        Hide
        Dan Poltawski added a comment -

        Tested and passed - thanks

        Show
        Dan Poltawski added a comment - Tested and passed - thanks
        Hide
        Eloy Lafuente (stronk7) added a comment -

        And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay!

        Many, many thanks for your hard work!

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay! Many, many thanks for your hard work! Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: