MaxSem has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/332916 )

Change subject: Add logging of pages with broken data
......................................................................

Add logging of pages with broken data

Bug: T155057
Change-Id: Ide9d9c363b767e69ccfc8e9f490d1c2176f10edf
(cherry picked from commit 25b65d83ec04b0e9057ab4ff21eb9ec1849edf3c)
---
M includes/ApiGraph.php
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Graph 
refs/changes/16/332916/1

diff --git a/includes/ApiGraph.php b/includes/ApiGraph.php
index a0bdc2d..f2f387e 100644
--- a/includes/ApiGraph.php
+++ b/includes/ApiGraph.php
@@ -160,6 +160,9 @@
                                $st = FormatJson::parse( $ppValue );
                                if ( $st->isOK() ) {
                                        $allGraphs = $st->getValue();
+                                       if ( !is_object( $allGraphs ) ) {
+                                               wfDebugLog( 'AdHocDebug', 
"ApiGraph: invalid data on page '$titleText'" );
+                                       }
                                        if ( is_object( $allGraphs ) && 
property_exists( $allGraphs, $hash ) ) {
                                                $graph = $allGraphs->$hash;
                                        }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide9d9c363b767e69ccfc8e9f490d1c2176f10edf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Graph
Gerrit-Branch: wmf/1.29.0-wmf.8
Gerrit-Owner: MaxSem <maxsem.w...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to