Daniel Kinzler has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/387063 )
Change subject: Warn when page_latest is 0 unexpectedly.
......................................................................
Warn when page_latest is 0 unexpectedly.
Change-Id: I5aec379f2df0d39aad7c90a7ae2ea69abb7b4210
---
M includes/Title.php
M includes/cache/LinkCache.php
2 files changed, 11 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/63/387063/1
diff --git a/includes/Title.php b/includes/Title.php
index 718239d..22b98b3 100644
--- a/includes/Title.php
+++ b/includes/Title.php
@@ -481,6 +481,10 @@
}
if ( isset( $row->page_latest ) ) {
$this->mLatestID = (int)$row->page_latest;
+ if ( !$this->mLatestID ) {
+ wfWarn( 'Initializing Title from page
row with page_latest = 0!' );
+ $this->mLatestID = false;
+ }
}
if ( !$this->mForcedContentModel && isset(
$row->page_content_model ) ) {
$this->mContentModel = strval(
$row->page_content_model );
@@ -3431,6 +3435,7 @@
$cached = $linkCache->getGoodLinkFieldObj( $this, 'revision' );
if ( $cached === null ) {
# Trust LinkCache's state over our own, as for
isRedirect()
+ // XXX: Shouldn't we set $this->mArticleID = 0 here? Or
call resetArticleID()?
$this->mLatestID = 0;
return $this->mLatestID;
}
diff --git a/includes/cache/LinkCache.php b/includes/cache/LinkCache.php
index 2d08895..1a87c01 100644
--- a/includes/cache/LinkCache.php
+++ b/includes/cache/LinkCache.php
@@ -271,8 +271,12 @@
}
if ( $row ) {
- $this->addGoodLinkObjFromRow( $nt, $row );
- $id = intval( $row->page_id );
+ if ( intval( $row->page_latest ) ) {
+ $this->addGoodLinkObjFromRow( $nt, $row );
+ $id = intval( $row->page_id );
+ } else {
+ wfWarn( 'Attempt to cache incomplete page row
for page ID ' . $row->page_id . '!' );
+ }
} else {
$this->addBadLinkObj( $nt );
$id = 0;
--
To view, visit https://gerrit.wikimedia.org/r/387063
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5aec379f2df0d39aad7c90a7ae2ea69abb7b4210
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Daniel Kinzler <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
