jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/400688 )
Change subject: Fix twitter count grep
......................................................................
Fix twitter count grep
Bug: T165463
Change-Id: I16199d6841c0423d48fa0832cf1a613130dc3b68
---
M src/wikidata/social/twitter.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Thiemo Kreuz (WMDE): Looks good to me, approved
diff --git a/src/wikidata/social/twitter.php b/src/wikidata/social/twitter.php
index 6879d0a..29e3a07 100755
--- a/src/wikidata/social/twitter.php
+++ b/src/wikidata/social/twitter.php
@@ -38,7 +38,7 @@
if( $nodes->length !== 1 ) {
return null;
}
- return str_replace( ',', '', $nodes->item(0)->textContent );
+ return str_replace( ',', '',
$nodes->item(0)->getAttribute('data-count') );
}
}
--
To view, visit https://gerrit.wikimedia.org/r/400688
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I16199d6841c0423d48fa0832cf1a613130dc3b68
Gerrit-PatchSet: 1
Gerrit-Project: analytics/wmde/scripts
Gerrit-Branch: master
Gerrit-Owner: Ladsgroup <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: Thiemo Kreuz (WMDE) <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
