jenkins-bot has submitted this change and it was merged.

Change subject: (bug 47179) Page Link web or email bundling always show 1 other
......................................................................


(bug 47179) Page Link web or email bundling always show 1 other

This would affect both web and email bundling

Change-Id: If21acb1189ff9de95670bc5519be0700354bb0e7
---
M formatters/PageLinkFormatter.php
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Lwelling: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/formatters/PageLinkFormatter.php b/formatters/PageLinkFormatter.php
index 3145b23..25c5b46 100644
--- a/formatters/PageLinkFormatter.php
+++ b/formatters/PageLinkFormatter.php
@@ -33,7 +33,11 @@
 
                $count = 1;
                foreach ( $data as $row ) {
-                       $extra = $row->event_extra;
+                       $extra = $row->event_extra ? unserialize( 
$row->event_extra ) : null;
+                       if ( !$extra ) {
+                               continue;
+                       }
+
                        if ( $this->isTitleSet( $extra ) ) {
                                $key = $this->getTitleHash( $extra );
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If21acb1189ff9de95670bc5519be0700354bb0e7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>
Gerrit-Reviewer: Kaldari <[email protected]>
Gerrit-Reviewer: Lwelling <[email protected]>
Gerrit-Reviewer: Matthias Mullie <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to