Brion VIBBER has submitted this change and it was merged.
Change subject: Handle page redirects properly
......................................................................
Handle page redirects properly
Bug: 63179
Change-Id: I2547de1b7139266b95bb267e4b7b9ac9d8dba8dc
---
M wikipedia/src/main/java/org/wikipedia/page/PageViewFragment.java
1 file changed, 6 insertions(+), 1 deletion(-)
Approvals:
Brion VIBBER: Verified; Looks good to me, approved
diff --git a/wikipedia/src/main/java/org/wikipedia/page/PageViewFragment.java
b/wikipedia/src/main/java/org/wikipedia/page/PageViewFragment.java
index 1fb8a69..ccea456 100644
--- a/wikipedia/src/main/java/org/wikipedia/page/PageViewFragment.java
+++ b/wikipedia/src/main/java/org/wikipedia/page/PageViewFragment.java
@@ -245,7 +245,12 @@
@Override
public List<Section> processResult(ApiResult result) throws Throwable {
Log.d("Wikipedia", result.asObject().toString(4));
- pageProperties = new
PageProperties(Utils.parseMWDate(result.asObject().optJSONObject("mobileview").optString("lastmodified")));
+ JSONObject mobileView =
result.asObject().optJSONObject("mobileview");
+ pageProperties = new
PageProperties(Utils.parseMWDate(mobileView.optString("lastmodified")));
+ if (mobileView.has("redirected")) {
+ // Handle redirects properly.
+ title = new PageTitle(mobileView.optString("redirected"),
title.getSite());
+ }
return super.processResult(result);
}
--
To view, visit https://gerrit.wikimedia.org/r/121321
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2547de1b7139266b95bb267e4b7b9ac9d8dba8dc
Gerrit-PatchSet: 3
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Yuvipanda <yuvipa...@gmail.com>
Gerrit-Reviewer: Brion VIBBER <br...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
