Jgoettlich has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/305262

Change subject: Found bug where the extension constructor is used
......................................................................

Found bug where the extension constructor is used

*Extension constructor was used for avatars so that the filesystemhelper
was not able to create a the directories
*was scanning the project and also found the same call in
ResponsibleEditors

Change-Id: I9373af10603c2aefeccbdf7be8264c465d11a720
---
M Avatars/Avatars.class.php
M ResponsibleEditors/ResponsibleEditors.class.php
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions 
refs/changes/62/305262/1

diff --git a/Avatars/Avatars.class.php b/Avatars/Avatars.class.php
index 7344bbc..1ad495b 100644
--- a/Avatars/Avatars.class.php
+++ b/Avatars/Avatars.class.php
@@ -230,7 +230,7 @@
                                        throw new MWException('FATAL: Avatar 
generator not found!');
                                        break;
                        }
-                       $oStatus = 
BsFileSystemHelper::saveToDataDirectory($sAvatarFileName, $sRawPNGAvatar, 
$this->mInfo[EXTINFO::NAME]);
+                       $oStatus = 
BsFileSystemHelper::saveToDataDirectory($sAvatarFileName, $sRawPNGAvatar, 
'avatars');
                        if (!$oStatus->isGood())
                                throw new MWException('FATAL: Avatar could not 
be saved!');
                        # found no way to regenerate thumbs. just delete thumb 
folder if it exists
diff --git a/ResponsibleEditors/ResponsibleEditors.class.php 
b/ResponsibleEditors/ResponsibleEditors.class.php
index b7c4828..5ce534e 100644
--- a/ResponsibleEditors/ResponsibleEditors.class.php
+++ b/ResponsibleEditors/ResponsibleEditors.class.php
@@ -655,7 +655,7 @@
                $sDispalyName = 
$this->mCore->getUserDisplayName($oFirstResponsibleEditor);
 
                $oResponsibleEditorsTopView->setKey('ResponsibleEditors-Top');
-               $oResponsibleEditorsTopView->setIconSrc( $wgScriptPath . 
'/extensions/BlueSpiceExtensions/' . $this->mInfo[EXTINFO::NAME] . 
'/resources/images/bs-infobar-responsibleeditor.png' );
+               $oResponsibleEditorsTopView->setIconSrc( $wgScriptPath . 
'/extensions/BlueSpiceExtensions/ResponsibleEditors/resources/images/bs-infobar-responsibleeditor.png'
 );
                $oResponsibleEditorsTopView->setIconAlt( wfMessage( 
'bs-responsibleeditors-statebartop-icon-alt' )->plain() );
                $oResponsibleEditorsTopView->setText($sDispalyName);
                $oResponsibleEditorsTopView->setTextLinkTitle($sDispalyName);

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9373af10603c2aefeccbdf7be8264c465d11a720
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: master
Gerrit-Owner: Jgoettlich <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to