version 1.21.0

jmgrassau · jmgrassau · commit 3cd2f6e8ee54 · 2024-12-16T09:22:31.000+01:00
diff --git a/com.sap.adt.abapcleaner.app/abapcleaner.product b/com.sap.adt.abapcleaner.app/abapcleaner.product
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <?pde version="3.5"?>
 
-<product name="ABAP cleaner" uid="com.sap.adt.abapcleaner.app" application="com.sap.adt.abapcleaner.standalone.app" version="1.20.2.qualifier" type="features" includeLaunchers="true" autoIncludeRequirements="true">
+<product name="ABAP cleaner" uid="com.sap.adt.abapcleaner.app" application="com.sap.adt.abapcleaner.standalone.app" version="1.21.0" type="features" includeLaunchers="true" autoIncludeRequirements="true">
 
    <configIni use="default">
    </configIni>
diff --git a/com.sap.adt.abapcleaner.app/pom.xml b/com.sap.adt.abapcleaner.app/pom.xml
@@ -3,14 +3,14 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xs
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.sap.adt.abapcleaner</groupId>
   <artifactId>com.sap.adt.abapcleaner.app</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>com.sap.adt.abapcleaner.app</name>
   <packaging>eclipse-repository</packaging>
 
   <parent>
     <artifactId>parent</artifactId>
     <groupId>com.sap.adt.abapcleaner</groupId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
diff --git a/com.sap.adt.abapcleaner.feature/feature.xml b/com.sap.adt.abapcleaner.feature/feature.xml
@@ -2,7 +2,7 @@
 <feature
       id="com.sap.adt.abapcleaner.feature"
       label="ABAP cleaner"
-      version="1.20.2.qualifier">
+      version="1.21.0">
 
    <description>
       ABAP cleaner plug-in for ABAP Development Tools
diff --git a/com.sap.adt.abapcleaner.feature/pom.xml b/com.sap.adt.abapcleaner.feature/pom.xml
@@ -3,14 +3,14 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xs
   <modelVersion>4.0.0</modelVersion>
   <artifactId>com.sap.adt.abapcleaner.feature</artifactId>
   <groupId>com.sap.adt.abapcleaner</groupId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>com.sap.adt.abapcleaner.feature</name>
   <packaging>eclipse-feature</packaging>
 
   <parent>
     <artifactId>parent</artifactId>
     <groupId>com.sap.adt.abapcleaner</groupId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
diff --git a/com.sap.adt.abapcleaner.gui/META-INF/MANIFEST.MF b/com.sap.adt.abapcleaner.gui/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %Bundle-Name
 Bundle-SymbolicName: com.sap.adt.abapcleaner.gui;singleton:=true
-Bundle-Version: 1.20.2.qualifier
+Bundle-Version: 1.21.0
 Bundle-Vendor: %Provider-Name
 Bundle-Localization: plugin
 Bundle-RequiredExecutionEnvironment: JavaSE-17
diff --git a/com.sap.adt.abapcleaner.gui/pom.xml b/com.sap.adt.abapcleaner.gui/pom.xml
@@ -3,14 +3,14 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xs
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.sap.adt.abapcleaner</groupId>
   <artifactId>com.sap.adt.abapcleaner.gui</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>com.sap.adt.abapcleaner.gui</name>
   <packaging>eclipse-plugin</packaging>
 
   <parent>
     <artifactId>parent</artifactId>
     <groupId>com.sap.adt.abapcleaner</groupId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
diff --git a/com.sap.adt.abapcleaner.updatesite/pom.xml b/com.sap.adt.abapcleaner.updatesite/pom.xml
@@ -2,14 +2,14 @@
 xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://maven.apache.org/POM/4.0.0">
   <modelVersion>4.0.0</modelVersion>
   <artifactId>com.sap.adt.abapcleaner.updatesite</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>ABAP Cleaner for ABAP Development Tools (ADT)</name>
   <packaging>eclipse-repository</packaging>
 
   <parent>
     <artifactId>parent</artifactId>
     <groupId>com.sap.adt.abapcleaner</groupId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
diff --git a/com.sap.adt.abapcleaner/META-INF/MANIFEST.MF b/com.sap.adt.abapcleaner/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %Bundle-Name
 Bundle-SymbolicName: com.sap.adt.abapcleaner;singleton:=true
-Bundle-Version: 1.20.2.qualifier
+Bundle-Version: 1.21.0
 Bundle-Vendor: %Provider-Name
 Bundle-Localization: plugin
 Bundle-RequiredExecutionEnvironment: JavaSE-17
diff --git a/com.sap.adt.abapcleaner/pom.xml b/com.sap.adt.abapcleaner/pom.xml
@@ -3,14 +3,14 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xs
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.sap.adt.abapcleaner</groupId>
   <artifactId>com.sap.adt.abapcleaner</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>com.sap.adt.abapcleaner</name>
   <packaging>eclipse-plugin</packaging>
 
   <parent>
     <artifactId>parent</artifactId>
     <groupId>com.sap.adt.abapcleaner</groupId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
diff --git a/com.sap.adt.abapcleaner/src/com/sap/adt/abapcleaner/programbase/Program.java b/com.sap.adt.abapcleaner/src/com/sap/adt/abapcleaner/programbase/Program.java
@@ -62,6 +62,7 @@ public static String getVersion() {
 	
 	public static Release[] getReleases() {
 		return new Release[] { 
+				Release.create("1.21.0", 2024, 12, 16), 
 				Release.create("1.20.0", 2024, 10, 19), 
 				Release.create("1.19.0", 2024,  9, 18), 
 				Release.create("1.18.0", 2024,  7,  8), 
diff --git a/docs/release-notes.md b/docs/release-notes.md
@@ -6,7 +6,30 @@ as well as bugfixes of reported issues, i.e. anything that enhances or changes t
 For a complete list of changes (including documentation, tests, refactoring etc.), please refer to 
 the list of [commits](../../../commits/main).
 
-## 2024-11-11 (version 1.21.1)
+## 2024-12-16 (version 1.21.0)
+
+Great **thanks** to [**bnichell**](https://github.com/bnichell), [**Ronson1909**](https://github.com/Ronson1909),
+ [**vonglan**](https://github.com/vonglan), [**StefanRutzmoser**](https://github.com/StefanRutzmoser),
+ [**blackfish5**](https://github.com/blackfish5), [**ConjuringCoffee**](https://github.com/ConjuringCoffee),
+ [**frostrubin**](https://github.com/frostrubin) and [**fabianlupa**](https://github.com/fabianlupa) for inspiring these improvements and fixes!
+
+* Enhanced rule '**Align CLEAR:, FREE:, SORT and CATCH**' for **CATCH** ([#376](../../../issues/376))
+* Enhanced rule '**Align parameters and components**' for **IMPORT** ... and **EXPORT** ... ([#378](../../../issues/378))
+* Enhanced rule '**Align parameters and components**' for **RAISE EVENT** ... EXPORTING ([#377](../../../issues/377))
+* Enhanced rule '**Align parameters and components**' etc. for lower **maximum line length** ([#362](../../../issues/362))
+* Enhanced rule '**Align METHODS ... FOR TESTING**' etc. for multiple **periods** ([#379](../../../issues/379))
+* Enhanced **command line** with **--simulate** parameter ([#264](../../../issues/264))
+* Enhanced **command line** with **--version** parameter ([#339](../../../issues/339))
+* Fixed rule '**Align logical expressions**' for **parenthesis at line start** ([#384](../../../issues/384))
+* Fixed rule '**Align parameters and components**' for **comment** inside **inner parenthesis** ([#380](../../../issues/380))
+* Fixed rule '**Replace CREATE OBJECT with NEW constructor**' for **target variable usage** in parameters ([#381](../../../issues/381))
+* Fixed rule '**Make implicit type explicit**' for making TYPE explicit, but not **LENGTH** ([#375](../../../issues/375))
+* Fixed **Parser** to **exclude** processing of **RAP behavior definitions**
+* Enhanced **README** with link to **Devtoberfest 2024 session**
+* Updated **CONTRIBUTING**.md, including section on **AI-generated code**
+* Updated **known CamelCase names** for CDS views and fields
+
+## 2024-11-11 (version 1.20.1)
 
 **Thank you** very much, [**openPhiL**](https://github.com/openPhiL) and
  [**ConjuringCoffee**](https://github.com/ConjuringCoffee), for your bug reports!
diff --git a/docs/rules.md b/docs/rules.md
@@ -1,6 +1,6 @@
 # Available Cleanup Rules
 
-ABAP cleaner offers 98 cleanup rules with a total of 462 configuration options:
+ABAP cleaner offers 98 cleanup rules with a total of 463 configuration options:
 
 ## Empty Lines
 
@@ -90,7 +90,7 @@ ABAP cleaner offers 98 cleanup rules with a total of 462 configuration options:
 * [Align declarations](rules/AlignDeclarationsRule.md)
 * [Align assignments to the same object](rules/AlignAssignmentsRule.md)
 * [Align keywords with second word of first line](rules/AlignWithSecondWordRule.md)
-* [Align CLEAR:, FREE: and SORT](rules/AlignClearFreeAndSortRule.md)
+* [Align CLEAR:, FREE:, SORT and CATCH](rules/AlignClearFreeAndSortRule.md)
 * [Align SELECT clauses](rules/AlignSelectClausesRule.md)
 * [Align SELECT ... FROM ... JOIN](rules/AlignSelectFromRule.md)
 * [Align SELECT lists](rules/AlignSelectListsRule.md)
diff --git a/docs/rules/AlignClearFreeAndSortRule.md b/docs/rules/AlignClearFreeAndSortRule.md
@@ -1,22 +1,23 @@
 [<-- previous rule](AlignWithSecondWordRule.md) | [overview](../rules.md) | [next rule -->](AlignSelectClausesRule.md)
 
-# Align CLEAR:, FREE: and SORT
+# Align CLEAR:, FREE:, SORT and CATCH
 
-Aligns lists of variables after CLEAR: and FREE: and lists of components after SORT ... BY.
+Aligns lists of variables after CLEAR: and FREE:, lists of components after SORT ... BY, and lists of exception classes after CATCH.
 
 ## Options
 
 * Maximum line length \[120\] 
 * CLEAR: Use one line per variable: \[always\]
 * FREE: Use one line per variable: \[always\]
 * SORT: Use one line per variable: \[always\]
+* CATCH: Use one line per exception: \[always\]
 
 ## Examples
 
 
 ```ABAP
 
-  METHOD align_clear_free_and_sort.
+  METHOD align_clear_free_sort_catch.
     CLEAR: mv_any_value,
       mv_other_value,
          ls_any_structure-any_component,
@@ -39,14 +40,24 @@ Aligns lists of variables after CLEAR: and FREE: and lists of components after S
      comp3 comp4 AS TEXT,
           mt_third_table BY  component1 AS TEXT
       component2   component3.
+
+    TRY.
+        any_method( ).
+
+      CATCH cx_any_exception  cx_other_exception
+        cx_third_exception INTO DATA(lx_exception) ##NO_HANDLER.
+
+      CATCH cx_fourth_exception cx_fifth_exception   cx_sixth_exception cx_seventh_exception cx_eighth_exception  cx_ninth_exception INTO lx_exception.
+        " handle exceptions
+    ENDTRY.
   ENDMETHOD.
 ```
 
 Resulting code:
 
 ```ABAP
 
-  METHOD align_clear_free_and_sort.
+  METHOD align_clear_free_sort_catch.
     CLEAR: mv_any_value,
            mv_other_value,
            ls_any_structure-any_component,
@@ -76,6 +87,22 @@ Resulting code:
           mt_third_table BY component1 AS TEXT
                             component2
                             component3.
+
+    TRY.
+        any_method( ).
+
+      CATCH cx_any_exception
+            cx_other_exception
+            cx_third_exception INTO DATA(lx_exception) ##NO_HANDLER.
+
+      CATCH cx_fourth_exception
+            cx_fifth_exception
+            cx_sixth_exception
+            cx_seventh_exception
+            cx_eighth_exception
+            cx_ninth_exception INTO lx_exception.
+        " handle exceptions
+    ENDTRY.
   ENDMETHOD.
 ```
 
diff --git a/docs/rules/CreateObjectRule.md b/docs/rules/CreateObjectRule.md
@@ -43,6 +43,11 @@ This rule is part of the **essential** profile, as it is explicitly demanded by
         cx_message     = 1
         others         = 2.
 
+    " evaluating the target variable as an actual parameter
+    " only works with CREATE OBJECT, which immediately assigns mo_target:
+    CREATE OBJECT mo_target
+      EXPORTING iv_value = mo_target->gc_static_value.
+
     " chains can only be processed if they are first unchained
     CREATE OBJECT: lx_message, lx_other_message.
 
@@ -77,6 +82,11 @@ Resulting code:
         cx_message     = 1
         others         = 2.
 
+    " evaluating the target variable as an actual parameter
+    " only works with CREATE OBJECT, which immediately assigns mo_target:
+    CREATE OBJECT mo_target
+      EXPORTING iv_value = mo_target->gc_static_value.
+
     " chains can only be processed if they are first unchained
     lx_message = NEW #( ).
     lx_other_message = NEW #( ).
diff --git a/pom.xml b/pom.xml
@@ -3,7 +3,7 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.sap.adt.abapcleaner</groupId>
   <artifactId>parent</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>ABAP cleaner</name>
   <packaging>pom</packaging>
   <modules>
diff --git a/test/com.sap.adt.abapcleaner.test/META-INF/MANIFEST.MF b/test/com.sap.adt.abapcleaner.test/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: Unit Tests for com.sap.adt.abapcleaner
 Bundle-SymbolicName: com.sap.adt.abapcleaner.test;singleton:=true
-Bundle-Version: 1.20.2.qualifier
+Bundle-Version: 1.21.0
 Bundle-Vendor: abap-dev
 Fragment-Host: com.sap.adt.abapcleaner
 Bundle-RequiredExecutionEnvironment: JavaSE-17
diff --git a/test/com.sap.adt.abapcleaner.test/pom.xml b/test/com.sap.adt.abapcleaner.test/pom.xml
@@ -3,14 +3,14 @@ xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xs
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.sap.adt.abapcleaner</groupId>
   <artifactId>com.sap.adt.abapcleaner.test</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>com.sap.adt.abapcleaner.test</name>
   <packaging>eclipse-test-plugin</packaging>
 
   <parent>
     <artifactId>tests</artifactId>
     <groupId>com.sap.adt.abapcleaner</groupId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
diff --git a/test/pom.xml b/test/pom.xml
@@ -4,14 +4,14 @@
   <modelVersion>4.0.0</modelVersion>
   <groupId>com.sap.adt.abapcleaner</groupId>
   <artifactId>tests</artifactId>
-  <version>1.20.2-SNAPSHOT</version>
+  <version>1.21.0</version>
   <name>Test Fragments</name>
   <packaging>pom</packaging>
 
   <parent>
     <groupId>com.sap.adt.abapcleaner</groupId>
     <artifactId>parent</artifactId>
-    <version>1.20.2-SNAPSHOT</version>
+    <version>1.21.0</version>
     <relativePath>../</relativePath>
   </parent>
 
