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

Language packages contain various syntax errors

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.7
    • Fix Version/s: None
    • Component/s: Language
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE

      Description

      Various syntax errors were discovered during the fix of MDL-20981. Some of them were introduced because of the bug in admin/lang.php tool, some are typos made by translators. One day (hopefully soon) we should have strong syntax checker/compiler to avoid them. Particular bugs will be reported as comments to this issues.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            mudrd8mz David Mudrák added a comment -

            Fixed missing dollar sign in placeholders, spaces between dollar sign and variable name, missing placeholder variable, placeholder syntax in RTL languagesand some other various placeholder syntax errors.

            TODO: find and escape all non-placeholders like $CFG, $SESSION and other that would get eval()'uated in get_string()

            Show
            mudrd8mz David Mudrák added a comment - Fixed missing dollar sign in placeholders, spaces between dollar sign and variable name, missing placeholder variable, placeholder syntax in RTL languagesand some other various placeholder syntax errors. TODO: find and escape all non-placeholders like $CFG, $SESSION and other that would get eval()'uated in get_string()
            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting this issue.

            We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

            If you believe that this issue is still relevant to current versions (2.3 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

            Michael d;

            4d6f6f646c6521

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.3 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; 4d6f6f646c6521
            Hide
            mudrd8mz David Mudrák added a comment -

            This has been resolved by introducing AMOS. Closing.

            Show
            mudrd8mz David Mudrák added a comment - This has been resolved by introducing AMOS. Closing.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: