Mglaser has uploaded a new change for review. https://gerrit.wikimedia.org/r/253888
Change subject: Fixed deletion of skin ...................................................................... Fixed deletion of skin When deleting an active flexiskin, the active skin was not reset to standard. The reason is a wrong variable name ($iId instead of $sId). Fixed now. Change-Id: Id2c02b34e8752507df15829d1201c12e25cf232e --- M Flexiskin/includes/FlexiskinApi.class.php 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions refs/changes/88/253888/1 diff --git a/Flexiskin/includes/FlexiskinApi.class.php b/Flexiskin/includes/FlexiskinApi.class.php index a137a4b..5681667 100644 --- a/Flexiskin/includes/FlexiskinApi.class.php +++ b/Flexiskin/includes/FlexiskinApi.class.php @@ -182,8 +182,8 @@ return FormatJson::encode( array( 'success' => false, 'msg' => wfMessage( 'bs-flexiskin-api-error-missing-param', 'id' )->plain() ) ); } - $oStatus = BsFileSystemHelper::deleteFolder( "flexiskin" . DS . $iId ); - if ( BsConfig::get( "MW::Flexiskin::Active" ) == $iId ) { + $oStatus = BsFileSystemHelper::deleteFolder( "flexiskin" . DS . $sId ); + if ( BsConfig::get( "MW::Flexiskin::Active" ) == $sId ) { BsConfig::set( "MW::Flexiskin::Active", "" ); BsConfig::saveSettings(); } -- To view, visit https://gerrit.wikimedia.org/r/253888 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id2c02b34e8752507df15829d1201c12e25cf232e Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions Gerrit-Branch: master Gerrit-Owner: Mglaser <gla...@hallowelt.biz> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits