[MDL-46229] Additional names fields missing on badges external page - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_26_STABLE, MOODLE_27_STABLE
Fixed Branches:
MOODLE_26_STABLE, MOODLE_27_STABLE
Pull from Repository:
https://github.com/totara/openbadges.git
Testing Instructions:

Hide

With a user who earned at least one internal badge, check the issued badge page. Make sure there are no warnings about missing names fields.

The same should be done for external badges. In this case you will need to have a user with a backpack connected and a badge from the backpack displayed.

Show
With a user who earned at least one internal badge, check the issued badge page. Make sure there are no warnings about missing names fields. The same should be done for external badges. In this case you will need to have a user with a backpack connected and a badge from the backpack displayed.

Warning message on external badges page, see screenshot. Affects 2.6 and 2.7 but not master

Also both external and internal badges pages look weird in 2.6 and 2.7 in Clean. Somehow was fixed in master.

2.6:

You need to update your sql to include additional name fields in the user object.

line 3570 of /lib/moodlelib.php: call to debugging()

line 416 of /badges/renderer.php: call to fullname()

line 221 of /lib/outputrenderers.php: call to core_badges_renderer->render_external_badge()

line 96 of /badges/external.php: call to plugin_renderer_base->render()

2.7

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 416 of /badges/renderer.php: call to fullname()

line 221 of /lib/outputrenderers.php: call to core_badges_renderer->render_external_badge()

line 96 of /badges/external.php: call to plugin_renderer_base->render()

Testing discovered

MDL-46107 Remove empty breadcrumb on external badges page

Participants:: CiBoT, Eloy Lafuente (stronk7), Marina Glancy, Sam Hemelryk, Simey Lameze, Yuliya Bozhko

Component watchers:: Yuliya Bozhko, Amaia Anabitarte, Carlos Escobedo, Laurent David, Sabina Abellan, Sara Arjona (@sarjona)