Pwirth has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/372143 )
Change subject: BSFoundation: Moved entity classes to the ps4 autoloader dir ...................................................................... BSFoundation: Moved entity classes to the ps4 autoloader dir Change-Id: Idb7e29f796688dcc6716dca66524c06b3a12a829 --- M extension.json D includes/content/EntityContent.php D includes/content/EntityContentHandler.php A src/Content/Entity.php A src/Content/EntityHandler.php R src/Entity.php R src/EntityConfig.php R src/EntityRegistry.php 8 files changed, 228 insertions(+), 226 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceFoundation refs/changes/43/372143/1 diff --git a/extension.json b/extension.json index 6b6a86c..8affeca 100644 --- a/extension.json +++ b/extension.json @@ -572,11 +572,6 @@ "BsSTYLEMEDIA": "includes/Common.php", "EXTINFO": "includes/Common.php", "EXTTYPE": "includes/Common.php", - "BSEntity": "includes/Entity.php", - "BSEntityRegistry": "includes/EntityRegistry.php", - "BSEntityConfig": "includes/entityconfigs/EntityConfig.php", - "BSEntityContent": "includes/content/EntityContent.php", - "BSEntityContentHandler": "includes/content/EntityContentHandler.php", "BSApiTestCase": "tests/BSApiTestCase.php", "BSFixturesProvider": "tests/BSFixturesProvider.php", "BSPageFixturesProvider": "tests/BSPageFixturesProvider.php", diff --git a/includes/content/EntityContent.php b/includes/content/EntityContent.php deleted file mode 100644 index 6e317f4..0000000 --- a/includes/content/EntityContent.php +++ /dev/null @@ -1,108 +0,0 @@ -<?php - -abstract class BSEntityContent extends JsonContent { - - public function __construct( $text, $modelId = '' ) { - parent::__construct( $text, $modelId ); - } - - /** - * Decodes the JSON into a PHP associative array. - * @return array - */ - public function getJsonData() { - return FormatJson::decode( $this->getNativeData(), true ); - } - - /** - * @return bool Whether content is valid JSON. - */ - public function isValid() { - return $this->getJsonData() !== null; - } - - /** - * Pretty-print JSON - * - * @return bool|null|string - */ - public function beautifyJSON() { - $decoded = FormatJson::decode( $this->getNativeData(), true ); - if ( !is_array( $decoded ) ) { - return null; - } - return FormatJson::encode( $decoded, true ); - - } - - /** - * Beautifies JSON prior to save. - * @param Title $title Title - * @param User $user User - * @param ParserOptions $popts - * @return JsonContent - */ - public function preSaveTransform( Title $title, User $user, ParserOptions $popts ) { - return new static( $this->beautifyJSON() ); - } - - /** - * Set the HTML and add the appropriate styles - * - * - * @param Title $title - * @param int $revId - * @param ParserOptions $options - * @param bool $generateHtml - * @param ParserOutput $output - */ - protected function fillParserOutput( Title $title, $revId, - ParserOptions $options, $generateHtml, ParserOutput &$output - ) { - return parent::fillParserOutput( - $title, - $revId, - $options, - $generateHtml, - $output - ); - } - /** - * Constructs an HTML representation of a JSON object. - * @param array $mapping - * @return string HTML - */ - protected function objectTable( $mapping ) { - $rows = array(); - - foreach ( $mapping as $key => $val ) { - $rows[] = $this->objectRow( $key, $val ); - } - return Xml::tags( 'table', array( 'class' => 'mw-json' ), - Xml::tags( 'tbody', array(), join( "\n", $rows ) ) - ); - } - - /** - * Constructs HTML representation of a single key-value pair. - * @param string $key - * @param mixed $val - * @return string HTML. - */ - protected function objectRow( $key, $val ) { - $th = Xml::elementClean( 'th', array(), $key ); - if ( is_array( $val ) ) { - $td = Xml::tags( 'td', array(), self::objectTable( $val ) ); - } else { - if ( is_string( $val ) ) { - $val = '"' . $val . '"'; - } else { - $val = FormatJson::encode( $val ); - } - - $td = Xml::elementClean( 'td', array( 'class' => 'value' ), $val ); - } - - return Xml::tags( 'tr', array(), $th . $td ); - } -} \ No newline at end of file diff --git a/includes/content/EntityContentHandler.php b/includes/content/EntityContentHandler.php deleted file mode 100644 index ad86f3c..0000000 --- a/includes/content/EntityContentHandler.php +++ /dev/null @@ -1,15 +0,0 @@ -<?php - -abstract class BSEntityContentHandler extends JsonContentHandler { - - public function __construct( $modelId = '' ) { - parent::__construct( $modelId ); - } - - /** - * @return string - */ - protected function getContentClass() { - return 'BSEntityContent'; - } -} \ No newline at end of file diff --git a/src/Content/Entity.php b/src/Content/Entity.php new file mode 100644 index 0000000..ae5910e --- /dev/null +++ b/src/Content/Entity.php @@ -0,0 +1,109 @@ +<?php +namespace BlueSpice\Content; + +class Entity extends \JsonContent { + + public function __construct( $text, $modelId = '' ) { + parent::__construct( $text, $modelId ); + } + + /** + * Decodes the JSON into a PHP associative array. + * @return array + */ + public function getJsonData() { + return \FormatJson::decode( $this->getNativeData(), true ); + } + + /** + * @return bool Whether content is valid JSON. + */ + public function isValid() { + return $this->getJsonData() !== null; + } + + /** + * Pretty-print JSON + * + * @return bool|null|string + */ + public function beautifyJSON() { + $decoded = \FormatJson::decode( $this->getNativeData(), true ); + if ( !is_array( $decoded ) ) { + return null; + } + return \FormatJson::encode( $decoded, true ); + + } + + /** + * Beautifies JSON prior to save. + * @param \Title $title Title + * @param \User $user User + * @param ParserOptions $popts + * @return JsonContent + */ + public function preSaveTransform( \Title $title, \User $user, \ParserOptions $popts ) { + return new static( $this->beautifyJSON() ); + } + + /** + * Set the HTML and add the appropriate styles + * + * + * @param \Title $title + * @param int $revId + * @param \ParserOptions $options + * @param bool $generateHtml + * @param \ParserOutput $output + */ + protected function fillParserOutput( \Title $title, $revId, + \ParserOptions $options, $generateHtml, \ParserOutput &$output + ) { + return parent::fillParserOutput( + $title, + $revId, + $options, + $generateHtml, + $output + ); + } + /** + * Constructs an HTML representation of a JSON object. + * @param array $mapping + * @return string HTML + */ + protected function objectTable( $mapping ) { + $rows = array(); + + foreach ( $mapping as $key => $val ) { + $rows[] = $this->objectRow( $key, $val ); + } + return \Xml::tags( 'table', array( 'class' => 'mw-json' ), + \Xml::tags( 'tbody', array(), join( "\n", $rows ) ) + ); + } + + /** + * Constructs HTML representation of a single key-value pair. + * @param string $key + * @param mixed $val + * @return string HTML. + */ + protected function objectRow( $key, $val ) { + $th = \Xml::elementClean( 'th', array(), $key ); + if ( is_array( $val ) ) { + $td = \Xml::tags( 'td', array(), self::objectTable( $val ) ); + } else { + if ( is_string( $val ) ) { + $val = '"' . $val . '"'; + } else { + $val = \FormatJson::encode( $val ); + } + + $td = \Xml::elementClean( 'td', array( 'class' => 'value' ), $val ); + } + + return \Xml::tags( 'tr', array(), $th . $td ); + } +} \ No newline at end of file diff --git a/src/Content/EntityHandler.php b/src/Content/EntityHandler.php new file mode 100644 index 0000000..1cfd85e --- /dev/null +++ b/src/Content/EntityHandler.php @@ -0,0 +1,16 @@ +<?php +namespace BlueSpice\Content; + +abstract class EntityHandler extends \JsonContentHandler { + + public function __construct( $modelId = '' ) { + parent::__construct( $modelId ); + } + + /** + * @return string + */ + protected function getContentClass() { + return "\\BlueSpice\\Content\\Entity"; + } +} \ No newline at end of file diff --git a/includes/Entity.php b/src/Entity.php similarity index 72% rename from includes/Entity.php rename to src/Entity.php index df4bcb0..ddfa552 100644 --- a/includes/Entity.php +++ b/src/Entity.php @@ -1,6 +1,6 @@ <?php /** - * BSEntity base class for BlueSpice + * Entity base class for BlueSpice * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -26,13 +26,16 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU Public License v2 or later * @filesource */ -class BSEntity implements JsonSerializable { +namespace BlueSpice; +use BlueSpice\Content\Entity as EntityContent; + +abstract class Entity implements \JsonSerializable { const NS = ''; protected static $aEntitiesByID = array(); protected $bUnsavedChanges = true; /** - * @var BSEntityConfig + * @var EntityConfig */ protected $oConfig = null; @@ -41,7 +44,7 @@ protected $sType = ''; protected $bArchived = false; - protected function __construct( stdClass $oStdClass, BSEntityConfig $oConfig ) { + protected function __construct( \stdClass $oStdClass, EntityConfig $oConfig ) { $this->oConfig = $oConfig; if( !empty($oStdClass->id) ) { $this->iID = (int) $oStdClass->id; @@ -60,8 +63,8 @@ } protected static function factory( $sType, $oData ) { - $oConfig = BSEntityConfig::factory( $sType ); - if( !$oConfig instanceof BSEntityConfig ) { + $oConfig = EntityConfig::factory( $sType ); + if( !$oConfig instanceof EntityConfig ) { //TODO: Return a DummyEntity instead of null. return null; } @@ -72,7 +75,7 @@ } protected function generateID() { - //this is the case if the current BSEntity is new (no Title created yet) + //this is the case if the current Entity is new (no Title created yet) //Get the page_title of the last created title in entity namespace and //add +1. Entities are stored like: MYEntityNamespace:1, //MYEntityNamespace:2, MYEntityNamespace:3 @@ -98,11 +101,11 @@ } /** - * Get BSEntity by BSEntityContent Object, wrapper for newFromObject - * @param BSEntityContent $sContent - * @return BSEntity + * Get Entity by EntityContent Object, wrapper for newFromObject + * @param EntityContent $sContent + * @return Entity */ - public static function newFromContent( BSEntityContent $sContent ) { + public static function newFromContent( EntityContent $sContent ) { $aContent = $sContent->getJsonData(); if ( empty($aContent) ) { return null; @@ -112,9 +115,9 @@ } /** - * Get BSEntity by Json Object + * Get Entity by Json Object * @param Object $oObject - * @return BSEntity + * @return Entity */ public static function newFromObject( $oObject ) { if( !is_object($oObject) ) { @@ -122,10 +125,10 @@ } if( empty($oObject->type) ) { - //$oObject->type = BSEntityRegistry::getDefaultHandlerType(); + //$oObject->type = EntityRegistry::getDefaultHandlerType(); return null; } - if( !BSEntityRegistry::isRegisteredType($oObject->type) ) { + if( !EntityRegistry::isRegisteredType($oObject->type) ) { return null; } @@ -142,7 +145,7 @@ /** * Gets the related config object - * @return BSEntityConfig + * @return EntityConfig */ public function getConfig() { return $this->oConfig; @@ -150,10 +153,10 @@ /** * Gets the related Title object by ID - * @return Title + * @return \Title */ public static function getTitleFor( $iID ) { - return Title::newFromText( $iID, static::NS ); + return \Title::makeTitle( static::NS, $iID ); } /** @@ -173,10 +176,10 @@ } /** - * Get BSEntity from ID, wrapper for newFromTitle + * Get Entity from ID, wrapper for newFromTitle * @param int $iID * @param boolean $bForceReload - * @return BSEntity | null + * @return Entity | null */ public static function newFromID( $iID, $bForceReload = false ) { if ( !is_numeric( $iID ) ) { @@ -194,18 +197,18 @@ return null; } - $sText = BsPageContentProvider::getInstance()->getContentFromTitle( + $sText = \BsPageContentProvider::getInstance()->getContentFromTitle( $oTitle ); - $oBSEntityContent = new JsonContent( $sText ); - $oData = (object) $oBSEntityContent->getData()->getValue(); + $oEntityContent = new EntityContent( $sText ); + $oData = (object) $oEntityContent->getData()->getValue(); if( empty($oData->type) ) { return null; - //$oData->type = BSEntityRegistry::getDefaultHandlerType(); + //$oData->type = EntityRegistry::getDefaultHandlerType(); } - if( !BSEntityRegistry::isRegisteredType($oData->type) ) { + if( !EntityRegistry::isRegisteredType($oData->type) ) { return null; } @@ -217,12 +220,12 @@ } /** - * Main method for getting a BSEntity from a Title - * @param Title $oTitle + * Main method for getting a Entity from a Title + * @param \Title $oTitle * @param boolean $bForceReload - * @return BSEntity + * @return Entity */ - public static function newFromTitle( Title $oTitle, $bForceReload = false ) { + public static function newFromTitle( \Title $oTitle, $bForceReload = false ) { if ( is_null( $oTitle ) || $oTitle->getNamespace() !== static::NS ) { return null; } @@ -232,37 +235,37 @@ } /** - * Saves the current BSEntity - * @return Status + * Saves the current Entity + * @return \Status */ - public function save( User $oUser = null, $aOptions = array() ) { + public function save( \User $oUser = null, $aOptions = [] ) { if( !$oUser instanceof User ) { - return Status::newFatal( 'No User' ); + return \Status::newFatal( 'No User' ); } if( $this->exists() && !$this->hasUnsavedChanges() ) { - return Status::newGood( $this ); + return \Status::newGood( $this ); } if( empty($this->getID()) ) { $this->generateID(); } if( empty($this->getID()) ) { - return Status::newFatal( 'No ID generated' ); + return \Status::newFatal( 'No ID generated' ); } if( empty($this->getOwnerID()) ) { $this->setOwnerID( (int) $oUser->getId() ); } $sType = $this->getType(); if( empty($sType) ) { - return Status::newFatal('Related Type error'); + return \Status::newFatal( 'Related Type error' ); } $oTitle = $this->getTitle(); if ( is_null( $oTitle ) ) { - return Status::newFatal('Related Title error'); + return \Status::newFatal( 'Related Title error' ); } - $oWikiPage = WikiPage::factory( $oTitle ); - $sContentClass = $this->getConfig()->get('ContentClass'); + $oWikiPage = \WikiPage::factory( $oTitle ); + $sContentClass = $this->getConfig()->get( 'ContentClass' ); try { $oStatus = $oWikiPage->doEditContent( new $sContentClass( json_encode( $this ) ), @@ -272,9 +275,9 @@ $oUser, null ); - } catch( Exception $e ) { + } catch( \Exception $e ) { //Something probalby breaks json - return Status::newFatal( $e->getMessage() ); + return \Status::newFatal( $e->getMessage() ); } //TODO: check why this is not good if ( !$oStatus->isOK() ) { @@ -283,24 +286,24 @@ $this->setUnsavedChanges( false ); - wfRunHooks( 'BSEntitySaveComplete', array( $this, $oStatus, $oUser ) ); + \Hooks::run( 'BSEntitySaveComplete', [ $this, $oStatus, $oUser ] ); $this->invalidateCache(); return $oStatus; } /** - * Archives the current BSEntity - * @return Status + * Archives the current Entity + * @return \Status */ - public function delete( User $oUser = null, $aOptions = array() ) { + public function delete( \User $oUser = null, $aOptions = [] ) { $oTitle = $this->getTitle(); $oStatus = null; - $oWikiPage = WikiPage::factory( $oTitle ); + $oWikiPage = \WikiPage::factory( $oTitle ); - wfRunHooks( 'BSEntityDelete', array( $this, $oStatus, $oUser ) ); - if( $oStatus instanceof Status && $oStatus->isOK() ) { + \Hooks::run( 'BSEntityDelete', [ $this, $oStatus, $oUser ] ); + if( $oStatus instanceof \Status && $oStatus->isOK() ) { return $oStatus; } $this->bArchived = true; @@ -308,11 +311,11 @@ try { $oStatus = $this->save(); - } catch( Exception $e ) { - return Status::newFatal( $e->getMessage() ); + } catch( \Exception $e ) { + return \Status::newFatal( $e->getMessage() ); } - wfRunHooks( 'BSEntityDeleteComplete', array( $this, $oStatus ) ); + \Hooks::run( 'BSEntityDeleteComplete', [ $this, $oStatus ] ); if( !$oStatus->isOK() ) { return $oStatus; } @@ -322,8 +325,8 @@ } /** - * Gets the BSEntity attributes formated for the api - * @return object + * Gets the Entity attributes formated for the api + * @return array */ public function getFullData( $aData = array() ) { $aData = array_merge( @@ -335,7 +338,7 @@ 'archived' => $this->isArchived(), ) ); - Hooks::run('BSEntityGetFullData', [ + \Hooks::run('BSEntityGetFullData', [ $this, &$aData ]); @@ -343,12 +346,12 @@ } /** - * Adds a BSEntity to the cache - * @param BSEntity $oInstance - * @return BSEntity + * Adds a Entity to the cache + * @param Entity $oInstance + * @return Entity */ - protected static function appendCache( BSEntity &$oInstance ) { - if( static::hasCacheEntry($oInstance->getID()) ) { + protected static function appendCache( Entity &$oInstance ) { + if( static::hasCacheEntry( $oInstance->getID() ) ) { return $oInstance; } static::$aEntitiesByID[$oInstance->getID()] = $oInstance; @@ -356,11 +359,11 @@ } /** - * Removes a BSEntity from the cache if it's in - * @param BSEntity $oInstance - * @return BSEntity + * Removes a Entity from the cache if it's in + * @param Entity $oInstance + * @return Entity */ - protected static function detachCache( BSEntity &$oInstance ) { + protected static function detachCache( Entity &$oInstance ) { if( !static::hasCacheEntry($oInstance->getID()) ) { return $oInstance; } @@ -369,9 +372,9 @@ } /** - * Gets a instance of the BSEntity from the cache by ID + * Gets a instance of the Entity from the cache by ID * @param int $iID - * @return BSEntity + * @return Entity */ protected static function getInstanceFromCacheByID( $iID ) { if( !static::hasCacheEntry( $iID ) ) { @@ -385,7 +388,7 @@ } /** - * Checks, if the current BSEntity exists in the Wiki + * Checks, if the current Entity exists in the Wiki * @return boolean */ public function exists() { @@ -416,7 +419,7 @@ } /** - * Returns the current id for the BSEntity + * Returns the current id for the Entity * @return int */ public function getID() { @@ -424,7 +427,7 @@ } /** - * Returns the current user id for the BSEntity + * Returns the current user id for the Entity * @return int */ public function getOwnerID() { @@ -440,9 +443,9 @@ } /** - * Sets the current BSEntity to an unsaved changes mode, refreshes cache + * Sets the current Entity to an unsaved changes mode, refreshes cache * @param String $bStatus - * @return BSEntity + * @return Entity */ public function setUnsavedChanges( $bStatus = true ) { $this->bUnsavedChanges = (bool) $bStatus; @@ -452,7 +455,7 @@ /** * Sets the current user ID * @param int - * @return BSEntity + * @return Entity */ public function setOwnerID( $iOwnerID ) { return $this->setUnsavedChanges( @@ -461,7 +464,7 @@ } /** - * Subclass needs to return the current BSEntity as a Json encoded + * Subclass needs to return the current Entity as a Json encoded * string * @deprecated since 2.27.0 - Use json_encode( $oInstance ) instead * @return stdObject - Subclass needs to return encoded string! @@ -479,13 +482,13 @@ } /** - * @param stdClass $oData + * @param \stdClass $oData */ - public function setValuesByObject( stdClass $oData ) { + public function setValuesByObject( \stdClass $oData ) { if( isset($oData->ownerid) ) { $this->setOwnerID( $oData->ownerid ); } - Hooks::run('BSEntitySetValuesByObject', [ + \Hooks::run('BSEntitySetValuesByObject', [ $this, $oData ]); @@ -493,10 +496,10 @@ /** * Checks if the given User is the owner of this entity - * @param User $oUser + * @param \User $oUser * @return boolean */ - public function userIsOwner( User $oUser ) { + public function userIsOwner( \User $oUser ) { if( $oUser->isAnon() || $this->getOwnerID() < 1) { return false; } @@ -505,12 +508,12 @@ /** * Invalidated the cache - * @return BSEntity + * @return Entity */ public function invalidateCache() { $this->invalidateTitleCache( wfTimestampNow() ); static::detachCache( $this ); - Hooks::run( 'BSEntityInvalidate', array( $this ) ); + \Hooks::run( 'BSEntityInvalidate', [ $this ] ); return $this; } diff --git a/includes/entityconfigs/EntityConfig.php b/src/EntityConfig.php similarity index 84% rename from includes/entityconfigs/EntityConfig.php rename to src/EntityConfig.php index 5d86140..99d33e9 100644 --- a/includes/entityconfigs/EntityConfig.php +++ b/src/EntityConfig.php @@ -1,7 +1,7 @@ <?php /** - * BSEntityConfig class for BlueSpice + * EntityConfig class for BlueSpice * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -26,30 +26,31 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU Public License v2 or later * @filesource */ +namespace BlueSpice; /** - * BSEntityConfig class for BlueSpice + * EntityConfig class for BlueSpice * @package BlueSpiceFoundation */ -abstract class BSEntityConfig { +abstract class EntityConfig implements \JsonSerializable { protected static $aEntityConfigs = null; protected static $aDefaults = array(); protected $sType = ''; /** - * BSEntityConfig factory + * EntityConfig factory * @param string $sType - Entity type - * @return BSEntityConfig - or null + * @return EntityConfig - or null */ public static function factory( $sType ) { - if( !is_null(static::$aEntityConfigs) ) { - if( !isset(static::$aEntityConfigs[$sType]) ) { + if( !is_null( static::$aEntityConfigs ) ) { + if( !isset( static::$aEntityConfigs[$sType] ) ) { return null; } return static::$aEntityConfigs[$sType]; } //TODO: Check params and classes - $aRegisteredEntities = BSEntityRegistry::getRegisteredEntities(); + $aRegisteredEntities = EntityRegistry::getRegisteredEntities(); foreach( $aRegisteredEntities as $sKey => $sConfigClass ) { static::$aEntityConfigs[$sKey] = new $sConfigClass(); static::$aEntityConfigs[$sKey]->sType = $sKey; @@ -58,13 +59,13 @@ static::$aEntityConfigs[$sKey]->addGetterDefaults() ); } - Hooks::run( 'BSEntityConfigDefaults', array( &static::$aDefaults ) ); + \Hooks::run( 'BSEntityConfigDefaults', [ &static::$aDefaults ] ); return static::$aEntityConfigs[$sType]; } protected static function getDefault( $sOption ) { - if( !isset(static::$aDefaults[$sOption]) ) { + if( !isset( static::$aDefaults[$sOption] ) ) { return false; } return static::$aDefaults[$sOption]; @@ -115,6 +116,6 @@ abstract protected function get_EntityClass(); protected function get_ContentClass() { - return 'BSEntityContent'; + return "\\BlueSpice\\Content\\Entity"; } } diff --git a/includes/EntityRegistry.php b/src/EntityRegistry.php similarity index 89% rename from includes/EntityRegistry.php rename to src/EntityRegistry.php index dc78a6c..b3cce5d 100644 --- a/includes/EntityRegistry.php +++ b/src/EntityRegistry.php @@ -1,7 +1,7 @@ <?php /** - * BSEntityRegistry class for BlueSpice + * EntityRegistry class for BlueSpice * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -26,12 +26,13 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU Public License v2 or later * @filesource */ +namespace BlueSpice; /** - * BSEntityRegistry class for BSSocial extension + * EntityRegistry class for BlueSpice * @package BlueSpiceFoundation */ -class BSEntityRegistry { +class EntityRegistry { private function __construct() {} private static $bEntitiesRegistered = false; private static $aEntities = array(); @@ -41,9 +42,9 @@ return true; } - $b = wfRunHooks( 'BSEntityRegister', array( + $b = \Hooks::run( 'BSEntityRegister', [ &self::$aEntities, - )); + ]); return $b ? static::$bEntitiesRegistered = true : $b; } @@ -54,7 +55,7 @@ */ public static function getRegisteredEntities() { if( !self::runRegister() ) { - return array(); + return []; } return self::$aEntities; } @@ -77,8 +78,8 @@ * @return array */ public static function getRegisteredEntityByType( $sType ) { - if( !self::isRegisteredType($sType) ) { - return array(); + if( !self::isRegisteredType( $sType ) ) { + return []; } return self::$aEntities[$sType]; } -- To view, visit https://gerrit.wikimedia.org/r/372143 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Idb7e29f796688dcc6716dca66524c06b3a12a829 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation Gerrit-Branch: master Gerrit-Owner: Pwirth <wi...@hallowelt.biz> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits