Skip to content

Commit 093840b

Browse files
committed
remove sonar violations
1 parent dbef595 commit 093840b

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

src/main/java/io/asfjava/ui/core/generators/ComboBoxGenerator.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,19 @@ public void generate(ObjectNode fieldFormDefinition, Field field) {
4444
fieldFormDefinition.set("titleMap", titlesMap);
4545
} catch (InstantiationException | IllegalAccessException e) {
4646
ASFUILogger.getLogger().error(e.getMessage());
47+
throw new RuntimeException(e);
4748
}
4849
}
4950

5051
}
5152

5253
private void buildValueDefinition(ObjectMapper comboMapper, ArrayNode titlesMap, String value) {
5354
ObjectNode entry = comboMapper.createObjectNode();
54-
if (value.equals(value.toUpperCase())) {
55+
String upperCasedValue = value.toUpperCase();
56+
String lowerCasedValue = value.toLowerCase();
57+
if (value.equals(upperCasedValue)) {
5558
entry.put("name", value.toLowerCase());
56-
} else if (value.equals(value.toLowerCase())) {
59+
} else if (value.equals(lowerCasedValue)) {
5760
entry.put("name", value.replace(value.substring(0, 1), value.substring(0, 1).toUpperCase()));
5861
} else {
5962
entry.put("name", value);

src/main/java/io/asfjava/ui/core/generators/RadioBoxGenerator.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public void generate(ObjectNode fieldFormDefinition, Field field) {
2121
fieldFormDefinition.put("readOnly", annotation.readOnly());
2222
fieldFormDefinition.put("type", "radios");
2323

24-
JsonNode radioFieldFormDefinition = ((JsonNode) fieldFormDefinition);
24+
JsonNode radioFieldFormDefinition = fieldFormDefinition;
2525

2626
ObjectMapper radioMapper = new ObjectMapper();
2727

@@ -41,6 +41,7 @@ public void generate(ObjectNode fieldFormDefinition, Field field) {
4141
}
4242
} catch (InstantiationException | IllegalAccessException e) {
4343
ASFUILogger.getLogger().error(e.getMessage());
44+
throw new RuntimeException(e);
4445
}
4546

4647
((ObjectNode) radioFieldFormDefinition).set("titleMap", titlesMap);

src/main/java/io/asfjava/ui/core/logging/ASFUILogger.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,8 @@ public final class ASFUILogger {
99
public static Logger getLogger(){
1010
return LOGGER;
1111
}
12+
13+
private ASFUILogger() {
14+
}
15+
1216
}

0 commit comments

Comments
 (0)