Uploaded image for project: '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
            rwijaya Rossiani Wijaya added a comment -

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

            Show
            rwijaya Rossiani Wijaya added a comment - Since this issue is located at admin page, I think MDLQA-1412 is best issue to be linked.
            Hide
            scriby 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
            scriby 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
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

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

            Passed, the notice does not happen anymore.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Passed, the notice does not happen anymore.
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  5/Dec/11