Skip to content

Commit e6b0ea4

Browse files
committed
Avoid NPE
1 parent 60e543c commit e6b0ea4

File tree

4 files changed

+9
-9
lines changed

4 files changed

+9
-9
lines changed

library/src/main/java/com/proxerme/library/connection/info/entity/Entry.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ public String getName() {
191191
@NonNull
192192
@GenreParameter.Genre
193193
public String[] getGenres() {
194-
if (genres.isEmpty()) {
194+
if (genres == null || genres.isEmpty()) {
195195
return new String[0];
196196
} else {
197197
return genres.split(DELIMITER);
@@ -207,7 +207,7 @@ public String[] getGenres() {
207207
@NonNull
208208
@FskParameter.FskConstraint
209209
public String[] getFsk() {
210-
if (fsk.isEmpty()) {
210+
if (fsk == null || fsk.isEmpty()) {
211211
return new String[0];
212212
} else {
213213
return fsk.split(DELIMITER);

library/src/main/java/com/proxerme/library/connection/info/entity/EntryCore.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ public String getName() {
148148
@NonNull
149149
@Genre
150150
public String[] getGenres() {
151-
if (genres.isEmpty()) {
151+
if (genres == null || genres.isEmpty()) {
152152
return new String[0];
153153
} else {
154154
return genres.split(DELIMITER);
@@ -164,7 +164,7 @@ public String[] getGenres() {
164164
@NonNull
165165
@FskConstraint
166166
public String[] getFsk() {
167-
if (fsk.isEmpty()) {
167+
if (fsk == null || fsk.isEmpty()) {
168168
return new String[0];
169169
} else {
170170
return fsk.split(DELIMITER);

library/src/main/java/com/proxerme/library/connection/info/entity/Relation.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ public String getName() {
175175
@NonNull
176176
@GenreParameter.Genre
177177
public String[] getGenres() {
178-
if (genres.isEmpty()) {
178+
if (genres == null || genres.isEmpty()) {
179179
return new String[0];
180180
} else {
181181
return genres.split(DELIMITER);
@@ -191,7 +191,7 @@ public String[] getGenres() {
191191
@NonNull
192192
@FskParameter.FskConstraint
193193
public String[] getFsk() {
194-
if (fsk.isEmpty()) {
194+
if (fsk == null || fsk.isEmpty()) {
195195
return new String[0];
196196
} else {
197197
return fsk.split(DELIMITER);
@@ -297,7 +297,7 @@ public int getLicense() {
297297
@NonNull
298298
@GeneralLanguageParameter.GeneralLanguage
299299
public String[] getLanguages() {
300-
if (languages.isEmpty()) {
300+
if (languages == null || languages.isEmpty()) {
301301
return new String[0];
302302
} else {
303303
return languages.split(LANGUAGE_DELIMITER);

library/src/main/java/com/proxerme/library/connection/list/entity/MediaListEntry.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ public String getName() {
125125
@NonNull
126126
@Genre
127127
public String[] getGenres() {
128-
if (genres.isEmpty()) {
128+
if (genres == null || genres.isEmpty()) {
129129
return new String[0];
130130
} else {
131131
return genres.split(" ");
@@ -206,7 +206,7 @@ public float getRating() {
206206
@NonNull
207207
@SubDubLanguage
208208
public String[] getLanguages() {
209-
if (languages.isEmpty()) {
209+
if (languages == null || languages.isEmpty()) {
210210
return new String[0];
211211
} else {
212212
return languages.split(",");

0 commit comments

Comments
 (0)