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

context_module::get_context_name does not do format_string

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.4, 2.3, 2.4
    • Fix Version/s: 2.2.5, 2.3.2
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide

      0. Ensure you have the multi-lang filter turned on.

      1. Create some activities with multi-lang names. (If you need more detailed instructions than this, see MDL-27818.)

      2. Go to somewhere that prints context names, for example the check permissions page or the question bank. Ensure that the context name is displayed correctly, that is, just the current language shown.

      Show
      0. Ensure you have the multi-lang filter turned on. 1. Create some activities with multi-lang names. (If you need more detailed instructions than this, see MDL-27818 .) 2. Go to somewhere that prints context names, for example the check permissions page or the question bank. Ensure that the context name is displayed correctly, that is, just the current language shown.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Context course, context coursecat, etc. do. Seems like a simple oversight to me, I will make a fix.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              timhunt Tim Hunt added a comment -

              Petr, please could you take a quick look at this one-line change. Thanks.

              Show
              timhunt Tim Hunt added a comment - Petr, please could you take a quick look at this one-line change. Thanks.
              Hide
              timhunt Tim Hunt added a comment -

              Submitting for integration before I go on holiday. I am confident that this is right.

              Show
              timhunt Tim Hunt added a comment - Submitting for integration before I go on holiday. I am confident that this is right.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated (22, 23 & master), thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (22, 23 & master), thanks!
              Hide
              rajeshtaneja Rajesh Taneja added a comment -

              Thanks Tim,

              seems to be working fine.

              Show
              rajeshtaneja Rajesh Taneja added a comment - Thanks Tim, seems to be working fine.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Many thanks for the hard work.

              These changes have been spread upstream and are already available in the git and cvs repositories.

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work. These changes have been spread upstream and are already available in the git and cvs repositories. Ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    10/Sep/12