Skip to content

Commit 5520a68

Browse files
author
Suneet Srivastava
committed
feat: Removed volley and shifted api calls to retrofit
1 parent 7c4f5cf commit 5520a68

File tree

6 files changed

+147
-104
lines changed

6 files changed

+147
-104
lines changed

‎app/build.gradle‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,6 @@ android {
9292

9393
configurations {
9494
implementation.exclude group: 'org.jetbrains', module: 'annotations'
95-
all*.exclude group: 'com.android.volley'
9695
}
9796

9897
lintOptions {
@@ -145,7 +144,10 @@ dependencies {
145144
androidTestImplementation 'androidx.test:runner:1.2.0'
146145
androidTestImplementation 'androidx.test:rules:1.2.0'
147146

148-
implementation 'com.mcxiaoke.volley:library:1.0.19'
147+
//retrofit
148+
implementation "com.squareup.retrofit2:retrofit:$rootProject.retrofitVersion"
149+
implementation "com.squareup.retrofit2:converter-gson:$rootProject.retrofitVersion"
150+
implementation 'com.squareup.okhttp3:logging-interceptor:4.2.0'
149151

150152
//google and support
151153
implementation "androidx.appcompat:appcompat:$rootProject.supportLibraryVersion"

‎app/src/main/java/org/fossasia/phimpme/share/SharingActivity.java‎

Lines changed: 86 additions & 101 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
import android.text.Editable;
3838
import android.text.TextUtils;
3939
import android.text.TextWatcher;
40+
import android.util.ArrayMap;
4041
import android.util.Log;
4142
import android.view.View;
4243
import android.view.WindowManager;
@@ -54,14 +55,6 @@
5455
import androidx.recyclerview.widget.RecyclerView;
5556
import butterknife.BindView;
5657
import butterknife.ButterKnife;
57-
import com.android.volley.AuthFailureError;
58-
import com.android.volley.DefaultRetryPolicy;
59-
import com.android.volley.Request;
60-
import com.android.volley.RequestQueue;
61-
import com.android.volley.Response;
62-
import com.android.volley.VolleyError;
63-
import com.android.volley.toolbox.StringRequest;
64-
import com.android.volley.toolbox.Volley;
6558
import com.box.androidsdk.content.BoxApiFile;
6659
import com.box.androidsdk.content.BoxConfig;
6760
import com.box.androidsdk.content.BoxException;
@@ -78,6 +71,7 @@
7871
import com.dropbox.core.v2.DbxClientV2;
7972
import com.dropbox.core.v2.files.WriteMode;
8073
import com.google.android.material.snackbar.Snackbar;
74+
import com.google.gson.JsonElement;
8175
import com.mikepenz.community_material_typeface_library.CommunityMaterial;
8276
import com.mikepenz.iconics.view.IconicsImageView;
8377
import com.owncloud.android.lib.common.OwnCloudClient;
@@ -109,8 +103,6 @@
109103
import java.text.SimpleDateFormat;
110104
import java.util.ArrayList;
111105
import java.util.Date;
112-
import java.util.HashMap;
113-
import java.util.Map;
114106
import org.fossasia.phimpme.R;
115107
import org.fossasia.phimpme.base.PhimpmeProgressBarHandler;
116108
import org.fossasia.phimpme.base.RecyclerItemClickListner;
@@ -126,11 +118,16 @@
126118
import org.fossasia.phimpme.share.twitter.HelperMethods;
127119
import org.fossasia.phimpme.utilities.ActivitySwitchHelper;
128120
import org.fossasia.phimpme.utilities.Constants;
121+
import org.fossasia.phimpme.utilities.ImgurApiInterface;
122+
import org.fossasia.phimpme.utilities.ImgurRetrofitClient;
129123
import org.fossasia.phimpme.utilities.NotificationHandler;
130124
import org.fossasia.phimpme.utilities.SnackBarHandler;
131125
import org.fossasia.phimpme.utilities.Utils;
132126
import org.json.JSONException;
133127
import org.json.JSONObject;
128+
import retrofit2.Call;
129+
import retrofit2.Callback;
130+
import retrofit2.Response;
134131

135132
/**
136133
* Class which deals with Sharing images to multiple Account logged in by the user in the app. If
@@ -202,6 +199,8 @@ public class SharingActivity extends ThemedActivity
202199
public String uploadName;
203200
private int positionShareOption;
204201
private boolean triedUploading = false;
202+
private ImgurApiInterface imgurApiInterface =
203+
ImgurRetrofitClient.getImgurRetrofitClient().create(ImgurApiInterface.class);
205204

206205
public static String getClientAuth() {
207206
return Constants.IMGUR_HEADER_CLIENt + " " + Constants.MY_IMGUR_CLIENT_ID;
@@ -1081,107 +1080,93 @@ void uploadImgur() {
10811080
Bitmap bitmap = getBitmapFromPath(saveFilePath);
10821081
final String imageString = getStringImage(bitmap);
10831082
// sending image to server
1084-
StringRequest request =
1085-
new StringRequest(
1086-
Request.Method.POST,
1087-
Constants.IMGUR_IMAGE_UPLOAD_URL,
1088-
new Response.Listener<String>() {
1083+
ArrayMap<String, String> body = new ArrayMap<>();
1084+
body.put("image", imageString);
1085+
if (caption != null && !caption.isEmpty()) {
1086+
body.put("title", caption);
1087+
}
1088+
String authorization;
1089+
if (isPersonal && imgurAuth != null) {
1090+
authorization = imgurAuth;
1091+
} else {
1092+
authorization = getClientAuth();
1093+
}
1094+
imgurApiInterface
1095+
.uploadImageToImgur(authorization, body)
1096+
.enqueue(
1097+
new Callback<JsonElement>() {
10891098
@Override
1090-
public void onResponse(String s) {
1091-
dialog.dismiss();
1092-
JSONObject jsonObject = null;
1093-
1094-
try {
1095-
jsonObject = new JSONObject(s);
1096-
Boolean success = jsonObject.getBoolean("success");
1097-
if (success) {
1098-
final String url = jsonObject.getJSONObject("data").getString("link");
1099+
public void onResponse(Call<JsonElement> call, Response<JsonElement> response) {
1100+
if (response.body() != null && response.isSuccessful()) {
1101+
dialog.dismiss();
1102+
JSONObject jsonObject = null;
1103+
try {
1104+
jsonObject = new JSONObject(response.body().toString());
1105+
boolean success = jsonObject.getBoolean("success");
1106+
if (success) {
1107+
final String url = jsonObject.getJSONObject("data").getString("link");
1108+
1109+
if (isPersonal) {
1110+
imgurString = getString(R.string.upload_personal) + "\n" + url;
1111+
} else {
1112+
imgurString = getString(R.string.upload_anonymous) + "\n" + url;
1113+
}
10991114

1100-
if (isPersonal) {
1101-
imgurString = getString(R.string.upload_personal) + "\n" + url;
1115+
AlertDialogsHelper.getTextDialog(
1116+
SharingActivity.this,
1117+
dialogBuilder,
1118+
R.string.imgur_uplaoded_dialog_title,
1119+
0,
1120+
imgurString);
1121+
dialogBuilder.setPositiveButton(
1122+
getString(R.string.share).toUpperCase(),
1123+
new DialogInterface.OnClickListener() {
1124+
@Override
1125+
public void onClick(DialogInterface dialogInterface, int i) {
1126+
shareMsgOnIntent(SharingActivity.this, url);
1127+
sendResult(Constants.SUCCESS);
1128+
}
1129+
});
1130+
1131+
dialogBuilder.setNeutralButton(
1132+
getString(R.string.copy_action).toUpperCase(),
1133+
new DialogInterface.OnClickListener() {
1134+
@Override
1135+
public void onClick(DialogInterface dialogInterface, int i) {
1136+
copyToClipBoard(SharingActivity.this, url);
1137+
sendResult(Constants.SUCCESS);
1138+
}
1139+
});
1140+
dialogBuilder.setNegativeButton(getString(R.string.exit).toUpperCase(), null);
1141+
AlertDialog alertDialog = dialogBuilder.create();
1142+
alertDialog.show();
1143+
AlertDialogsHelper.setButtonTextColor(
1144+
new int[] {
1145+
DialogInterface.BUTTON_POSITIVE,
1146+
DialogInterface.BUTTON_NEGATIVE,
1147+
DialogInterface.BUTTON_NEUTRAL
1148+
},
1149+
getAccentColor(),
1150+
alertDialog);
11021151
} else {
1103-
imgurString = getString(R.string.upload_anonymous) + "\n" + url;
1152+
SnackBarHandler.create(parent, getString(R.string.error_on_imgur)).show();
1153+
sendResult(FAIL);
11041154
}
1105-
1106-
AlertDialogsHelper.getTextDialog(
1107-
SharingActivity.this,
1108-
dialogBuilder,
1109-
R.string.imgur_uplaoded_dialog_title,
1110-
0,
1111-
imgurString);
1112-
dialogBuilder.setPositiveButton(
1113-
getString(R.string.share).toUpperCase(),
1114-
new DialogInterface.OnClickListener() {
1115-
@Override
1116-
public void onClick(DialogInterface dialogInterface, int i) {
1117-
shareMsgOnIntent(SharingActivity.this, url);
1118-
sendResult(Constants.SUCCESS);
1119-
}
1120-
});
1121-
1122-
dialogBuilder.setNeutralButton(
1123-
getString(R.string.copy_action).toUpperCase(),
1124-
new DialogInterface.OnClickListener() {
1125-
@Override
1126-
public void onClick(DialogInterface dialogInterface, int i) {
1127-
copyToClipBoard(SharingActivity.this, url);
1128-
sendResult(Constants.SUCCESS);
1129-
}
1130-
});
1131-
dialogBuilder.setNegativeButton(getString(R.string.exit).toUpperCase(), null);
1132-
AlertDialog alertDialog = dialogBuilder.create();
1133-
alertDialog.show();
1134-
AlertDialogsHelper.setButtonTextColor(
1135-
new int[] {
1136-
DialogInterface.BUTTON_POSITIVE,
1137-
DialogInterface.BUTTON_NEGATIVE,
1138-
DialogInterface.BUTTON_NEUTRAL
1139-
},
1140-
getAccentColor(),
1141-
alertDialog);
1142-
} else {
1143-
SnackBarHandler.create(parent, getString(R.string.error_on_imgur)).show();
1144-
sendResult(FAIL);
1155+
} catch (JSONException e) {
1156+
e.printStackTrace();
11451157
}
1146-
} catch (JSONException e) {
1147-
e.printStackTrace();
1158+
} else {
1159+
dialog.dismiss();
1160+
SnackBarHandler.create(parent, getString(R.string.error_volly)).show();
11481161
}
11491162
}
1150-
},
1151-
new Response.ErrorListener() {
1163+
11521164
@Override
1153-
public void onErrorResponse(VolleyError volleyError) {
1165+
public void onFailure(Call<JsonElement> call, Throwable t) {
11541166
dialog.dismiss();
1155-
SnackBarHandler.create(parent, getString(R.string.error_volly))
1156-
.show(); // add volleyError to check error
1167+
SnackBarHandler.create(parent, getString(R.string.error_volly)).show();
11571168
}
1158-
}) {
1159-
@Override
1160-
protected Map<String, String> getParams() throws AuthFailureError {
1161-
Map<String, String> parameters = new HashMap<String, String>();
1162-
parameters.put("image", imageString);
1163-
if (caption != null && !caption.isEmpty()) parameters.put("title", caption);
1164-
return parameters;
1165-
}
1166-
1167-
@Override
1168-
public Map<String, String> getHeaders() throws AuthFailureError {
1169-
Map<String, String> headers = new HashMap<String, String>();
1170-
if (isPersonal) {
1171-
if (imgurAuth != null) {
1172-
headers.put(getString(R.string.header_auth), imgurAuth);
1173-
}
1174-
} else {
1175-
headers.put(getString(R.string.header_auth), getClientAuth());
1176-
}
1177-
1178-
return headers;
1179-
}
1180-
};
1181-
request.setRetryPolicy(
1182-
new DefaultRetryPolicy(50000, 5, DefaultRetryPolicy.DEFAULT_BACKOFF_MULT));
1183-
RequestQueue rQueue = Volley.newRequestQueue(SharingActivity.this);
1184-
rQueue.add(request);
1169+
});
11851170
}
11861171

11871172
/**

‎app/src/main/java/org/fossasia/phimpme/utilities/Constants.java‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ public class Constants {
88
public static final int REQUEST_SHARE_RESULT = 50;
99
public static final String SHARE_RESULT = "share_result";
1010

11-
public static final String IMGUR_IMAGE_UPLOAD_URL = "https://api.imgur.com/3/image/";
1211
public static String IMGUR_HEADER_CLIENt = "Client-ID";
1312
public static String IMGUR_HEADER_USER = "Bearer";
1413

‎app/src/main/java/org/fossasia/phimpme/utilities/ImgurApiInterface.java‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
package org.fossasia.phimpme.utilities;
2+
3+
import android.util.ArrayMap;
4+
import com.google.gson.JsonElement;
5+
import retrofit2.Call;
6+
import retrofit2.http.Body;
7+
import retrofit2.http.Header;
8+
import retrofit2.http.POST;
9+
10+
/** Created by @codedsun on 19/Oct/2019 */
11+
public interface ImgurApiInterface {
12+
13+
@POST("/image")
14+
Call<JsonElement> uploadImageToImgur(
15+
@Header("Authorization") String authorization, @Body ArrayMap<String, String> body);
16+
}

‎app/src/main/java/org/fossasia/phimpme/utilities/ImgurRetrofitClient.java‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
package org.fossasia.phimpme.utilities;
2+
3+
import okhttp3.OkHttpClient;
4+
import okhttp3.logging.HttpLoggingInterceptor;
5+
import retrofit2.Retrofit;
6+
import retrofit2.converter.gson.GsonConverterFactory;
7+
8+
/** Created by @codedsun on 19/Oct/2019 */
9+
10+
// Retrofit client used for Imgur
11+
public class ImgurRetrofitClient {
12+
13+
private Retrofit retrofit;
14+
private final String IMGUR_BASE_URL = "https://api.imgur.com/3/";
15+
private static ImgurRetrofitClient imgurRetrofitClient = null;
16+
17+
private ImgurRetrofitClient() {
18+
retrofit =
19+
new Retrofit.Builder()
20+
.baseUrl(IMGUR_BASE_URL)
21+
.client(httpClient().build())
22+
.addConverterFactory(GsonConverterFactory.create())
23+
.build();
24+
}
25+
26+
private static OkHttpClient.Builder httpClient() {
27+
OkHttpClient.Builder httpClient = new OkHttpClient.Builder();
28+
HttpLoggingInterceptor interceptor = new HttpLoggingInterceptor();
29+
interceptor.level(HttpLoggingInterceptor.Level.BODY);
30+
httpClient.interceptors().add(interceptor);
31+
return httpClient;
32+
}
33+
34+
public static Retrofit getImgurRetrofitClient() {
35+
if (imgurRetrofitClient == null) {
36+
imgurRetrofitClient = new ImgurRetrofitClient();
37+
}
38+
return imgurRetrofitClient.retrofit;
39+
}
40+
}

‎build.gradle‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,5 +54,6 @@ ext {
5454
nextCloudVersion = "1.5.0"
5555
lifecycleVersion = "2.1.0"
5656
glideVersion = "4.10.0"
57+
retrofitVersion = "2.6.2"
5758
}
5859

0 commit comments

Comments
 (0)