Moodle
  1. Moodle
  2. MDL-30295

Invalid string for LTI component in define roles page.

    Details

    • Rank:
      32640

      Description

      Notice invalid string error in admin/roles/define.php page
      (Home > Site administration > Users > Permissions > Define roles)

      Invalid get_string() identifier: 'lti:manage' or component 'mod_lti'
      line 6216 of /lib/moodlelib.php: call to debugging()
      line 6808 of /lib/moodlelib.php: call to core_string_manager->get_string()
      line 2719 of /lib/accesslib.php: call to get_string()
      line 2577 of /lib/accesslib.php: call to get_capability_string()
      line 116 of /admin/roles/lib.php: call to get_capability_docs_link()
      line 482 of /admin/roles/lib.php: call to capability_table_base->display()
      line 792 of /admin/roles/lib.php: call to capability_table_with_risks->display()
      line 161 of /admin/roles/define.php: call to define_role_table_advanced->display()
      

        Issue Links

          Activity

          Hide
          Rossiani Wijaya added a comment -

          Since this issue is located at admin page, I think MDLQA-1412 is best issue to be linked.

          Show
          Rossiani Wijaya added a comment - Since this issue is located at admin page, I think MDLQA-1412 is best issue to be linked.
          Hide
          Chris Scribner added a comment -

          Integrate MDL-30339 before this one, and just take the ec580c1b65c8fb657b12b8ab22f927aa5d9012fa revision.

          Code for this is in branch: https://github.com/scriby/moodle/tree/MDL-30295

          Show
          Chris Scribner added a comment - Integrate MDL-30339 before this one, and just take the ec580c1b65c8fb657b12b8ab22f927aa5d9012fa revision. Code for this is in branch: https://github.com/scriby/moodle/tree/MDL-30295
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Passed, the notice does not happen anymore.

          Show
          Eloy Lafuente (stronk7) added a comment - Passed, the notice does not happen anymore.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Closing as fixed, many thanks for your effort!

          Note that the changes related to master (2.2beta) have been already sent upstream. But the stable ones will be part of next weeklies (Wed/Thu) as usual.

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Closing as fixed, many thanks for your effort! Note that the changes related to master (2.2beta) have been already sent upstream. But the stable ones will be part of next weeklies (Wed/Thu) as usual. Ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: