Niedzielski has uploaded a new change for review.
https://gerrit.wikimedia.org/r/288056
Change subject: Hygiene: consolidate URL usage
......................................................................
Hygiene: consolidate URL usage
Rename new Wikipedia URL constant and replace duplicate string in Prefs.
No functional changes intended.
Change-Id: I794d7e37ed415d2f6d505c255e1f76882fc9dfc2
---
M app/src/main/java/org/wikipedia/Constants.java
M
app/src/main/java/org/wikipedia/createaccount/authmanager/AMCreateAccountTask.java
M app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java
M app/src/main/java/org/wikipedia/settings/Prefs.java
4 files changed, 6 insertions(+), 5 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia
refs/changes/56/288056/1
diff --git a/app/src/main/java/org/wikipedia/Constants.java
b/app/src/main/java/org/wikipedia/Constants.java
index 92a39b4..99605ae 100644
--- a/app/src/main/java/org/wikipedia/Constants.java
+++ b/app/src/main/java/org/wikipedia/Constants.java
@@ -8,7 +8,7 @@
public static final int RECENT_SEARCHES_FRAGMENT_LOADER_ID = 102;
public static final int USER_OPTION_ROW_FRAGMENT_LOADER_ID = 103;
- public static final String DEFAULT_AUTH_MANAGER_RETURN_URL =
"https://www.wikipedia.org";;
+ public static final String WIKIPEDIA_URL = "https://www.wikipedia.org";;
private Constants() { }
-}
\ No newline at end of file
+}
diff --git
a/app/src/main/java/org/wikipedia/createaccount/authmanager/AMCreateAccountTask.java
b/app/src/main/java/org/wikipedia/createaccount/authmanager/AMCreateAccountTask.java
index db29151..882470b 100644
---
a/app/src/main/java/org/wikipedia/createaccount/authmanager/AMCreateAccountTask.java
+++
b/app/src/main/java/org/wikipedia/createaccount/authmanager/AMCreateAccountTask.java
@@ -52,7 +52,7 @@
.param("password", password)
.param("retype", repeatPassword)
.param("createtoken", token)
- .param("createreturnurl",
Constants.DEFAULT_AUTH_MANAGER_RETURN_URL);
+ .param("createreturnurl", Constants.WIKIPEDIA_URL);
if (email != null) {
builder.param("email", email);
}
diff --git a/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java
b/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java
index 87e5595..5d2286a 100644
--- a/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java
+++ b/app/src/main/java/org/wikipedia/login/authmanager/AMLoginTask.java
@@ -45,7 +45,7 @@
.param("username", username)
.param("password", password)
.param("logintoken", token)
- .param("loginreturnurl",
Constants.DEFAULT_AUTH_MANAGER_RETURN_URL);
+ .param("loginreturnurl", Constants.WIKIPEDIA_URL);
}
@Override
diff --git a/app/src/main/java/org/wikipedia/settings/Prefs.java
b/app/src/main/java/org/wikipedia/settings/Prefs.java
index ab61d90..dfb7dfa 100644
--- a/app/src/main/java/org/wikipedia/settings/Prefs.java
+++ b/app/src/main/java/org/wikipedia/settings/Prefs.java
@@ -4,6 +4,7 @@
import android.support.annotation.NonNull;
import android.support.annotation.Nullable;
+import org.wikipedia.Constants;
import org.wikipedia.R;
import org.wikipedia.SharedPreferenceCookieManager;
import org.wikipedia.WikipediaApp;
@@ -310,7 +311,7 @@
@NonNull
public static Uri getMediaWikiBaseUri() {
return
Uri.parse(StringUtil.defaultIfBlank(getString(R.string.preference_key_mediawiki_base_uri,
null),
- "https://wikipedia.org";));
+ Constants.WIKIPEDIA_URL));
}
public static long getLastRunTime(@NonNull String task) {
--
To view, visit https://gerrit.wikimedia.org/r/288056
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I794d7e37ed415d2f6d505c255e1f76882fc9dfc2
Gerrit-PatchSet: 1
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Niedzielski <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
