Skip to content

Commit 1a83c81

Browse files
authored
Merge pull request #49 from creatubbles/null_safety
Add annotations and remove unused fields/files
2 parents dfdbe91 + ae3706b commit 1a83c81

69 files changed

Lines changed: 338 additions & 240 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

api/src/main/java/com/creatubbles/api/Configuration.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.creatubbles.api;
22

33
import android.app.Application;
4+
import android.support.annotation.NonNull;
45

56
import com.creatubbles.api.exception.InvalidParametersException;
67

@@ -22,7 +23,7 @@ public class Configuration {
2223
private final Application context;
2324
private final Locale locale;
2425

25-
public Configuration(Builder builder) {
26+
public Configuration(@NonNull Builder builder) {
2627
this.clientId = builder.clientId;
2728
this.clientSecret = builder.clientSecret;
2829
this.baseUrl = builder.baseUrl;
@@ -38,31 +39,32 @@ public static class Builder {
3839
Application context;
3940
Locale locale;
4041

41-
public Builder clientId(String clientId) {
42+
public Builder clientId(@NonNull String clientId) {
4243
this.clientId = clientId;
4344
return this;
4445
}
4546

46-
public Builder clientSecret(String clientSecret) {
47+
public Builder clientSecret(@NonNull String clientSecret) {
4748
this.clientSecret = clientSecret;
4849
return this;
4950
}
5051

51-
public Builder baseUrl(String baseUrl) {
52+
public Builder baseUrl(@NonNull String baseUrl) {
5253
this.baseUrl = baseUrl;
5354
return this;
5455
}
5556

56-
public Builder application(Application application) {
57+
public Builder application(@NonNull Application application) {
5758
this.context = application;
5859
return this;
5960
}
6061

61-
public Builder locale(Locale locale) {
62+
public Builder locale(@NonNull Locale locale) {
6263
this.locale = locale;
6364
return this;
6465
}
6566

67+
@NonNull
6668
public Configuration build() {
6769
if (clientId == null) {
6870
throw new InvalidParametersException(INVALID_CLIENT_ID_MESSAGE);

api/src/main/java/com/creatubbles/api/CreatubblesApi.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api;
22

3+
import android.support.annotation.NonNull;
4+
35
import com.creatubbles.api.di.modules.ApiModule;
46
import com.creatubbles.api.exception.InitializationException;
57

@@ -11,7 +13,7 @@ public final class CreatubblesApi {
1113

1214
private static Configuration apiConfiguration = null;
1315

14-
public static void initialize(Configuration configuration) {
16+
public static void initialize(@NonNull Configuration configuration) {
1517
if (configuration == null) {
1618
throw new NullPointerException("Configuration can't be null");
1719
}

api/src/main/java/com/creatubbles/api/exception/Error.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api.exception;
22

3+
import android.support.annotation.NonNull;
4+
35
/**
46
* Created by mariuszostapowicz on 08.03.2016.
57
*/
@@ -9,18 +11,22 @@ public class Error {
911
private String title;
1012
private String detail;
1113

14+
@NonNull
1215
public Integer getStatus() {
1316
return status;
1417
}
1518

19+
@NonNull
1620
public String getCode() {
1721
return code;
1822
}
1923

24+
@NonNull
2025
public String getTitle() {
2126
return title;
2227
}
2328

29+
@NonNull
2430
public String getDetail() {
2531
return detail;
2632
}

api/src/main/java/com/creatubbles/api/exception/ErrorResponse.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,17 @@
11
package com.creatubbles.api.exception;
22

3+
import android.support.annotation.NonNull;
4+
5+
import java.util.Collections;
36
import java.util.List;
47

58
/**
69
* Created by mariuszostapowicz on 08.03.2016.
710
*/
811
public class ErrorResponse {
9-
private List<Error> errors;
12+
private final List<Error> errors = Collections.emptyList();
1013

14+
@NonNull
1115
public List<Error> getErrors() {
1216
return errors;
1317
}

api/src/main/java/com/creatubbles/api/model/CreatubblesResponse.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
package com.creatubbles.api.model;
22

3+
import android.support.annotation.NonNull;
4+
import android.support.annotation.Nullable;
5+
36
import com.github.jasminb.jsonapi.Links;
47

58
public class CreatubblesResponse<D> {
@@ -16,14 +19,17 @@ public CreatubblesResponse(D data, Meta meta, Links links) {
1619
this.links = links;
1720
}
1821

22+
@NonNull
1923
public D getData() {
2024
return data;
2125
}
2226

27+
@Nullable
2328
public Meta getMeta() {
2429
return meta;
2530
}
2631

32+
@Nullable
2733
public Links getLinks() {
2834
return links;
2935
}

api/src/main/java/com/creatubbles/api/model/GallerySubmission.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api.model;
22

3+
import android.support.annotation.NonNull;
4+
35
import com.creatubbles.api.model.creation.Creation;
46
import com.creatubbles.api.model.gallery.Gallery;
57
import com.fasterxml.jackson.annotation.JsonCreator;
@@ -26,15 +28,17 @@ public class GallerySubmission {
2628
public GallerySubmission() {
2729
}
2830

29-
public GallerySubmission(String galleryId, String creationId) {
31+
public GallerySubmission(@NonNull String galleryId, @NonNull String creationId) {
3032
this.gallery = new Gallery(galleryId);
3133
this.creation = new Creation(creationId);
3234
}
3335

36+
@NonNull
3437
public Gallery getGallery() {
3538
return gallery;
3639
}
3740

41+
@NonNull
3842
public Creation getCreation() {
3943
return creation;
4044
}

api/src/main/java/com/creatubbles/api/model/Meta.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api.model;
22

3+
import android.support.annotation.Nullable;
4+
35
import com.creatubbles.api.converter.AbilitiesConverter;
46
import com.fasterxml.jackson.annotation.JsonProperty;
57
import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
@@ -31,30 +33,37 @@ public class Meta {
3133
@JsonDeserialize(using = AbilitiesConverter.class)
3234
private List<Ability> abilities;
3335

36+
@Nullable
3437
public Integer getTotalPages() {
3538
return totalPages;
3639
}
3740

41+
@Nullable
3842
public Integer getTotalCount() {
3943
return totalCount;
4044
}
4145

46+
@Nullable
4247
public List<String> getBubbledCreations() {
4348
return unmodifiableList(userBubbledCreations);
4449
}
4550

51+
@Nullable
4652
public List<String> getBubbledGalleries() {
4753
return unmodifiableList(userBubbledGalleries);
4854
}
4955

56+
@Nullable
5057
public List<String> getBubbledUsers() {
5158
return unmodifiableList(userBubbledUsers);
5259
}
5360

61+
@Nullable
5462
public List<String> getFollowedUsers() {
5563
return unmodifiableList(followedUsers);
5664
}
5765

66+
@Nullable
5867
public List<Ability> getAbilities() {
5968
return unmodifiableList(abilities);
6069
}

api/src/main/java/com/creatubbles/api/model/ObjectType.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api.model;
22

3+
import android.support.annotation.NonNull;
4+
35
import com.fasterxml.jackson.annotation.JsonCreator;
46
import com.fasterxml.jackson.annotation.JsonValue;
57

@@ -25,6 +27,7 @@ public enum ObjectType {
2527
}
2628

2729
@JsonValue
30+
@NonNull
2831
public String getTypeName() {
2932
return typeName;
3033
}

api/src/main/java/com/creatubbles/api/model/PasswordChange.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api.model;
22

3+
import android.support.annotation.NonNull;
4+
35
import com.fasterxml.jackson.annotation.JsonCreator;
46
import com.fasterxml.jackson.annotation.JsonProperty;
57
import com.github.jasminb.jsonapi.annotations.Id;
@@ -29,7 +31,7 @@ public class PasswordChange {
2931
*
3032
* @see PasswordChange#create(String, String)
3133
*/
32-
public static PasswordChange create(String newPassword) {
34+
public static PasswordChange create(@NonNull String newPassword) {
3335
return new PasswordChange(newPassword, null, null);
3436
}
3537

@@ -38,7 +40,7 @@ public static PasswordChange create(String newPassword) {
3840
*
3941
* @see PasswordChange#create(String)
4042
*/
41-
public static PasswordChange create(String newPassword, String currentPassword) {
43+
public static PasswordChange create(@NonNull String newPassword, @NonNull String currentPassword) {
4244
return new PasswordChange(newPassword, currentPassword, newPassword);
4345
}
4446

@@ -56,6 +58,7 @@ public String getId() {
5658
return id;
5759
}
5860

61+
@NonNull
5962
public String getNewPassword() {
6063
return newPassword;
6164
}

api/src/main/java/com/creatubbles/api/model/Report.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.creatubbles.api.model;
22

3+
import android.support.annotation.NonNull;
4+
35
import com.fasterxml.jackson.annotation.JsonCreator;
46
import com.github.jasminb.jsonapi.annotations.Id;
57
import com.github.jasminb.jsonapi.annotations.Type;
@@ -22,14 +24,16 @@ private Report(String message) {
2224
this.message = message;
2325
}
2426

25-
public static Report create(String message) {
27+
public static Report create(@NonNull String message) {
2628
return new Report(message);
2729
}
2830

31+
@NonNull
2932
public String getId() {
3033
return id;
3134
}
3235

36+
@NonNull
3337
public String getMessage() {
3438
return message;
3539
}

0 commit comments

Comments
 (0)