Merge pull request #13 from QuickWrite/develop

Version `0.2.3-alpha`

Author: QuickWrite

.github/workflows/codeql.yml

@@ -61,7 +61,7 @@ jobs:
         distribution: adopt
 
     - name: Build with Maven
-      run: mvn --batch-mode --update-snapshots verify
+      run: mvn -B package --file pom.xml
 
     - name: Perform CodeQL Analysis
       uses: github/codeql-action/analyze@v2

.github/workflows/maven-deploy.yml

@@ -16,8 +16,10 @@ jobs:
           server-id: cloudsmith
           server-username: MAVEN_USERNAME
           server-password: MAVEN_PASSWORD
+      - name: Build with Maven
+        run: mvn -B package --file pom.xml
       - name: Publish package
-        run: mvn --batch-mode deploy
+        run: mvn deploy
         env:
           MAVEN_USERNAME: ${{ secrets.CLOUDSMITH_USERNAME }}
           MAVEN_PASSWORD: ${{ secrets.CLOUDSMITH_TOKEN }}

.github/workflows/maven.yml

@@ -26,7 +26,7 @@ jobs:
           java-version: '16'
           distribution: adopt
       - name: Build with Maven
-        run: mvn --batch-mode --update-snapshots verify
+        run: mvn -B package --file pom.xml
       - name: Create necessary directories
         run: mkdir staging && cp target/*.jar staging
       - name: Upload artifact

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>net.quickwrite</groupId>
     <artifactId>fluent4j</artifactId>
-    <version>0.2.2-alpha</version>
+    <version>0.2.3-alpha</version>
     <build>
         <plugins>
             <plugin>

src/main/java/net/quickwrite/fluent4j/ast/FluentBase.java

@@ -89,15 +89,15 @@ public String getIdentifier() {
     }
 
     @Override
-    public CharSequence getResult(final AccessorBundle bundle) {
+    public CharSequence getResult(final AccessorBundle bundle, final int recursionDepth) {
         if (this.fluentElements.size() == 1) {
-            return this.fluentElements.get(0).getResult(bundle);
+            return this.fluentElements.get(0).getResult(bundle, recursionDepth - 1);
         }
 
         final StringBuilder builder = new StringBuilder();
 
         for (final FluentElement element : this.fluentElements) {
-            builder.append(element.getResult(bundle));
+            builder.append(element.getResult(bundle, recursionDepth - 1));
         }
 
         return builder;

src/main/java/net/quickwrite/fluent4j/ast/FluentElement.java

@@ -42,7 +42,8 @@ public interface FluentElement {
      * it has been called from.
      *
      * @param bundle The bundle that is getting passed down
+     * @param recursionDepth The amount of recursive calls that can still be made
      * @return The {@link CharSequence} value of the argument with the parameters
      */
-    CharSequence getResult(final AccessorBundle bundle);
+    CharSequence getResult(final AccessorBundle bundle, final int recursionDepth);
 }

src/main/java/net/quickwrite/fluent4j/ast/FluentTextElement.java

@@ -82,7 +82,7 @@ public String stringValue() {
     }
 
     @Override
-    public CharSequence getResult(AccessorBundle bundle) {
+    public CharSequence getResult(final AccessorBundle bundle, final int recursionDepth) {
         return this.text;
     }
 

src/main/java/net/quickwrite/fluent4j/ast/FluentVariant.java

@@ -53,8 +53,8 @@ public String stringValue() {
         return identifier.stringValue();
     }
 
-    public CharSequence getResult(AccessorBundle bundle) {
-        return this.content.getResult(bundle);
+    public CharSequence getResult(final AccessorBundle bundle, final int recursionDepth) {
+        return this.content.getResult(bundle, recursionDepth - 1);
     }
 
     @Override

src/main/java/net/quickwrite/fluent4j/ast/placeable/AttributeReference.java

@@ -6,6 +6,7 @@
 import net.quickwrite.fluent4j.ast.placeable.base.FluentArgumentResult;
 import net.quickwrite.fluent4j.ast.placeable.base.FluentPlaceable;
 import net.quickwrite.fluent4j.ast.placeable.base.FluentSelectable;
+import net.quickwrite.fluent4j.exception.RecursionDepthReachedException;
 import net.quickwrite.fluent4j.util.StringSlice;
 import net.quickwrite.fluent4j.util.args.FluentArgs;
 import net.quickwrite.fluent4j.util.bundle.DirectFluentBundle;
@@ -42,7 +43,7 @@ public String stringValue() {
     }
 
     @Override
-    public CharSequence getResult(final AccessorBundle bundle) {
+    public CharSequence getResult(final AccessorBundle bundle, int recursionDepth) {
         final FluentMessage fluentMessage = this.getMessage(bundle.getBundle(), reference.stringValue());
         if (fluentMessage == null) {
             return getErrorString();
@@ -54,11 +55,11 @@ public CharSequence getResult(final AccessorBundle bundle) {
             return getErrorString();
         }
 
-        return attribute.getResult(bundle);
+        return attribute.getResult(bundle, recursionDepth - 1);
     }
 
     @Override
-    public FluentElement getArgumentResult(final AccessorBundle bundle) {
+    public FluentElement getArgumentResult(final AccessorBundle bundle, int recursionDepth) {
         final FluentMessage fluentMessage = this.getMessage(bundle.getBundle(), reference.stringValue());
         if (fluentMessage == null) {
             return this;
@@ -70,12 +71,10 @@ public FluentElement getArgumentResult(final AccessorBundle bundle) {
             return this;
         }
 
-        if (bundle.getAccessedStorage().alreadyAccessed(attribute)) {
-            return this;
+        if (recursionDepth <= 0) {
+            throw new RecursionDepthReachedException();
         }
 
-        bundle.getAccessedStorage().addElement(attribute);
-
         final List<FluentElement> elementList = attribute.getElements();
 
         if (elementList.size() != 1) {

src/main/java/net/quickwrite/fluent4j/ast/placeable/FunctionReference.java

@@ -20,13 +20,17 @@ public FunctionReference(final String functionName, final FluentArgs arguments)
     }
 
     @Override
-    public FluentElement getArgumentResult(final AccessorBundle bundle) {
+    public FluentElement getArgumentResult(final AccessorBundle bundle, int recursionDepth) {
         try {
             return bundle
                     .getBundle()
                     .getFunction(this.functionName)
                     .orElseThrow()
-                    .getResult(bundle, (FunctionFluentArgs) this.getArguments(bundle));
+                    .getFunctionResult(
+                            bundle,
+                            (FunctionFluentArgs) this.getArguments(bundle, recursionDepth),
+                            recursionDepth - 1
+                    );
         } catch (final Exception exception) {
             return new StringLiteral("{" + functionName + "()}");
         }
@@ -70,11 +74,12 @@ protected boolean check(String string) {
      * this function will return <code>{NUMBER()}</code>.
      *
      *
-     * @param bundle@return The result of the function with the specific parameters
+     * @param bundle @return The result of the function with the specific parameters
+     * @param recursionDepth The amount of recursive calls that can be made
      */
     @Override
-    public CharSequence getResult(AccessorBundle bundle) {
-            return this.getArgumentResult(bundle).getResult(bundle);
+    public CharSequence getResult(AccessorBundle bundle, final int recursionDepth) {
+            return this.getArgumentResult(bundle, recursionDepth - 1).getResult(bundle, recursionDepth - 1);
     }
 
     @Override