Moodle
  1. Moodle
  2. MDL-45036

Mentee block needs an update to include additional name fields.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6.3
    • Component/s: Blocks
    • Labels:

      Description

      The mentee block currently produces the following developer dubugging message:
      You need to update your sql to include additional name fields in the user object.

      line 3571 of /lib/moodlelib.php: call to debugging()
      line 62 of /blocks/mentees/block_mentees.php: call to fullname()
      line 294 of /blocks/moodleblock.class.php: call to block_mentees->get_content()
      line 236 of /blocks/moodleblock.class.php: call to block_base->formatted_contents()
      line 956 of /lib/blocklib.php: call to block_base->get_content_for_output()
      line 1008 of /lib/blocklib.php: call to block_manager->create_block_contents()
      line 361 of /lib/outputrenderers.php: call to block_manager->ensure_content_created()
      line 45 of /theme/clean/layout/columns3.php: call to core_renderer->standard_head_html()
      line 877 of /lib/outputrenderers.php: call to include()
      line 807 of /lib/outputrenderers.php: call to core_renderer->render_page_layout()
      line 101 of /index.php: call to core_renderer->header()

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            CiBoT added a comment -

            Results for MDL-45036

            • Remote repository: git://github.com/abgreeve/moodle.git
            Show
            CiBoT added a comment - Results for MDL-45036 Remote repository: git://github.com/abgreeve/moodle.git Remote branch wip- MDL-45036 -26 to be integrated into upstream MOODLE_26_STABLE Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/2887 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/2887/artifact/work/smurf.html Remote branch wip- MDL-45036 -master to be integrated into upstream master Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/2888 Details: http://integration.moodle.org/job/Precheck%20remote%20branch/2888/artifact/work/smurf.html
            Hide
            Michael Aherne added a comment -

            This change looks good to me, and I don't see any need for changes before it's sent to integration.

            [Y] Syntax
            [Y] Whitespace
            [-] Output
            [-] Language
            [-] Databases
            [Y] Testing (instructions and automated tests)
            [-] Security
            [-] Documentation
            [Y] Git
            [-] Third party code
            [Y] Sanity check

            Show
            Michael Aherne added a comment - This change looks good to me, and I don't see any need for changes before it's sent to integration. [Y] Syntax [Y] Whitespace [-] Output [-] Language [-] Databases [Y] Testing (instructions and automated tests) [-] Security [-] Documentation [Y] Git [-] Third party code [Y] Sanity check
            Hide
            Adrian Greeve added a comment -

            Thanks Michael for the peer-review. I greatly appreciate your help.

            Submitting for integration.

            Show
            Adrian Greeve added a comment - Thanks Michael for the peer-review. I greatly appreciate your help. Submitting for integration.
            Hide
            Dan Poltawski added a comment -

            Integrated to master and 26 - thanks Adrian

            Show
            Dan Poltawski added a comment - Integrated to master and 26 - thanks Adrian
            Hide
            Andrew Nicols added a comment -

            Thanks Adrian,

            Working as described and I got to learn a bit more about relative role assignments. Thanks

            Andrew

            Show
            Andrew Nicols added a comment - Thanks Adrian, Working as described and I got to learn a bit more about relative role assignments. Thanks Andrew
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Your code is now part of Moodle upstream, many thanks!

            It's a constant, continuous,
            spectacular world we live in,
            and every day you see things that
            just knock you out, if you pay attention.

            Robert Irwin

            Show
            Eloy Lafuente (stronk7) added a comment - Your code is now part of Moodle upstream, many thanks! It's a constant, continuous, spectacular world we live in, and every day you see things that just knock you out, if you pay attention. Robert Irwin

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: