[MediaWiki-commits] [Gerrit] mediawiki...BlueSpiceFoundation[master]: BSFoundation: Replaced all wfRunHooks with Hooks::run

2017-08-17 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/372404 )

Change subject: BSFoundation: Replaced all wfRunHooks with Hooks::run
..


BSFoundation: Replaced all wfRunHooks with Hooks::run

Change-Id: I9617e0ce162ae792aecd84bde35e8eac20e7c2f8
---
M includes/Config.class.php
M includes/Core.class.php
M includes/CoreHooks.php
M includes/Entity.php
M includes/EntityRegistry.php
M includes/api/BSApiExtJSStoreBase.php
M includes/api/BSApiPingTasks.php
M includes/outputhandler/views/view.UserMiniProfile.php
M includes/skins/BsBaseTemplate.php
M includes/utility/WidgetListHelper.class.php
M maintenance/DeployJavaWebApps.php
11 files changed, 20 insertions(+), 20 deletions(-)

Approvals:
  Robert Vogel: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/Config.class.php b/includes/Config.class.php
index 2486bd6..f87a727 100644
--- a/includes/Config.class.php
+++ b/includes/Config.class.php
@@ -183,7 +183,7 @@
wfProfileIn ( 'BS::Core::ConfigGet' );
 
$mReturn = null;
-   if( !wfRunHooks ( "BSCoreConfigGet", array ( $sPath, &$mReturn 
) ) ) {
+   if( !Hooks::run ( "BSCoreConfigGet", array ( $sPath, &$mReturn 
) ) ) {
wfProfileOut ( 'BS::Core::ConfigGet' );
return $mReturn;
}
@@ -331,12 +331,12 @@
);
}
 
-   wfRunHooks( 'BsSettingsBeforeSaveSettings', array( &$aSettings 
) );
+   Hooks::run( 'BsSettingsBeforeSaveSettings', array( &$aSettings 
) );
 
# write the settings array to the database
$bReturn = $dbw->insert('bs_settings', $aSettings);
 
-   wfRunHooks( 'BsSettingsAfterSaveSettings', array( $aSettings ) 
);
+   Hooks::run( 'BsSettingsAfterSaveSettings', array( $aSettings ) 
);
 
BsCacheHelper::invalidateCache(
BsCacheHelper::getCacheKey( 'BlueSpice', 'BsConfig', 
'settings' )
diff --git a/includes/Core.class.php b/includes/Core.class.php
index 8f83052..de26310 100644
--- a/includes/Core.class.php
+++ b/includes/Core.class.php
@@ -519,7 +519,7 @@
$oUserMiniProfileView->setOptions( $aParams );
$oUserMiniProfileView->setOption( 'user', $oUser );
 
-   wfRunHooks( 'BSCoreGetUserMiniProfileBeforeInit', array( 
&$oUserMiniProfileView, &$oUser, &$aParams ) );
+   Hooks::run( 'BSCoreGetUserMiniProfileBeforeInit', array( 
&$oUserMiniProfileView, &$oUser, &$aParams ) );
 
$oUserMiniProfileView->init();
 
diff --git a/includes/CoreHooks.php b/includes/CoreHooks.php
index e53c7ac..da40d81 100755
--- a/includes/CoreHooks.php
+++ b/includes/CoreHooks.php
@@ -203,7 +203,7 @@
$oUser = $out->getUser();
 
$aScriptSettings = BsConfig::getScriptSettings();
-   wfRunHooks('BsFoundationBeforeMakeGlobalVariablesScript', 
array( $oUser, &$aScriptSettings ) );
+   Hooks::run('BsFoundationBeforeMakeGlobalVariablesScript', 
array( $oUser, &$aScriptSettings ) );
 
foreach ( $aScriptSettings as $oVar ) {
$mValue = $oVar->getValue();
@@ -535,7 +535,7 @@
$oTitle = $skin->getTitle();
 
$aSettingViews = array();
-   wfRunHooks( 'BS:UserPageSettings', array( $oUser, $oTitle, 
&$aSettingViews ) );
+   Hooks::run( 'BS:UserPageSettings', array( $oUser, $oTitle, 
&$aSettingViews ) );
 
$oUserPageSettingsView = new ViewBaseElement();
$oUserPageSettingsView->setAutoWrap(
diff --git a/includes/Entity.php b/includes/Entity.php
index df4bcb0..c5e816d 100644
--- a/includes/Entity.php
+++ b/includes/Entity.php
@@ -283,7 +283,7 @@
 
$this->setUnsavedChanges( false );
 
-   wfRunHooks( 'BSEntitySaveComplete', array( $this, $oStatus, 
$oUser ) );
+   Hooks::run( 'BSEntitySaveComplete', array( $this, $oStatus, 
$oUser ) );
 
$this->invalidateCache();
return $oStatus;
@@ -299,7 +299,7 @@
$oStatus = null;
$oWikiPage = WikiPage::factory( $oTitle );
 
-   wfRunHooks( 'BSEntityDelete', array( $this, $oStatus, $oUser ) 
);
+   Hooks::run( 'BSEntityDelete', array( $this, $oStatus, $oUser ) 
);
if( $oStatus instanceof Status && $oStatus->isOK() ) {
return $oStatus;
}
@@ -312,7 +312,7 @@
return Status::newFatal( $e->getMessage() );
}
 
-   wfRunHooks( 'BSEntityDeleteComplete', array( $this, $oStatus ) 
);
+   Hooks::run( 'BSEntityDeleteComplete', array( $this, $oStatus ) 
);
if( !$oStatus->isOK() ) {
return $oStatus;
  

[MediaWiki-commits] [Gerrit] mediawiki...BlueSpiceFoundation[master]: BSFoundation: Replaced all wfRunHooks with Hooks::run

2017-08-17 Thread Pwirth (Code Review)
Pwirth has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/372404 )

Change subject: BSFoundation: Replaced all wfRunHooks with Hooks::run
..

BSFoundation: Replaced all wfRunHooks with Hooks::run

Change-Id: I9617e0ce162ae792aecd84bde35e8eac20e7c2f8
---
M includes/Config.class.php
M includes/Core.class.php
M includes/CoreHooks.php
M includes/Entity.php
M includes/EntityRegistry.php
M includes/api/BSApiExtJSStoreBase.php
M includes/api/BSApiPingTasks.php
M includes/outputhandler/views/view.UserMiniProfile.php
M includes/skins/BsBaseTemplate.php
M includes/utility/WidgetListHelper.class.php
M maintenance/DeployJavaWebApps.php
11 files changed, 20 insertions(+), 20 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceFoundation 
refs/changes/04/372404/1

diff --git a/includes/Config.class.php b/includes/Config.class.php
index 2486bd6..f87a727 100644
--- a/includes/Config.class.php
+++ b/includes/Config.class.php
@@ -183,7 +183,7 @@
wfProfileIn ( 'BS::Core::ConfigGet' );
 
$mReturn = null;
-   if( !wfRunHooks ( "BSCoreConfigGet", array ( $sPath, &$mReturn 
) ) ) {
+   if( !Hooks::run ( "BSCoreConfigGet", array ( $sPath, &$mReturn 
) ) ) {
wfProfileOut ( 'BS::Core::ConfigGet' );
return $mReturn;
}
@@ -331,12 +331,12 @@
);
}
 
-   wfRunHooks( 'BsSettingsBeforeSaveSettings', array( &$aSettings 
) );
+   Hooks::run( 'BsSettingsBeforeSaveSettings', array( &$aSettings 
) );
 
# write the settings array to the database
$bReturn = $dbw->insert('bs_settings', $aSettings);
 
-   wfRunHooks( 'BsSettingsAfterSaveSettings', array( $aSettings ) 
);
+   Hooks::run( 'BsSettingsAfterSaveSettings', array( $aSettings ) 
);
 
BsCacheHelper::invalidateCache(
BsCacheHelper::getCacheKey( 'BlueSpice', 'BsConfig', 
'settings' )
diff --git a/includes/Core.class.php b/includes/Core.class.php
index 8f83052..de26310 100644
--- a/includes/Core.class.php
+++ b/includes/Core.class.php
@@ -519,7 +519,7 @@
$oUserMiniProfileView->setOptions( $aParams );
$oUserMiniProfileView->setOption( 'user', $oUser );
 
-   wfRunHooks( 'BSCoreGetUserMiniProfileBeforeInit', array( 
&$oUserMiniProfileView, &$oUser, &$aParams ) );
+   Hooks::run( 'BSCoreGetUserMiniProfileBeforeInit', array( 
&$oUserMiniProfileView, &$oUser, &$aParams ) );
 
$oUserMiniProfileView->init();
 
diff --git a/includes/CoreHooks.php b/includes/CoreHooks.php
index e53c7ac..da40d81 100755
--- a/includes/CoreHooks.php
+++ b/includes/CoreHooks.php
@@ -203,7 +203,7 @@
$oUser = $out->getUser();
 
$aScriptSettings = BsConfig::getScriptSettings();
-   wfRunHooks('BsFoundationBeforeMakeGlobalVariablesScript', 
array( $oUser, &$aScriptSettings ) );
+   Hooks::run('BsFoundationBeforeMakeGlobalVariablesScript', 
array( $oUser, &$aScriptSettings ) );
 
foreach ( $aScriptSettings as $oVar ) {
$mValue = $oVar->getValue();
@@ -535,7 +535,7 @@
$oTitle = $skin->getTitle();
 
$aSettingViews = array();
-   wfRunHooks( 'BS:UserPageSettings', array( $oUser, $oTitle, 
&$aSettingViews ) );
+   Hooks::run( 'BS:UserPageSettings', array( $oUser, $oTitle, 
&$aSettingViews ) );
 
$oUserPageSettingsView = new ViewBaseElement();
$oUserPageSettingsView->setAutoWrap(
diff --git a/includes/Entity.php b/includes/Entity.php
index df4bcb0..c5e816d 100644
--- a/includes/Entity.php
+++ b/includes/Entity.php
@@ -283,7 +283,7 @@
 
$this->setUnsavedChanges( false );
 
-   wfRunHooks( 'BSEntitySaveComplete', array( $this, $oStatus, 
$oUser ) );
+   Hooks::run( 'BSEntitySaveComplete', array( $this, $oStatus, 
$oUser ) );
 
$this->invalidateCache();
return $oStatus;
@@ -299,7 +299,7 @@
$oStatus = null;
$oWikiPage = WikiPage::factory( $oTitle );
 
-   wfRunHooks( 'BSEntityDelete', array( $this, $oStatus, $oUser ) 
);
+   Hooks::run( 'BSEntityDelete', array( $this, $oStatus, $oUser ) 
);
if( $oStatus instanceof Status && $oStatus->isOK() ) {
return $oStatus;
}
@@ -312,7 +312,7 @@
return Status::newFatal( $e->getMessage() );
}
 
-   wfRunHooks( 'BSEntityDeleteComplete', array( $this, $oStatus ) 
);
+   Hooks::run( 'BSEntityDeleteComplete', array( $this, $oStatus ) 
);
if( !$oStatus->isOK() ) {
return