Added One Statement per line and Enforced checkstyle

This commit is contained in:
thesmallstar
2020-04-18 03:23:54 +02:00
committed by Michael Vorburger ⛑️
parent 1c74398d90
commit dd2e934cb1
4 changed files with 13 additions and 7 deletions
@@ -58,6 +58,7 @@
<property name="sortImportsInGroupAlphabetically" value="true"/> <property name="sortImportsInGroupAlphabetically" value="true"/>
<property name="customImportOrderRules" value="STATIC###THIRD_PARTY_PACKAGE"/> <property name="customImportOrderRules" value="STATIC###THIRD_PARTY_PACKAGE"/>
</module> </module>
<module name="OneStatementPerLine"/>
<!-- TODO Enable many more checks (go about this one by one, step by step, raise separate PRs fixing and then enforcing): <!-- TODO Enable many more checks (go about this one by one, step by step, raise separate PRs fixing and then enforcing):
@@ -130,7 +131,7 @@
value="WhitespaceAround: ''{0}'' is not preceded with whitespace."/> value="WhitespaceAround: ''{0}'' is not preceded with whitespace."/>
</module> </module>
<module name="ParenPad" /> <module name="ParenPad" />
<module name="OneStatementPerLine"/>
<module name="MultipleVariableDeclarations"/> <module name="MultipleVariableDeclarations"/>
<module name="ArrayTypeStyle"/> <module name="ArrayTypeStyle"/>
<module name="MissingSwitchDefault"/> <module name="MissingSwitchDefault"/>
@@ -70,7 +70,8 @@ public class EmailConfigurationWritePlatformServiceImpl implements EmailConfigur
for (EmailConfiguration config : configurations) { for (EmailConfiguration config : configurations) {
if(config.getName() !=null){ if(config.getName() !=null){
String value = command.stringValueOfParameterNamed(config.getName()); String value = command.stringValueOfParameterNamed(config.getName());
config.setValue(value); changes.put(config.getName(),value); config.setValue(value);
changes.put(config.getName(),value);
this.repository.saveAndFlush(config); this.repository.saveAndFlush(config);
} }
} }
@@ -51,13 +51,17 @@ public class CashierTxnType implements Serializable {
switch(id) { switch(id) {
case 101: case 101:
retVal = ALLOCATE; break; retVal = ALLOCATE;
break;
case 102: case 102:
retVal = SETTLE; break; retVal = SETTLE;
break;
case 103: case 103:
retVal = INWARD_CASH_TXN; break; retVal = INWARD_CASH_TXN;
break;
case 104: case 104:
retVal = OUTWARD_CASH_TXN; break; retVal = OUTWARD_CASH_TXN;
break;
default: default:
break; break;
} }
@@ -81,7 +81,7 @@ final class UsersApiResourceSwagger {
@ApiModelProperty(example = "false") @ApiModelProperty(example = "false")
public Boolean passwordNeverExpires; public Boolean passwordNeverExpires;
public StaffData staff; public StaffData staff;
public Collection<RoleData> availableRoles;; public Collection<RoleData> availableRoles;
public Collection<RoleData> selectedRoles; public Collection<RoleData> selectedRoles;
} }