Don't resolve to custom placeholder without brackets (#256)

Due to a missing check for brackets, word paragraphs which
contained the name of a custom placeholder without brackets were
resolved to the placeholder.
This commit adds the check

build.gradle

@@ -8,7 +8,7 @@ plugins {
 }
 
 group 'com.docutools'
-version = '4.2.1'
+version = '4.2.2'
 
 java {
     toolchain {

src/main/java/com/docutools/jocument/impl/ParsingUtils.java

@@ -40,4 +40,8 @@ public static List<String> getMatchingLoopEnds(String placeholder) {
   public static Matcher matchPlaceholders(String value) {
     return PLACEHOLDER_PATTERN.matcher(value);
   }
+
+  public static boolean containsPlaceholder(String text) {
+    return PLACEHOLDER_PATTERN.matcher(text).find();
+  }
 }

src/main/java/com/docutools/jocument/impl/word/WordGenerator.java

@@ -137,13 +137,19 @@ private boolean isLoopStart(IBodyElement element, List<IBodyElement> remaining)
   }
 
   private boolean isCustomPlaceholder(IBodyElement element) {
-    return element instanceof XWPFParagraph xwpfParagraph
-        && resolver.resolve(
-        ParsingUtils.stripBrackets(
-            WordUtilities.toString(xwpfParagraph)
-        )).map(PlaceholderData::getType)
-        .map(type -> type == PlaceholderType.CUSTOM)
-        .orElse(false);
+    if (!(element instanceof XWPFParagraph xwpfParagraph)) {
+      return false;
+    }
+    String paragraphText = WordUtilities.toString(xwpfParagraph);
+    if (!ParsingUtils.containsPlaceholder(paragraphText)) {
+      return false;
+    }
+    return resolver.resolve(
+    ParsingUtils.stripBrackets(
+        paragraphText
+    )).map(PlaceholderData::getType)
+    .map(type -> type == PlaceholderType.CUSTOM)
+    .orElse(false);
   }
 
   private String fillPlaceholder(MatchResult result, Locale locale) {

src/test/java/com/docutools/jocument/impl/word/WordGeneratorTest.java

@@ -558,6 +558,27 @@ void keepsPageNumbering() throws InterruptedException, IOException {
             .map(String::strip)
             .toList();
         assertThat(instructions, contains("PAGE", "NUMPAGES"));
+    }
+
+    @Test
+    void doesNotResolveCustomPlaceholderWithoutBrackets() throws InterruptedException, IOException {
+        // assemble
+        Template template = Template.fromClassPath("/templates/word/NoCustomPlaceholder.docx")
+            .orElseThrow();
+        CustomPlaceholderRegistry customPlaceholderRegistry = new CustomPlaceholderRegistryImpl();
+        customPlaceholderRegistry.addHandler("quote", QuotePlaceholder.class);
+        PlaceholderResolver resolver = new ReflectionResolver(SampleModelData.PLANET, customPlaceholderRegistry);
+
+        // act
+        Document document = template.startGeneration(resolver);
+        document.blockUntilCompletion(60000L); // 1 minute
 
+        // assert
+        assertThat(document.completed(), is(true));
+        xwpfDocument = TestUtils.getXWPFDocumentFromDocument(document);
+        var documentWrapper = new XWPFDocumentWrapper(xwpfDocument);
+        assertThat(documentWrapper.bodyElement(0).asParagraph().run(0).text(),
+            equalTo("quote"));
     }
+
 }