Mattflaschen has uploaded a new change for review.

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

Change subject: Exception fix to Converter
......................................................................

Exception fix to Converter

* Don't output error for normal 'end of list' cases.

Change-Id: I803458b2383a0ac7f091a0a8d3417ff444a3be4c
---
M includes/Import/LiquidThreadsApi/Source.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow 
refs/changes/39/313639/1

diff --git a/includes/Import/LiquidThreadsApi/Source.php 
b/includes/Import/LiquidThreadsApi/Source.php
index 53d2d5a..c4e70c5 100644
--- a/includes/Import/LiquidThreadsApi/Source.php
+++ b/includes/Import/LiquidThreadsApi/Source.php
@@ -316,12 +316,12 @@
                $data = $this->apiCall( $conditions );
 
                if ( ! isset( $data['query'] ) ) {
-                       $this->logger->error( __METHOD__ . ': Failed API call 
against ' . $this->getKey() . ' with conditions : ' . json_encode( $conditions 
) );
                        if ( $this->isNotFoundError( $data ) ) {
                                $message = "Did not find pages: " . 
json_encode( $conditions );
                                $this->logger->debug( __METHOD__ . ": $message" 
);
                                throw new ApiNotFoundException( $message );
                        } else {
+                               $this->logger->error( __METHOD__ . ': Failed 
API call against ' . $this->getKey() . ' with conditions : ' . json_encode( 
$conditions ) );
                                throw new ImportException( "Null response from 
API module: " . json_encode( $data ) );
                        }
                } elseif ( !$expectContinue && isset( $data['continue'] ) ) {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I803458b2383a0ac7f091a0a8d3417ff444a3be4c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <[email protected]>

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

Reply via email to