jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/332913 )
Change subject: Add logging of pages with broken data
......................................................................
Add logging of pages with broken data
Bug: T155057
Change-Id: Ide9d9c363b767e69ccfc8e9f490d1c2176f10edf
---
M includes/ApiGraph.php
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
JGirault: Looks good to me, approved
jenkins-bot: Verified
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/332913
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ide9d9c363b767e69ccfc8e9f490d1c2176f10edf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Graph
Gerrit-Branch: master
Gerrit-Owner: MaxSem <maxsem.w...@gmail.com>
Gerrit-Reviewer: JGirault <jgira...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
