Moodle
  1. Moodle
  2. MDL-30295

Invalid string for LTI component in define roles page.

    Details

      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()

        Gliffy Diagrams

          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: