Switch to Set.of(...) and move the "new HashSet(...)" to the check me…

…thod in the TemplateCompiler.

"new HashSet(...)" is needed because "Set.of(...)" is immutable.

Fixes: SE-13618

src/main/java/sirius/pasta/tagliatelle/compiler/TemplateCompiler.java

@@ -434,7 +434,7 @@ private void parseAttributes(TagHandler handler) {
      * @param attributes the set of attributes which were parsed
      */
     private void checkMissingAttributes(TagHandler handler, Set<String> attributes) {
-        Set<String> missingAttributes = handler.getRequiredAttributeNames();
+        Set<String> missingAttributes = new HashSet<>(handler.getRequiredAttributeNames());
         missingAttributes.removeAll(attributes);
         if (!missingAttributes.isEmpty()) {
             missingAttributes.forEach(attr -> context.error(reader.current(),

src/main/java/sirius/pasta/tagliatelle/tags/ArgTag.java

@@ -162,6 +162,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME, PARAM_TYPE));
+        return Set.of(PARAM_NAME, PARAM_TYPE);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/BlockTag.java

@@ -90,6 +90,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME));
+        return Set.of(PARAM_NAME);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/DefineTag.java

@@ -86,6 +86,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME));
+        return Set.of(PARAM_NAME);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/DynamicInvokeTag.java

@@ -74,6 +74,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(ATTR_TEMPLATE));
+        return Set.of(ATTR_TEMPLATE);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/ExtensionsTag.java

@@ -92,6 +92,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(ATTR_TARGET));
+        return Set.of(ATTR_TARGET);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/ExtraBlockTag.java

@@ -91,6 +91,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME));
+        return Set.of(PARAM_NAME);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/ForTag.java

@@ -121,6 +121,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_ITEMS, PARAM_TYPE, PARAM_VAR));
+        return Set.of(PARAM_ITEMS, PARAM_TYPE, PARAM_VAR);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/IfTag.java

@@ -76,6 +76,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of("test"));
+        return Set.of("test");
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/IncludeTag.java

@@ -79,6 +79,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(ATTR_NAME));
+        return Set.of(ATTR_NAME);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/LocalTag.java

@@ -97,6 +97,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME, PARAM_VALUE));
+        return Set.of(PARAM_NAME, PARAM_VALUE);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/PragmaTag.java

@@ -84,6 +84,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME));
+        return Set.of(PARAM_NAME);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/RenderTag.java

@@ -84,6 +84,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(PARAM_NAME));
+        return Set.of(PARAM_NAME);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/SassTag.java

@@ -98,6 +98,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of(SOURCE_ATTRIBUTE));
+        return Set.of(SOURCE_ATTRIBUTE);
     }
 }

src/main/java/sirius/pasta/tagliatelle/tags/SwitchTag.java

@@ -73,6 +73,6 @@ public Class<?> getExpectedAttributeType(String name) {
 
     @Override
     public Set<String> getRequiredAttributeNames() {
-        return new HashSet<>(List.of("test"));
+        return Set.of("test");
     }
 }