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

Change subject: Fix page thumbnail caching, for real.
......................................................................


Fix page thumbnail caching, for real.

The PageTitle object in the "model" variable wasn't being updated
correctly after the PageLead interface constructs the Page.

Bug: T115412
Change-Id: I82b8ce1e1dfaaa1eab3d1b1a26e03c63a87b85a9
---
M app/src/main/java/org/wikipedia/page/JsonPageLoadStrategy.java
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Mholloway: Looks good to me, approved
  Niedzielski: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/app/src/main/java/org/wikipedia/page/JsonPageLoadStrategy.java 
b/app/src/main/java/org/wikipedia/page/JsonPageLoadStrategy.java
index 35d9201..8d25d86 100644
--- a/app/src/main/java/org/wikipedia/page/JsonPageLoadStrategy.java
+++ b/app/src/main/java/org/wikipedia/page/JsonPageLoadStrategy.java
@@ -666,9 +666,9 @@
             return;
         }
 
-        final PageTitle title = model.getTitle();
-        Page page = pageLead.toPage(title);
+        Page page = pageLead.toPage(model.getTitle());
         model.setPage(page);
+        model.setTitle(page.getTitle());
 
         editHandler.setPage(model.getPage());
 
@@ -691,7 +691,7 @@
         // Update our history entry, in case the Title was changed (i.e. 
normalized)
         final HistoryEntry curEntry = model.getCurEntry();
         model.setCurEntry(
-                new HistoryEntry(title, curEntry.getTimestamp(), 
curEntry.getSource()));
+                new HistoryEntry(model.getTitle(), curEntry.getTimestamp(), 
curEntry.getSource()));
 
         // Save history entry and page image url
         new SaveHistoryTask(model.getCurEntry(), app).execute();

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I82b8ce1e1dfaaa1eab3d1b1a26e03c63a87b85a9
Gerrit-PatchSet: 1
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Dbrant <dbr...@wikimedia.org>
Gerrit-Reviewer: BearND <bsitzm...@wikimedia.org>
Gerrit-Reviewer: Brion VIBBER <br...@wikimedia.org>
Gerrit-Reviewer: Mholloway <mhollo...@wikimedia.org>
Gerrit-Reviewer: Niedzielski <sniedziel...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to