Merge branch 'MDL-66968' of https://github.com/stronk7/moodle
authorJun Pataleta <jun@moodle.com>
Fri, 25 Oct 2019 02:20:05 +0000 (10:20 +0800)
committerJun Pataleta <jun@moodle.com>
Fri, 25 Oct 2019 02:20:05 +0000 (10:20 +0800)
1  2 
lib/outputcomponents.php

diff --combined lib/outputcomponents.php
@@@ -228,7 -228,7 +228,7 @@@ class user_picture implements renderabl
          // only touch the DB if we are missing data and complain loudly...
          $needrec = false;
          foreach (self::$fields as $field) {
-             if (!array_key_exists($field, $user)) {
+             if (!property_exists($user, $field)) {
                  $needrec = true;
                  debugging('Missing '.$field.' property in $user object, this is a performance problem that needs to be fixed by a developer. '
                            .'Please use user_picture::fields() to get the full list of required fields.', DEBUG_DEVELOPER);
@@@ -4200,32 -4200,32 +4200,32 @@@ class action_menu implements renderable
  
      /**
       * An icon to use for the toggling the secondary menu (dropdown).
 -     * @var actionicon
 +     * @var pix_icon
       */
      public $actionicon;
  
      /**
       * Any text to use for the toggling the secondary menu (dropdown).
 -     * @var menutrigger
 +     * @var string
       */
      public $menutrigger = '';
  
      /**
       * Any extra classes for toggling to the secondary menu.
 -     * @var triggerextraclasses
 +     * @var string
       */
      public $triggerextraclasses = '';
  
      /**
       * Place the action menu before all other actions.
 -     * @var prioritise
 +     * @var bool
       */
      public $prioritise = false;
  
      /**
       * Constructs the action menu with the given items.
       *
 -     * @param array $actions An array of actions.
 +     * @param array $actions An array of actions (action_menu_link|pix_icon|string).
       */
      public function __construct(array $actions = array()) {
          static $initialised = 0;
       * Sets the label for the menu trigger.
       *
       * @param string $label The text
 -     * @return null
       */
      public function set_action_label($label) {
          $this->actionlabel = $label;
       *
       * @param string $trigger The text
       * @param string $extraclasses Extra classes to style the secondary menu toggle.
 -     * @return null
       */
      public function set_menu_trigger($trigger, $extraclasses = '') {
          $this->menutrigger = $trigger;