Merge pull request #696 from HubSpot/add-api-remove

mattcoley · web-flow · commit 512286e55b02 · 2021-06-23T15:24:01.000-04:00
Add method for removing last error
diff --git a/src/main/java/com/hubspot/jinjava/interpret/JinjavaInterpreter.java b/src/main/java/com/hubspot/jinjava/interpret/JinjavaInterpreter.java
@@ -650,8 +650,10 @@ public void addError(TemplateError templateError) {
     }
   }
 
-  public TemplateError removeError(int index) {
-    return errors.remove(index);
+  public void removeLastError() {
+    if (!errors.isEmpty()) {
+      errors.remove(errors.size() - 1);
+    }
   }
 
   public Optional<TemplateError> getLastError() {

Original file line number	Diff line number	Diff line change
`@@ -650,8 +650,10 @@ public void addError(TemplateError templateError) {`
`650`	`650`	`}`
`651`	`651`	`}`
`652`	`652`
`653`		`- public TemplateError removeError(int index) {`
`654`		`- return errors.remove(index);`
	`653`	`+ public void removeLastError() {`
	`654`	`+ if (!errors.isEmpty()) {`
	`655`	`+ errors.remove(errors.size() - 1);`
	`656`	`+ }`
`655`	`657`	`}`
`656`	`658`
`657`	`659`	`public Optional<TemplateError> getLastError() {`