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

Commit b8ce7e0

Browse files
koziodigitalincniftynei
authored andcommitted
Fixed silent error shown on crashlytics. Network response is now check for success
1 parent 965ec73 commit b8ce7e0

File tree

1 file changed

+22
-18
lines changed

1 file changed

+22
-18
lines changed

app/src/main/java/com/zulip/android/networking/AsyncGetEvents.java

Lines changed: 22 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,30 @@
11
package com.zulip.android.networking;
22

3-
import java.io.IOException;
4-
import java.net.SocketTimeoutException;
5-
import java.sql.SQLException;
6-
import java.util.ArrayList;
7-
import java.util.HashMap;
8-
import java.util.concurrent.Callable;
9-
10-
import org.apache.commons.lang.time.StopWatch;
11-
import org.json.JSONArray;
12-
import org.json.JSONException;
13-
import org.json.JSONObject;
14-
153
import android.content.Intent;
164
import android.os.SystemClock;
175
import android.util.Log;
186

197
import com.j256.ormlite.dao.RuntimeExceptionDao;
208
import com.j256.ormlite.misc.TransactionManager;
9+
import com.zulip.android.ZulipApp;
10+
import com.zulip.android.activities.ZulipActivity;
2111
import com.zulip.android.models.Message;
2212
import com.zulip.android.models.MessageRange;
2313
import com.zulip.android.models.Person;
2414
import com.zulip.android.models.Stream;
2515
import com.zulip.android.util.ZLog;
16+
17+
import org.apache.commons.lang.time.StopWatch;
18+
import org.json.JSONArray;
19+
import org.json.JSONException;
20+
import org.json.JSONObject;
21+
22+
import java.io.IOException;
23+
import java.net.SocketTimeoutException;
24+
import java.sql.SQLException;
25+
import java.util.ArrayList;
26+
import java.util.HashMap;
27+
import java.util.concurrent.Callable;
2628
import com.zulip.android.activities.ZulipActivity;
2729
import com.zulip.android.ZulipApp;
2830
import com.zulip.android.widget.ZulipWidget;
@@ -104,21 +106,23 @@ private void register() throws JSONException, IOException {
104106
StopWatch watch = new StopWatch();
105107
watch.start();
106108
request.setMethodAndUrl("POST", "v1/register");
107-
String responseData = request.execute().body().string();
109+
Response responseData = request.execute();
108110
watch.stop();
109111
Log.i("perf", "net: v1/register: " + watch.toString());
110112

111113
watch.reset();
112114
watch.start();
113-
JSONObject response = new JSONObject(responseData);
115+
JSONObject response = new JSONObject(responseData.body().toString());
114116
watch.stop();
115117
Log.i("perf", "json: v1/register: " + watch.toString());
116118

117-
registeredOrGotEventsThisRun = true;
118-
app.setEventQueueId(response.getString("queue_id"));
119-
app.setLastEventId(response.getInt("last_event_id"));
119+
if(responseData.isSuccessful()) {
120+
registeredOrGotEventsThisRun = true;
121+
app.setEventQueueId(response.getString("queue_id"));
122+
app.setLastEventId(response.getInt("last_event_id"));
120123

121-
processRegister(response);
124+
processRegister(response);
125+
}
122126
}
123127

124128
public void run() {

0 commit comments

Comments
 (0)