Skip to content
This repository was archived by the owner on Jul 25, 2024. It is now read-only.

Commit 87cf423

Browse files
committed
Fix #175 - properly save email for Google Auth logins
1 parent 516daa2 commit 87cf423

File tree

3 files changed

+6
-10
lines changed

3 files changed

+6
-10
lines changed

app/src/main/java/com/zulip/android/ZulipApp.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -381,11 +381,11 @@ public void useDefaultServerURL() {
381381
setServerURL(DEFAULT_SERVER_URL);
382382
}
383383

384-
public void setLoggedInApiKey(String apiKey) {
384+
public void setLoggedInApiKey(String apiKey, String email) {
385385
this.api_key = apiKey;
386386
Editor ed = this.settings.edit();
387-
ed.putString(EMAIL, this.getEmail());
388-
ed.putString(API_KEY, api_key);
387+
ed.putString(EMAIL, email);
388+
ed.putString(API_KEY, apiKey);
389389
ed.apply();
390390
afterLogin();
391391
}

app/src/main/java/com/zulip/android/activities/DevAuthActivity.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,15 +61,12 @@ protected void onCreate(Bundle savedInstanceState) {
6161
authEmailAdapter.setOnItemClickListener(new AuthClickListener() {
6262
@Override
6363
public void onItemClick(String email) {
64-
if(email.contains("@")) {
65-
getApp().setEmail(email);
66-
}
6764
getServices()
6865
.loginDEV(email)
6966
.enqueue(new DefaultCallback<LoginResponse>() {
7067
@Override
7168
public void onSuccess(Call<LoginResponse> call, Response<LoginResponse> response) {
72-
getApp().setLoggedInApiKey(response.body().getApiKey());
69+
getApp().setLoggedInApiKey(response.body().getApiKey(), response.body().getEmail());
7370
openHome();
7471
}
7572

app/src/main/java/com/zulip/android/activities/LoginActivity.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -275,7 +275,7 @@ private void handleSignInResult(final GoogleSignInResult result) {
275275
@Override
276276
public void onSuccess(Call<LoginResponse> call, Response<LoginResponse> response) {
277277
connectionProgressDialog.dismiss();
278-
getApp().setEmail(response.body().getEmail());
278+
getApp().setLoggedInApiKey(response.body().getApiKey(), response.body().getEmail());
279279
openHome();
280280
}
281281

@@ -372,15 +372,14 @@ public void onClick(View v) {
372372
connectionProgressDialog.show();
373373
String username = mUserName.getText().toString();
374374
String password = mPassword.getText().toString();
375-
getApp().setEmail(username);
376375
getServices()
377376
.login(username, password)
378377
.enqueue(new DefaultCallback<LoginResponse>() {
379378

380379
@Override
381380
public void onSuccess(Call<LoginResponse> call, Response<LoginResponse> response) {
382381
connectionProgressDialog.dismiss();
383-
getApp().setLoggedInApiKey(response.body().getApiKey());
382+
getApp().setLoggedInApiKey(response.body().getApiKey(), response.body().getEmail());
384383
openHome();
385384
}
386385

0 commit comments

Comments
 (0)