Remove deprecated PropertyUtil.writeProperty method (#125)

* refactor: Remove deprecated PropertyUtil.writeProperty method (#123)

- Removed deprecated writeProperty() method from PropertyUtil
- Updated all test usages to use setProperty() instead
- Cleaned up unused imports and formatting from pre-commit

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Prepare release

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: cuioss-oliver

.github/project.yml

@@ -2,5 +2,5 @@ name: cui-java-tools
 pages-reference: cui-java-tools
 sonar-project-key: cuioss_cui-java-tools
 release:
-  current-version: 2.5.1
-  next-version: 2.5-SNAPSHOT
+  current-version: 2.6.0
+  next-version: 2.6-SNAPSHOT

src/main/java/de/cuioss/tools/logging/LogLevel.java

@@ -19,7 +19,6 @@
 import de.cuioss.tools.reflect.MoreReflection;
 import lombok.AccessLevel;
 import lombok.Getter;
-import lombok.NonNull;
 import lombok.RequiredArgsConstructor;
 import org.jspecify.annotations.Nullable;
 

src/main/java/de/cuioss/tools/logging/LogRecordModel.java

@@ -172,7 +172,7 @@ protected String getParsedMessageTemplate() {
     @Override
     public String format(Object... parameter) {
         return resolveIdentifierString() + AFTER_PREFIX +
-            lenientFormat(getParsedMessageTemplate(), parameter);
+                lenientFormat(getParsedMessageTemplate(), parameter);
     }
 
     @Override

src/main/java/de/cuioss/tools/property/PropertyUtil.java

@@ -146,23 +146,6 @@ public static void setProperty(Object bean, String propertyName, Object property
         writePropertyWithChaining(bean, propertyName, propertyValue);
     }
 
-    /**
-     * Writes a value to a property of a bean using reflection and returns the bean for method chaining.
-     * This method violates Command-Query Separation by both modifying state and returning a value.
-     * Consider using {@link #setProperty(Object, String, Object)} for pure command operations.
-     *
-     * @param bean          the bean to write to, must not be null
-     * @param propertyName  the name of the property to write, must not be null or empty
-     * @param propertyValue the value to write to the property
-     * @return the bean instance (for method chaining)
-     * @throws IllegalArgumentException if the property cannot be written or does not exist
-     * @since 2.0
-     * @deprecated Use {@link #setProperty(Object, String, Object)} for pure command operations
-     */
-    @Deprecated(since = "2.4.1", forRemoval = true)
-    public static Object writeProperty(Object bean, String propertyName, Object propertyValue) {
-        return writePropertyWithChaining(bean, propertyName, propertyValue);
-    }
 
     /**
      * Internal method for property writing with return value support.

src/test/java/de/cuioss/tools/property/PropertyUtilTest.java

@@ -42,14 +42,14 @@ void shouldReadWriteHappyCase() {
         assertNotNull(readProperty(underTest, ATTRIBUTE_DEFAULT_VALUE));
 
         Integer number = 4;
-        assertNotNull(writeProperty(underTest, ATTRIBUTE_READ_WRITE, number));
+        setProperty(underTest, ATTRIBUTE_READ_WRITE, number);
         assertEquals(number, readProperty(underTest, ATTRIBUTE_READ_WRITE));
 
         assertNull(readProperty(underTest, ATTRIBUTE_READ_WRITE_WITH_BUILDER));
-        assertNotNull(writeProperty(underTest, ATTRIBUTE_READ_WRITE_WITH_BUILDER, number));
+        setProperty(underTest, ATTRIBUTE_READ_WRITE_WITH_BUILDER, number);
         assertEquals(number, readProperty(underTest, ATTRIBUTE_READ_WRITE));
 
-        assertNotNull(writeProperty(underTest, ATTRIBUTE_READ_WRITE, null));
+        setProperty(underTest, ATTRIBUTE_READ_WRITE, null);
         assertNull(readProperty(underTest, ATTRIBUTE_READ_WRITE));
     }
 
@@ -60,15 +60,15 @@ void shouldHandleOverloadedMethods() {
         assertNull(readProperty(underTest, PROPERTY_NAME));
 
         Integer number = 4;
-        assertNotNull(writeProperty(underTest, PROPERTY_NAME, number));
+        setProperty(underTest, PROPERTY_NAME, number);
         assertEquals(number, readProperty(underTest, PROPERTY_NAME));
 
-        assertNotNull(writeProperty(underTest, PROPERTY_NAME, "5"));
+        setProperty(underTest, PROPERTY_NAME, "5");
         assertEquals(5, readProperty(underTest, PROPERTY_NAME));
 
         var propertyValue = new ArrayList<>();
         assertThrows(IllegalArgumentException.class, () ->
-                writeProperty(underTest, PROPERTY_NAME, propertyValue));
+                setProperty(underTest, PROPERTY_NAME, propertyValue));
     }
 
     @Test
@@ -79,8 +79,8 @@ void shouldHandleMethodsWithPrimitives() {
         assertEquals(0, readProperty(underTest, PROPERTY_PRIMITIVE_NAME));
 
         Integer number = 4;
-        assertNotNull(writeProperty(underTest, PROPERTY_NAME, number));
-        assertNotNull(writeProperty(underTest, PROPERTY_PRIMITIVE_NAME, number));
+        setProperty(underTest, PROPERTY_NAME, number);
+        setProperty(underTest, PROPERTY_PRIMITIVE_NAME, number);
         assertEquals(4, readProperty(underTest, PROPERTY_PRIMITIVE_NAME));
 
     }
@@ -93,9 +93,9 @@ void shouldFailOnInvalidProperty() {
         assertThrows(IllegalArgumentException.class, () ->
                 readProperty(underTest, ATTRIBUTE_WRITE_ONLY));
         assertThrows(IllegalArgumentException.class, () ->
-                writeProperty(underTest, ATTRIBUTE_NOT_ACCESSIBLE, ""));
+                setProperty(underTest, ATTRIBUTE_NOT_ACCESSIBLE, ""));
         assertThrows(IllegalArgumentException.class, () ->
-                writeProperty(underTest, ATTRIBUTE_READ_ONLY, ""));
+                setProperty(underTest, ATTRIBUTE_READ_ONLY, ""));
     }
 
     @Test
@@ -106,7 +106,7 @@ void shouldFailOnOnRuntimeExceptionProperty() {
         assertThrows(IllegalStateException.class, () ->
                 readProperty(underTest, PROPERTY_NAME));
         assertThrows(IllegalStateException.class, () ->
-                writeProperty(underTest, PROPERTY_NAME, ""));
+                setProperty(underTest, PROPERTY_NAME, ""));
     }
 
     @Test
@@ -117,7 +117,7 @@ void shouldBeFlexibleRegardingCaseSensitivity() {
         assertNull(readProperty(underTest, name));
 
         var value = Generators.nonEmptyStrings().next();
-        writeProperty(underTest, name, value);
+        setProperty(underTest, name, value);
         assertEquals(value, readProperty(underTest, name));
     }