jenkins-bot has submitted this change and it was merged.
Change subject: Don't try to log a boolean for page.revid
......................................................................
Don't try to log a boolean for page.revid
It won't work.
Change-Id: I9f1a34dcdada5cd33a212ed6cb01750ccfa7abfe
(cherry picked from commit 1f1ec7d4b8e74ba2dc9ed1d7407867c74867e3f5)
---
M WikiEditor.hooks.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Alex Monk: Looks good to me, approved
jenkins-bot: Verified
diff --git a/WikiEditor.hooks.php b/WikiEditor.hooks.php
index fca9406..a727621 100644
--- a/WikiEditor.hooks.php
+++ b/WikiEditor.hooks.php
@@ -167,7 +167,7 @@
'page.id' => $page->getId(),
'page.title' => $title->getPrefixedText(),
'page.ns' => $title->getNamespace(),
- 'page.revid' => $page->getRevision() &&
$page->getRevision()->getId(),
+ 'page.revid' => $page->getRevision() ?
$page->getRevision()->getId() : 0,
'user.id' => $user->getId(),
'user.editCount' => $user->getEditCount(),
'mediawiki.version' => $wgVersion
--
To view, visit https://gerrit.wikimedia.org/r/196727
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9f1a34dcdada5cd33a212ed6cb01750ccfa7abfe
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiEditor
Gerrit-Branch: wmf/1.25wmf21
Gerrit-Owner: Jforrester <[email protected]>
Gerrit-Reviewer: Alex Monk <[email protected]>
Gerrit-Reviewer: TheDJ <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
