Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/395115 )
Change subject: Better undefined index protection
......................................................................
Better undefined index protection
* CollectionAPIResult::get has better error protection
then directly accessing $response
* A CollectionAPIResult may error, so check this before trying
to access information from it.
Bug: T182026
Change-Id: I0bbb8a00d010b8892cd12ec030c8358e42fd2e46
---
M Collection.body.php
1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Collection
refs/changes/15/395115/1
diff --git a/Collection.body.php b/Collection.body.php
index 6321fc7..452f41c 100644
--- a/Collection.body.php
+++ b/Collection.body.php
@@ -1054,7 +1054,7 @@
$api = CollectionRenderingAPI::instance( $writer );
$response = $api->render( $collection );
- if ( !$this->handleResult( $response ) ) {
+ if ( !$this->handleResult( $response ) || $response->isError()
) {
return;
}
@@ -1078,15 +1078,15 @@
$api = CollectionRenderingAPI::instance( $writer );
$response = $api->forceRender( $collectionID );
- if ( !$response ) {
+ if ( !$response || $response->isError() ) {
return;
}
$query = 'bookcmd=rendering'
. '&return_to=' . urlencode( $request->getVal(
'return_to', '' ) )
- . '&collection_id=' . urlencode(
$response->response['collection_id'] )
- . '&writer=' . urlencode( $response->response['writer']
);
- if ( isset( $response->response['is_cached'] ) &&
$response->response['is_cached'] ) {
+ . '&collection_id=' . urlencode( $response->get(
'collection_id' ) )
+ . '&writer=' . urlencode( $response->get( 'writer' ) );
+ if ( $response->get( 'is_cached' ) ) {
$query .= '&is_cached=1';
}
$this->getOutput()->redirect( SkinTemplate::makeSpecialUrl(
'Book', $query ) );
--
To view, visit https://gerrit.wikimedia.org/r/395115
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0bbb8a00d010b8892cd12ec030c8358e42fd2e46
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Collection
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
