Robert Vogel has submitted this change and it was merged.

Change subject: cherry pick from REL1_22 Change-Id: 
I6759c48b91afb9ac74edebed2c1e42989fa89052
......................................................................


cherry pick from REL1_22 Change-Id: I6759c48b91afb9ac74edebed2c1e42989fa89052

PatchSet 2: Fixed whitespace error

Change-Id: I31ab69f0959d48878862b8ccad609bf4c5b91dc8
---
M Flexiskin/Flexiskin.class.php
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Robert Vogel: Verified; Looks good to me, approved



diff --git a/Flexiskin/Flexiskin.class.php b/Flexiskin/Flexiskin.class.php
index 54c3167..404f8b7 100644
--- a/Flexiskin/Flexiskin.class.php
+++ b/Flexiskin/Flexiskin.class.php
@@ -294,7 +294,7 @@
                $oStatus = BsFileSystemHelper::uploadFile(self::getVal('name'), 
"flexiskin" . DS . self::getVal('id') . DS . "images");
 
                if (!$oStatus->isGood())
-                       $aResult = json_encode(array('success' => false, 'msg' 
=> "err_cd:" . $aStatus['status']));
+                       $aResult = json_encode(array('success' => false, 'msg' 
=> "err_cd:" . $oStatus->getMessage()));
                else
                        $aResult = json_encode(array('success' => true, 'name' 
=> $oStatus->getValue()));
                $oAjaxResponse = new AjaxResponse( $aResult );
@@ -313,7 +313,8 @@
                        $aConfigs = json_decode($aConfigs);
                foreach ($aConfigs as $aConfig) {
                        $func = "Flexiskin::format_" . $aConfig->id;
-                       if (is_callable($func))
+                       $bReturn = wfRunHooks("BSFlexiskinGenerateStyleFile", 
array(&$func, &$aConfig));
+                       if ( $bReturn === true && is_callable($func))
                                $aFile[] = call_user_func($func, $aConfig);
                }
                return implode(" \n", $aFile);

-- 
To view, visit https://gerrit.wikimedia.org/r/200815
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I31ab69f0959d48878862b8ccad609bf4c5b91dc8
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: master
Gerrit-Owner: Tweichart <weich...@hallowelt.biz>
Gerrit-Reviewer: Mglaser <gla...@hallowelt.biz>
Gerrit-Reviewer: Pigpen <reym...@hallowelt.biz>
Gerrit-Reviewer: Robert Vogel <vo...@hallowelt.biz>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to