Merge branch 'feature/2-use-new-version-of-commonservices' into 'master'

Stan Wijckmans · Stan Wijckmans · commit 8101ca70d897 · 2018-02-10T19:41:16.000Z
2-Update dependencies.

See merge request Stannieman/RestAndroid!5
diff --git a/build.gradle b/build.gradle
@@ -3,7 +3,7 @@ buildscript {
         jcenter()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:2.2.3'
+        classpath 'com.android.tools.build:gradle:3.0.1'
     }
 }
 
diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties
@@ -1,5 +1,5 @@
-distributionBase=GRADLE_USER_HOME
-distributionPath=wrapper/dists
-zipStoreBase=GRADLE_USER_HOME
-zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-3.3-all.zip
+distributionBase=GRADLE_USER_HOME
+distributionPath=wrapper/dists
+zipStoreBase=GRADLE_USER_HOME
+zipStorePath=wrapper/dists
+distributionUrl=https\://services.gradle.org/distributions/gradle-4.5.1-all.zip
diff --git a/rest/build.gradle b/rest/build.gradle
@@ -2,14 +2,14 @@ apply plugin: 'com.android.library'
 apply plugin: 'maven-publish'
 
 android {
-    compileSdkVersion 25
-    buildToolsVersion "25.0.2"
+    compileSdkVersion 27
+    buildToolsVersion "27.0.3"
 
     defaultConfig {
         minSdkVersion 16
-        targetSdkVersion 25
         versionCode 3
         versionName "1.0.1"
+        targetSdkVersion 27
     }
 
     lintOptions {
@@ -18,9 +18,9 @@ android {
 }
 
 dependencies {
-    compile 'be.stannieman:commonservices:1.0.1.0'
-    compile 'com.android.volley:volley:1.0.0'
-    compile 'com.fasterxml.jackson.core:jackson-databind:2.8.7'
+    compile 'be.stannieman:commonservices:1.0.2.0'
+    compile 'com.android.volley:volley:1.1.0'
+    compile 'com.fasterxml.jackson.core:jackson-databind:2.9.4'
 }
 
 task createJavaDoc(type: Javadoc) {
@@ -44,7 +44,7 @@ publishing {
         aar(MavenPublication) {
             groupId 'be.stannieman'
             artifactId 'rest'
-            version '1.0.1'
+            version '1.2.0'
             artifact "$buildDir\\outputs\\aar\\signed-${project.getName()}-release.aar"
             artifact createJavaDocJar
             artifact createSourcesJar
diff --git a/rest/src/main/java/stannieman/rest/RestClientBase.java b/rest/src/main/java/stannieman/rest/RestClientBase.java
@@ -21,6 +21,7 @@
 import java.util.concurrent.TimeoutException;
 
 import stannieman.commonservices.helpers.ResultCodeHelper;
+import stannieman.commonservices.models.DataServiceResult;
 import stannieman.commonservices.models.GeneralResultCodes;
 import stannieman.commonservices.models.IHasDataAndSuccessState;
 import stannieman.commonservices.models.ServiceResult;
@@ -69,26 +70,26 @@ protected <SuccessResponseDataType, ErrorResponseDataType extends ErrorResponseD
         if (!uriResult.isSuccess()) {
             RestClientResultCodes resultCode = ResultCodeHelper.GetResultCodeOrNull(uriResult, RestClientResultCodes.class);
             return resultCode != null
-                    ? new ServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(resultCode)
-                    : new ServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(RestClientResultCodes.CANNOT_CREATE_URI);
+                    ? new DataServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(resultCode)
+                    : new DataServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(RestClientResultCodes.CANNOT_CREATE_URI);
         }
         String uriString = uriResult.getData();
 
         IHasDataAndSuccessState<String> bodyStringResult = getBodyString(requestProperties.getBody());
         if (!bodyStringResult.isSuccess()) {
             RestClientResultCodes resultCode = ResultCodeHelper.GetResultCodeOrNull(uriResult, RestClientResultCodes.class);
             return resultCode != null
-                    ? new ServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(resultCode)
-                    : new ServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(RestClientResultCodes.CANNOT_CREATE_JSON_STRING_FROM_OBJECT);
+                    ? new DataServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(resultCode)
+                    : new DataServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(RestClientResultCodes.CANNOT_CREATE_JSON_STRING_FROM_OBJECT);
         }
         String bodyString = bodyStringResult.getData();
 
         IHasDataAndSuccessState<NetworkResponse> networkResponseResult = getNetworkResponse(method, uriString, headers, bodyString);
         if (!networkResponseResult.isSuccess()) {
             RestClientResultCodes resultCode = ResultCodeHelper.GetResultCodeOrNull(uriResult, RestClientResultCodes.class);
             return resultCode != null
-                    ? new ServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(resultCode)
-                    : new ServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(RestClientResultCodes.REQUEST_FAILED);
+                    ? new DataServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(resultCode)
+                    : new DataServiceResult<RestResult<SuccessResponseDataType, ErrorResponseDataType>, RestClientResultCodes>(RestClientResultCodes.REQUEST_FAILED);
         }
         NetworkResponse networkResponse = networkResponseResult.getData();
 
@@ -101,21 +102,21 @@ private IHasDataAndSuccessState<String> getUriString(String subPath, String[] su
         try {
             String uriString = new URI(scheme, null, host, port, absoluteEndpointPath + parametrizedSubPath, null, null).toString();
             String queryString = QueryParamsHelper.getQueryString(queryParameters, ENCODING);
-            return new ServiceResult<>(uriString + queryString, GeneralResultCodes.OK);
+            return new DataServiceResult<>(uriString + queryString, GeneralResultCodes.OK);
         } catch (Exception e) {
-            return new ServiceResult<>(CANNOT_CREATE_URI);
+            return new DataServiceResult<>(CANNOT_CREATE_URI);
         }
     }
 
     private IHasDataAndSuccessState<String> getBodyString(Object body) {
         if (body != null) {
             try {
-                return new ServiceResult<>(objectMapper.writeValueAsString(body), GeneralResultCodes.OK);
+                return new DataServiceResult<>(objectMapper.writeValueAsString(body), GeneralResultCodes.OK);
             } catch (Exception e) {
-                return new ServiceResult<>(RestClientResultCodes.CANNOT_CREATE_JSON_STRING_FROM_OBJECT);
+                return new DataServiceResult<>(RestClientResultCodes.CANNOT_CREATE_JSON_STRING_FROM_OBJECT);
             }
         }
-        return new ServiceResult<>();
+        return new DataServiceResult<>();
     }
 
     private IHasDataAndSuccessState<NetworkResponse> getNetworkResponse(int method, String uriString, Map<String, String> headers, String bodyString) {
@@ -127,58 +128,58 @@ private IHasDataAndSuccessState<NetworkResponse> getNetworkResponse(int method,
             response = future.get(timeout, TimeUnit.MILLISECONDS);
         }
         catch (InterruptedException e) {
-            return new ServiceResult<>(RestClientResultCodes.REQUEST_INTERRUPTED);
+            return new DataServiceResult<>(RestClientResultCodes.REQUEST_INTERRUPTED);
         } catch (ExecutionException e) {
             Throwable cause = e.getCause();
             if (!(cause instanceof VolleyError)) {
-                return new ServiceResult<>(RestClientResultCodes.REQUEST_FAILED);
+                return new DataServiceResult<>(RestClientResultCodes.REQUEST_FAILED);
             }
             VolleyError error = (VolleyError) cause;
             if (error.networkResponse == null) {
-                return new ServiceResult<>(RestClientResultCodes.REQUEST_FAILED);
+                return new DataServiceResult<>(RestClientResultCodes.REQUEST_FAILED);
             }
             response = error.networkResponse;
         } catch (TimeoutException e) {
-            return new ServiceResult<>(RestClientResultCodes.REQUEST_TIMED_OUT);
+            return new DataServiceResult<>(RestClientResultCodes.REQUEST_TIMED_OUT);
         }
 
-        return new ServiceResult<>(response, GeneralResultCodes.OK);
+        return new DataServiceResult<>(response, GeneralResultCodes.OK);
     }
 
     private <SuccessResponseDataType, ErrorResponseDataType extends ErrorResponseDataBase> IHasDataAndSuccessState<RestResult<SuccessResponseDataType, ErrorResponseDataType>> createRestResultFromNetworkResponse(Class<SuccessResponseDataType> successResponseDataType, Class<ErrorResponseDataType> errorResponseDataType, NetworkResponse networkResponse, Integer[] successStatusCodes) {
         String jsonString = NetworkResponseRequest.parseToString(networkResponse);
 
         if (isStatusCodeOk(networkResponse.statusCode, successStatusCodes)) {
             if (successResponseDataType == null) {
-                return new ServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(networkResponse.statusCode), GeneralResultCodes.OK);
+                return new DataServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(networkResponse.statusCode), GeneralResultCodes.OK);
             }
             else {
                 try {
                     SuccessResponseDataType successObject = objectMapper.readValue(jsonString, successResponseDataType);
-                    return new ServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(networkResponse.statusCode, successObject), GeneralResultCodes.OK);
+                    return new DataServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(networkResponse.statusCode, successObject), GeneralResultCodes.OK);
                 } catch (JsonMappingException e) {
-                    return new ServiceResult<>(RestClientResultCodes.JSON_RESPONSE_DATA_TYPE_MISMATCH);
+                    return new DataServiceResult<>(RestClientResultCodes.JSON_RESPONSE_DATA_TYPE_MISMATCH);
                 } catch (JsonParseException e) {
-                    return new ServiceResult<>(RestClientResultCodes.RESPONSE_IS_NOT_VALID_JSON);
+                    return new DataServiceResult<>(RestClientResultCodes.RESPONSE_IS_NOT_VALID_JSON);
                 } catch (IOException e) {
-                    return new ServiceResult<>(RestClientResultCodes.CANNOT_CREATE_OBJECT_FROM_SUCCESS_RESPONSE);
+                    return new DataServiceResult<>(RestClientResultCodes.CANNOT_CREATE_OBJECT_FROM_SUCCESS_RESPONSE);
                 }
             }
         }
 
         if (errorResponseDataType == null) {
-            return new ServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(false, networkResponse.statusCode));
+            return new DataServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(false, networkResponse.statusCode));
         }
         else {
             try {
                 ErrorResponseDataType errorObject = objectMapper.readValue(jsonString, errorResponseDataType);
-                return new ServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(networkResponse.statusCode, errorObject), GeneralResultCodes.OK);
+                return new DataServiceResult<>(new RestResult<SuccessResponseDataType, ErrorResponseDataType>(networkResponse.statusCode, errorObject), GeneralResultCodes.OK);
             } catch (JsonMappingException e) {
-                return new ServiceResult<>(RestClientResultCodes.JSON_ERROR_DATA_TYPE_MISMATCH);
+                return new DataServiceResult<>(RestClientResultCodes.JSON_ERROR_DATA_TYPE_MISMATCH);
             } catch (JsonParseException e) {
-                return new ServiceResult<>(RestClientResultCodes.RESPONSE_IS_NOT_VALID_JSON);
+                return new DataServiceResult<>(RestClientResultCodes.RESPONSE_IS_NOT_VALID_JSON);
             } catch (IOException e) {
-                return new ServiceResult<>(RestClientResultCodes.CANNOT_CREATE_OBJECT_FROM_ERROR_RESPONSE);
+                return new DataServiceResult<>(RestClientResultCodes.CANNOT_CREATE_OBJECT_FROM_ERROR_RESPONSE);
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@ buildscript {`
`3`	`3`	`jcenter()`
`4`	`4`	`}`
`5`	`5`	`dependencies {`
`6`		`- classpath 'com.android.tools.build:gradle:2.2.3'`
	`6`	`+ classpath 'com.android.tools.build:gradle:3.0.1'`
`7`	`7`	`}`
`8`	`8`	`}`
`9`	`9`