Pwirth has submitted this change and it was merged.
Change subject: BSApiFileBackendStore: Fixed SecureFileStore call
......................................................................
BSApiFileBackendStore: Fixed SecureFileStore call
Had to use html_entities_decode with the output of SecureFileStore,
because otherwise the URLs have not been valid
Change-Id: I363169daf9ceebe4be91336484be67dedf15d572
---
M includes/api/BSApiFileBackendStore.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Pwirth: Verified; Looks good to me, approved
diff --git a/includes/api/BSApiFileBackendStore.php
b/includes/api/BSApiFileBackendStore.php
index 214553f..e1c81f0 100644
--- a/includes/api/BSApiFileBackendStore.php
+++ b/includes/api/BSApiFileBackendStore.php
@@ -66,8 +66,8 @@
$sThumb = $oImg->createThumb( 48, 48 );
$sUrl = $oImg->getUrl();
if( $bUseSecureFileStore ) { //TODO: Remove
- $sThumb = SecureFileStore::secureStuff(
$sThumb, true );
- $sUrl = SecureFileStore::secureStuff( $sUrl,
true );
+ $sThumb = html_entity_decode(
SecureFileStore::secureStuff( $sThumb, true ) );
+ $sUrl = html_entity_decode(
SecureFileStore::secureStuff( $sUrl, true ) );
}
$aReturn[ $oRow->page_id ] = (object) array(
--
To view, visit https://gerrit.wikimedia.org/r/218879
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I363169daf9ceebe4be91336484be67dedf15d572
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceFoundation
Gerrit-Branch: master
Gerrit-Owner: Robert Vogel <[email protected]>
Gerrit-Reviewer: Mglaser <[email protected]>
Gerrit-Reviewer: Pwirth <[email protected]>
Gerrit-Reviewer: Tweichart <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
