Skip to content

Commit 4b07e04

Browse files
authored
Merge pull request #259 from microsoftgraph/dev
release 2.0.8
2 parents 4abc150 + 0b9319e commit 4b07e04

25 files changed

+572
-116
lines changed

.github/CODEOWNERS

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
* @baywet @ddyett @MichaelMainer @nikithauc @zengin
1+
* @baywet @ddyett @MichaelMainer @nikithauc @zengin @ramsessanchez

android/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,13 @@ apply plugin: "com.android.library"
2323
apply plugin: "com.github.ben-manes.versions"
2424

2525
android {
26-
compileSdkVersion 30
26+
compileSdkVersion 31
2727

2828
defaultConfig {
2929
versionCode 1
3030
versionName "1.0"
3131
minSdkVersion 26
32-
targetSdkVersion 30
32+
targetSdkVersion 31
3333
}
3434

3535
buildTypes {

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ mavenGroupId = com.microsoft.graph
2525
mavenArtifactId = microsoft-graph-core
2626
mavenMajorVersion = 2
2727
mavenMinorVersion = 0
28-
mavenPatchVersion = 7
28+
mavenPatchVersion = 8
2929
mavenArtifactSuffix =
3030

3131
#These values are used to run functional tests

gradle/dependencies.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@ dependencies {
22
// Use JUnit test framework
33
testImplementation 'org.junit.jupiter:junit-jupiter-api:5.7.2'
44
testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine:5.7.2'
5-
testImplementation 'org.mockito:mockito-inline:3.11.1'
5+
testImplementation 'org.mockito:mockito-inline:3.11.2'
66

77
api 'com.squareup.okhttp3:okhttp:4.9.1'
88

99
implementation 'com.google.guava:guava:30.1.1-jre'
1010

1111
implementation 'com.google.code.gson:gson:2.8.7'
12-
api 'com.azure:azure-core:1.17.0'
12+
api 'com.azure:azure-core:1.18.0'
1313
}

pom.xml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
<groupId>com.microsoft.graph</groupId>
1010
<artifactId>microsoft-graph-core</artifactId>
11-
<version>2.0.2</version>
11+
<version>2.0.8</version>
1212
<packaging>pom</packaging>
1313

1414
<properties>
@@ -35,7 +35,7 @@
3535
<dependency>
3636
<groupId>com.azure</groupId>
3737
<artifactId>azure-core</artifactId>
38-
<version>1.17.0</version>
38+
<version>1.18.0</version>
3939
</dependency>
4040
<dependency>
4141
<groupId>org.junit.jupiter</groupId>
@@ -46,8 +46,8 @@
4646
<dependency>
4747
<groupId>org.mockito</groupId>
4848
<artifactId>mockito-inline</artifactId>
49-
<version>3.11.1</version>
49+
<version>3.11.2</version>
5050
<scope>test</scope>
5151
</dependency>
5252
</dependencies>
53-
</project>
53+
</project>

readme.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ repositories {
2222
2323
dependencies {
2424
// Include the sdk as a dependency
25-
implementation 'com.microsoft.graph:microsoft-graph-core:2.0.7'
25+
implementation 'com.microsoft.graph:microsoft-graph-core:2.0.8'
2626
// This dependency is only needed if you are using the TokenCrendentialAuthProvider
2727
implementation 'com.azure:azure-identity:1.3.1'
2828
}
@@ -37,7 +37,7 @@ Add the dependency in `dependencies` in pom.xml
3737
<!-- Include the sdk as a dependency -->
3838
<groupId>com.microsoft.graph</groupId>
3939
<artifactId>microsoft-graph-core</artifactId>
40-
<version>2.0.7</version>
40+
<version>2.0.8</version>
4141
<!-- This dependency is only needed if you are using the TokenCrendentialAuthProvider -->
4242
<groupId>com.azure</groupId>
4343
<artifactId>azure-identity</artifactId>

src/main/java/com/microsoft/graph/http/CoreHttpProvider.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,15 @@ public <Result, Body> Request getHttpRequest(@Nonnull final IHttpRequest request
296296
}
297297
} else {
298298
logger.logDebug("Sending " + serializable.getClass().getName() + " as request body");
299-
final String serializeObject = serializer.serializeObject(serializable);
299+
300+
String serializeObject = null;
301+
302+
if ("text/plain".equals(contenttype) && serializable instanceof String) {
303+
serializeObject = (String)serializable;
304+
} else {
305+
serializeObject = serializer.serializeObject(serializable);
306+
}
307+
300308
if(serializeObject == null) {
301309
throw new ClientException("Error during serialization of request body, the result was null", null);
302310
}

src/main/java/com/microsoft/graph/httpcore/TelemetryHandler.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public class TelemetryHandler implements Interceptor{
2424
/**
2525
* Current SDK version
2626
*/
27-
public static final String VERSION = "v2.0.7";
27+
public static final String VERSION = "v2.0.8";
2828
/**
2929
* Verion prefix
3030
*/

src/main/java/com/microsoft/graph/serializer/CollectionPageSerializer.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424

2525
import java.lang.reflect.InvocationTargetException;
2626
import java.lang.reflect.Type;
27-
import java.util.ArrayList;
2827
import java.util.List;
2928
import java.util.Objects;
3029

@@ -112,7 +111,7 @@ public static <T1, T2 extends BaseRequestBuilder<T1>> BaseCollectionPage<T1, T2>
112111
final Class<?> responseClass = Class.forName(responseClassCanonicalName);
113112
final JsonObject responseJson = new JsonObject();
114113
responseJson.add("value", json);
115-
final BaseCollectionResponse<T1> response = CollectionResponseSerializer.deserialize(responseJson, responseClass, logger);
114+
final BaseCollectionResponse<T1> response = CollectionResponseDeserializer.deserialize(responseJson, responseClass, logger);
116115
/** eg: com.microsoft.graph.requests.AttachmentCollectionRequestBuilder */
117116
final String responseBuilderCanonicalName = responseClassCanonicalName
118117
.substring(0, responseClassCanonicalName.length() - responseLength) + "RequestBuilder";

src/main/java/com/microsoft/graph/serializer/CollectionResponseSerializer.java renamed to src/main/java/com/microsoft/graph/serializer/CollectionResponseDeserializer.java

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,13 @@
3737
import com.microsoft.graph.http.BaseCollectionResponse;
3838
import com.microsoft.graph.logger.ILogger;
3939

40-
/** Specialized serializer to handle collection responses */
41-
public class CollectionResponseSerializer {
40+
/** Specialized de-serializer to handle collection responses */
41+
public class CollectionResponseDeserializer {
4242
private static DefaultSerializer serializer;
4343
/**
4444
* Not available for instantiation
4545
*/
46-
private CollectionResponseSerializer() {}
46+
private CollectionResponseDeserializer() {}
4747
/**
4848
* Deserializes the JsonElement
4949
*
@@ -86,15 +86,7 @@ public static <T1> BaseCollectionResponse<T1> deserialize(@Nonnull final JsonEle
8686
for(JsonElement sourceElement : sourceArray) {
8787
if(sourceElement.isJsonObject()) {
8888
final JsonObject sourceObject = sourceElement.getAsJsonObject();
89-
Class<?> entityClass = serializer.getDerivedClass(sourceObject, baseEntityClass);
90-
if(entityClass == null) {
91-
if(baseEntityClass == null) {
92-
logger.logError("Could not find target class for object " + sourceObject.toString(), null);
93-
continue;
94-
} else
95-
entityClass = baseEntityClass; // it is possible the odata type is absent or we can't find the derived type (not in SDK yet)
96-
}
97-
final T1 targetObject = (T1)serializer.deserializeObject(sourceObject, entityClass);
89+
final T1 targetObject = (T1)serializer.deserializeObject(sourceObject, baseEntityClass);
9890
((IJsonBackedObject)targetObject).setRawObject(serializer, sourceObject);
9991
list.add(targetObject);
10092
} else if (sourceElement.isJsonPrimitive()) {

0 commit comments

Comments
 (0)