MDL-69262 core: Make instance count variable as a static class variable
authorJun Pataleta <jun@moodle.com>
Thu, 27 Aug 2020 05:17:45 +0000 (13:17 +0800)
committerJun Pataleta <jun@moodle.com>
Thu, 27 Aug 2020 05:24:42 +0000 (13:24 +0800)
commit7ef7aaecac7def504e360a6e6fc5f9e132a2fa00
tree3907b9efd9eb6222187dd4c46938fbb4b34edac6
parenta04309d40ed938112310691f382bcc17beff7ccc
MDL-69262 core: Make instance count variable as a static class variable

With the static function variable $instance, calls to
\action_menu_link::export_for_template() from its subclasses are stored
in different variables. This causes duplicate IDs when different
implementations of action menu links are rendered on the action menu
trigger/link template. (e.g. action_menu_link and
action_menu_link_secondary both rendered on the same page).
To make the incrementing uniform for the action_menu_link class and its
implementations, the $instance variable is moved out of the
export_to_template() method and is now made to a static class variable.
lib/outputcomponents.php