fix runtime exceptions for forms pages

spacious-team · Nov 18, 2024 · bcbb508 · bcbb508
1 parent 83e54f1
commit bcbb508
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 7 deletions.
diff --git a/pom.xml b/pom.xml
@@ -253,7 +253,7 @@
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
                 <configuration>
-                    <fork>true</fork> <!-- Must fork or else JVM arguments are ignored. -->
+                    <fork>true</fork> <!-- Must fork or else JVM arguments are ignored -->
                     <showDeprecation>true</showDeprecation>
                     <showWarnings>true</showWarnings>
                     <annotationProcessorPaths>

diff --git a/src/main/java/ru/investbook/web/forms/model/EventCashFlowModel.java b/src/main/java/ru/investbook/web/forms/model/EventCashFlowModel.java
@@ -30,6 +30,7 @@
 import java.time.LocalDate;
 import java.time.LocalTime;
 
+import static java.util.Objects.isNull;
 import static java.util.Objects.requireNonNull;
 import static org.spacious_team.broker.pojo.CashFlowType.*;
 import static org.springframework.util.StringUtils.hasLength;
@@ -47,7 +48,7 @@ public class EventCashFlowModel {
     @DateTimeFormat(pattern = "HH:mm:ss")
     private @NotNull LocalTime time = LocalTime.NOON;
 
-    private @NotNull CashFlowType type;
+    private @NotNull CashFlowType type = CASH;
 
     /**
      * Negative value for cash out, otherwise - positive
@@ -98,7 +99,8 @@ public void setStringType(String value) {
     }
 
     private boolean isValuePositive() {
-        return value.compareTo(BigDecimal.ZERO) >= 0;
+        //noinspection ConstantValue
+        return isNull(value) || value.compareTo(BigDecimal.ZERO) >= 0;
     }
 
     public boolean isAttachedToSecurity() {

diff --git a/src/main/java/ru/investbook/web/forms/model/TransactionModel.java b/src/main/java/ru/investbook/web/forms/model/TransactionModel.java
@@ -136,14 +136,14 @@ public String getSecurityName() {
         return SecurityHelper.getSecurityIsin(security);
     }
 
-    public String getTradeId() {
+    public String getOrGenerateTradeId() {
         if (!hasText(tradeId)) {
-            setTradeId(createTradeId());
+            setTradeId(generateTradeId());
         }
         return requireNonNull(tradeId, "Не задан trade-id");
     }
 
-    private String createTradeId() {
+    private String generateTradeId() {
         //noinspection ConstantValue
         Assert.isTrue(portfolio != null && security != null && date != null && action != null,
                 "Невалидные данные, ошибка вычисления trade-id");

diff --git a/src/main/java/ru/investbook/web/forms/service/TransactionFormsService.java b/src/main/java/ru/investbook/web/forms/service/TransactionFormsService.java
@@ -176,7 +176,7 @@ public void save(TransactionModel tr) {
         }
 
         AbstractTransaction transaction = builder
-                .tradeId(tr.getTradeId())
+                .tradeId(tr.getOrGenerateTradeId())
                 .portfolio(tr.getPortfolio())
                 .timestamp(tr.getDate().atTime(tr.getTime()).atZone(zoneId).toInstant())
                 .security(savedSecurityId)