Moodle
  1. Moodle
  2. MDL-35133

context_module::get_context_name does not do format_string

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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:
    • Rank:
      43763

      Description

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

        Issue Links

          Activity

          Hide
          Tim Hunt added a comment -

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

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

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

          Show
          Tim Hunt added a comment - Submitting for integration before I go on holiday. I am confident that this is right.
          Hide
          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
          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
          Eloy Lafuente (stronk7) added a comment -

          Integrated (22, 23 & master), thanks!

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

          Thanks Tim,

          seems to be working fine.

          Show
          Rajesh Taneja added a comment - Thanks Tim, seems to be working fine.
          Hide
          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
          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: