Fixed getter name generator according to JavaBeans specification

Fixes #375
eclipse-esmf · Nov 2, 2023 · 0f429cc · 0f429cc
1 parent 6f5177a
commit 0f429cc
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 7 deletions.
diff --git a/core/esmf-aspect-model-java-generator/pom.xml b/core/esmf-aspect-model-java-generator/pom.xml
@@ -57,6 +57,10 @@
          <artifactId>record-builder-processor</artifactId>
          <scope>provided</scope>
       </dependency>
+      <dependency>
+         <groupId>org.apache.jena</groupId>
+         <artifactId>jena-core</artifactId>
+      </dependency>
 
       <!-- Test dependencies -->
       <dependency>

diff --git a/...l-java-generator/src/main/java/org/eclipse/esmf/aspectmodel/java/AspectModelJavaUtil.java b/...l-java-generator/src/main/java/org/eclipse/esmf/aspectmodel/java/AspectModelJavaUtil.java
@@ -622,4 +622,13 @@ public static AbstractEntity castToAbstractEntity( final ComplexType complexType
    public static Quantifiable castToQuantifiable( final Characteristic characteristic ) {
       return (Quantifiable) characteristic;
    }
+
+   public static String getterName( final Property property ) {
+      final boolean isBooleanType = !property.isOptional()
+            && property.getDataType()
+            .filter( Type::isScalar )
+            .map( type -> XSD.xboolean.getURI().equals( type.getUrn() ) )
+            .orElse( false );
+      return (isBooleanType ? "is" : "get") + StringUtils.capitalize( property.getPayloadName() );
+   }
 }
diff --git a/core/esmf-aspect-model-java-generator/src/main/resources/java-pojo-getter-lib.vm b/core/esmf-aspect-model-java-generator/src/main/resources/java-pojo-getter-lib.vm
@@ -10,26 +10,23 @@
  ~
  ~ SPDX-License-Identifier: MPL-2.0
  *#
-#macro( methodName )
-    #if( $property.getDataType().isPresent() )
-        #set( $type = $property.dataType.get() )
-        #if( $type.isScalar() && $type.getUrn().equals( $XSD.xboolean.getURI() ) )is#{else}get#{end}
-    #{else}get#{end}${StringUtils.capitalize( $property.getPayloadName() )}#{end}
 
 #macro( javaPojoGetter $property $propertyIndex )
 
+#set( $methodName = $util.getterName( $property ) )
+
 /**
 * Returns $property.getPreferredName( $localeEn )
 *
 * @return {@link #$property.getPayloadName()}
 */
 #if( $property.isAbstract() )
-    public abstract T${propertyIndex}  #methodName()();
+    public abstract T${propertyIndex}  $methodName();
 #else
     #set( $propertyType = $util.getPropertyType( $property, false, $codeGenerationConfig ) )
     #if( $property.getExtends().isPresent() )@Override #end
     #if( $util.isPropertyNotInPayload( $property, $codeGenerationConfig ) )@JsonIgnore #end
-public $propertyType #methodName()() {
+public $propertyType $methodName() {
     return this.$property.getPayloadName();
 }
 #end