Merge "Hooks: Use private instead of public for internal functions"

This commit is contained in:
jenkins-bot 2024-03-05 02:03:46 +00:00 committed by Gerrit Code Review
commit 49a1f27bfe

View file

@ -365,7 +365,7 @@ class Hooks implements
* @param string $name
* @return string of HTML
*/
public static function makeIcon( $name ) {
private static function makeIcon( $name ) {
// Html::makeLink will pass this through rawElement
return '<span class="vector-icon mw-ui-icon-' . $name . ' mw-ui-icon-wikimedia-' . $name . '"></span>';
}
@ -424,7 +424,7 @@ class Hooks implements
* @param bool $unsetIcon should the icon field be unset?
* @return array $item Updated menu item data
*/
public static function updateMenuItemData( $item, $unsetIcon = true ) {
private static function updateMenuItemData( $item, $unsetIcon = true ) {
$buttonClassProp = 'link-class';
$iconHtmlProp = 'link-html';
return self::updateItemData( $item, $buttonClassProp, $iconHtmlProp, $unsetIcon );