fix: another minor markup fix

Author: quintesse

twinkle-text/src/main/java/org/codejive/twinkle/fluent/impl/DefaultMarkupParser.java

@@ -23,7 +23,7 @@ public class DefaultMarkupParser implements MarkupParser {
     private static Map<String, Color.BasicColor> colors;
 
     private static final Pattern markupPattern = Pattern.compile("(?<!\\{)\\{([^{}]+)}");
-    private static final Pattern varPattern = Pattern.compile("(?<!\\$)\\$(/?\\d+)");
+    private static final Pattern varPattern = Pattern.compile("(?<!\\$)\\$(\\d+)");
 
     @Override
     public void parse(Fluent fluent, String textWithMarkup, Object... args) {
@@ -95,11 +95,7 @@ private String applySubstitutions(String markup, Object... args) {
 
     protected void applyVar(Appendable appendable, String varName, Object... args) {
         try {
-            if (varName.startsWith("/")) {
-                appendable.append("/").append(getVar(varName.substring(1), args));
-            } else {
-                appendable.append(getVar(varName, args));
-            }
+            appendable.append(getVar(varName, args));
         } catch (IOException e) {
             // Ignore
         }

twinkle-text/src/test/java/org/codejive/twinkle/fluent/TestDefaultMarkupParser.java

@@ -448,4 +448,11 @@ public void testTextWithArgumentInMarkup() {
         s.parse("{i}{$1}{/i}", "bold");
         assertThat(s.result()).isEqualTo(Ansi.italic() + Ansi.bold() + Ansi.italicOff());
     }
+
+    @Test
+    public void testTextWithArgumentInMarkup2() {
+        Setup s = new Setup();
+        s.parse("{i}{/$1}{/i}", "ul");
+        assertThat(s.result()).isEqualTo(Ansi.italic() + Ansi.underlinedOff() + Ansi.italicOff());
+    }
 }