Skip to content

Commit f3a112f

Browse files
DaanHooglandDaan Hoogland
andauthored
use upstream method for creating enums from strings (#12158)
Co-authored-by: Daan Hoogland <[email protected]>
1 parent db61470 commit f3a112f

File tree

3 files changed

+2
-28
lines changed

3 files changed

+2
-28
lines changed

plugins/storage/volume/scaleio/src/main/java/org/apache/cloudstack/storage/datastore/api/VTreeMigrationInfo.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public MigrationStatus getMigrationStatus() {
5959
}
6060

6161
public void setMigrationStatus(String migrationStatus) {
62-
this.migrationStatus = EnumUtils.fromString(MigrationStatus.class, migrationStatus, MigrationStatus.None);
62+
this.migrationStatus = EnumUtils.getEnumIgnoreCase(MigrationStatus.class, migrationStatus, MigrationStatus.None);
6363
}
6464

6565
public void setMigrationStatus(MigrationStatus migrationStatus) {

server/src/main/java/com/cloud/api/ApiDBUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1767,7 +1767,7 @@ public static String findJobInstanceUuid(AsyncJob job) {
17671767
return null;
17681768
}
17691769
String jobInstanceId = null;
1770-
ApiCommandResourceType jobInstanceType = EnumUtils.fromString(ApiCommandResourceType.class, job.getInstanceType(), ApiCommandResourceType.None);
1770+
ApiCommandResourceType jobInstanceType = EnumUtils.getEnumIgnoreCase(ApiCommandResourceType.class, job.getInstanceType(), ApiCommandResourceType.None);
17711771

17721772
if (job.getInstanceId() == null) {
17731773
// when assert is hit, implement 'getInstanceId' of BaseAsyncCmd and return appropriate instance id

utils/src/main/java/com/cloud/utils/EnumUtils.java

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -29,30 +29,4 @@ public static String listValues(Enum<?>[] enums) {
2929
b.append("]");
3030
return b.toString();
3131
}
32-
33-
public static <T extends Enum<T>> T fromString(Class<T> clz, String value, T defaultVal) {
34-
assert (clz != null);
35-
36-
if (value != null) {
37-
try {
38-
return Enum.valueOf(clz, value.trim());
39-
} catch (IllegalArgumentException ex) {
40-
assert (false);
41-
}
42-
}
43-
return defaultVal;
44-
}
45-
46-
public static <T extends Enum<T>> T fromString(Class<T> clz, String value) {
47-
assert (clz != null);
48-
49-
if (value != null) {
50-
try {
51-
return Enum.valueOf(clz, value.trim());
52-
} catch (IllegalArgumentException ex) {
53-
assert (false);
54-
}
55-
}
56-
return null;
57-
}
5832
}

0 commit comments

Comments
 (0)