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

The string 'Table of contents' can not be translated

    Details

    • Testing Instructions:
      Hide

      Verify that the wiki displays a text string 'Table of contents'
      Create a page in a wiki with text and headers. Add text and headers until the wiki automatically create a TOC

      Search for the string on the Language customization page (or in AMOS, lang.moodle.org)

      Show
      Verify that the wiki displays a text string 'Table of contents' Create a page in a wiki with text and headers. Add text and headers until the wiki automatically create a TOC Search for the string on the Language customization page (or in AMOS, lang.moodle.org)
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:

      Description

      When a wiki auto create a Table of contents it displays a header 'Table of contents'. This text string can not be translated, it can not be found in the language pack.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              Thanks for reporting this.

              I've put that on the backlog.

              In the meantime feel free to help us work on this issue. If you are able to provide a patch, please add a patch label so we will spot it.

              Show
              salvetore Michael de Raadt added a comment - Thanks for reporting this. I've put that on the backlog. In the meantime feel free to help us work on this issue. If you are able to provide a patch, please add a patch label so we will spot it.
              Hide
              sunner Sunner Sun added a comment - - edited

              I just made the patch:

              Pull from Repository: git://github.com/sunner/moodle.git
              Pull 2.1 Branch: MDL-31201-m21
              Pull 2.1 Diff URL: https://github.com/sunner/moodle/compare/MOODLE_21_STABLE...MDL-31201-m21
              Pull 2.2 Branch: MDL-31201-m22
              Pull 2.2 Diff URL: https://github.com/sunner/moodle/compare/MOODLE_22_STABLE...MDL-31201-m22
              Pull Master Branch: MDL-31201-master
              Pull Master Diff URL: https://github.com/sunner/moodle/compare/master...MDL-31201-master

              Wish they are helpful

              Show
              sunner Sunner Sun added a comment - - edited I just made the patch: Pull from Repository: git://github.com/sunner/moodle.git Pull 2.1 Branch: MDL-31201 -m21 Pull 2.1 Diff URL: https://github.com/sunner/moodle/compare/MOODLE_21_STABLE...MDL-31201-m21 Pull 2.2 Branch: MDL-31201 -m22 Pull 2.2 Diff URL: https://github.com/sunner/moodle/compare/MOODLE_22_STABLE...MDL-31201-m22 Pull Master Branch: MDL-31201 -master Pull Master Diff URL: https://github.com/sunner/moodle/compare/master...MDL-31201-master Wish they are helpful
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks for the patches Sunner - assigning to myself to push through integration.

              Show
              poltawski Dan Poltawski added a comment - Thanks for the patches Sunner - assigning to myself to push through integration.
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks Sun!

              Putting your patch up for peer review.

              Show
              poltawski Dan Poltawski added a comment - Thanks Sun! Putting your patch up for peer review.
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              Looks simple enough.
              +1 for integration.
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - Looks simple enough. +1 for integration. Thanks
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks Ankit.

              To Integrator: can be cherry-picked to 23, 22, 21

              Show
              poltawski Dan Poltawski added a comment - Thanks Ankit. To Integrator: can be cherry-picked to 23, 22, 21
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated (21, 22, 23 & master), thanks!

              PS: Note 21_STABLE is already out of support so we are stopping bug fixes to land there along the next days...

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (21, 22, 23 & master), thanks! PS: Note 21_STABLE is already out of support so we are stopping bug fixes to land there along the next days...
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Passing, the TOC is now using the string as expected.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Passing, the TOC is now using the string as expected.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Closing this as fixed, changes are now available upstream.

              Big thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Closing this as fixed, changes are now available upstream. Big thanks!

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    9/Jul/12