Fix typo in exception message throws from

garydgregory · garydgregory · commit 2cdff8fa28f2 · 2025-07-20T16:45:09.000-04:00
org.apache.commons.beanutils2.PropertyUtilsBean.setMappedProperty(Object,
String, String, Object)
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -29,6 +29,8 @@
   <body>
     <release version="2.0.0-M3" date="YYYY-MM-DD" description="This is a major release and requires Java 8.">
       <!-- FIX -->
+      <action type="fix" dev="ggregory" due-to="Gary Gregory">Fix typo in log event message in org.apache.commons.beanutils2.MethodUtils.setMethodAccessible(Method).</action>
+      <action type="fix" dev="ggregory" due-to="Gary Gregory">Fix typo in exception message throws from org.apache.commons.beanutils2.PropertyUtilsBean.setMappedProperty(Object, String, String, Object).</action>
       <!-- ADD -->
       <!-- UPDATE -->
       <action type="update" dev="ggregory" due-to="Gary Gregory, Dependabot">Bump org.apache.commons:commons-lang3 from 3.17.0 to 3.18.0 #357.</action>
diff --git a/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java b/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java
@@ -390,7 +390,7 @@ public Object getIndexedProperty(final Object bean, final String name, final int
         // Otherwise, the underlying property must be an array
         final Method readMethod = getReadMethod(bean.getClass(), descriptor);
         if (readMethod == null) {
-            throw new NoSuchMethodException("Property '" + name + "' has no " + "getter method on bean class '" + bean.getClass() + "'");
+            throw new NoSuchMethodException("Property '" + name + "' has no getter method on bean class '" + bean.getClass() + "'");
         }
 
         // Call the property getter and return the value
@@ -792,7 +792,7 @@ protected Object getPropertyOfMapBean(final Map<?, ?> bean, String propertyName)
         }
 
         if (resolver.isIndexed(propertyName) || resolver.isMapped(propertyName)) {
-            throw new IllegalArgumentException("Indexed or mapped properties are not supported on" + " objects of type Map: " + propertyName);
+            throw new IllegalArgumentException("Indexed or mapped properties are not supported on objects of type Map: " + propertyName);
         }
 
         return bean.get(propertyName);
@@ -1399,7 +1399,7 @@ public void setMappedProperty(final Object bean, final String name, final String
             Method mappedWriteMethod = ((MappedPropertyDescriptor) descriptor).getMappedWriteMethod();
             mappedWriteMethod = MethodUtils.getAccessibleMethod(bean.getClass(), mappedWriteMethod);
             if (mappedWriteMethod == null) {
-                throw new NoSuchMethodException("Property '" + name + "' has no mapped setter method" + "on bean class '" + bean.getClass() + "'");
+                throw new NoSuchMethodException("Property '" + name + "' has no mapped setter method on bean class '" + bean.getClass() + "'");
             }
             if (LOG.isTraceEnabled()) {
                 final String valueClassName = value == null ? "<null>" : value.getClass().getName();
@@ -1537,7 +1537,7 @@ protected void setPropertyOfMapBean(final Map<String, Object> bean, String prope
         }
 
         if (resolver.isIndexed(propertyName) || resolver.isMapped(propertyName)) {
-            throw new IllegalArgumentException("Indexed or mapped properties are not supported on" + " objects of type Map: " + propertyName);
+            throw new IllegalArgumentException("Indexed or mapped properties are not supported on objects of type Map: " + propertyName);
         }
 
         bean.put(propertyName, value);

Original file line number	Diff line number	Diff line change
`@@ -390,7 +390,7 @@ public Object getIndexedProperty(final Object bean, final String name, final int`
`390`	`390`	`// Otherwise, the underlying property must be an array`
`391`	`391`	`final Method readMethod = getReadMethod(bean.getClass(), descriptor);`
`392`	`392`	`if (readMethod == null) {`
`393`		`- throw new NoSuchMethodException("Property '" + name + "' has no " + "getter method on bean class '" + bean.getClass() + "'");`
	`393`	`+ throw new NoSuchMethodException("Property '" + name + "' has no getter method on bean class '" + bean.getClass() + "'");`
`394`	`394`	`}`
`395`	`395`
`396`	`396`	`// Call the property getter and return the value`
`@@ -792,7 +792,7 @@ protected Object getPropertyOfMapBean(final Map<?, ?> bean, String propertyName)`
`792`	`792`	`}`
`793`	`793`
`794`	`794`	`if (resolver.isIndexed(propertyName) \|\| resolver.isMapped(propertyName)) {`
`795`		`- throw new IllegalArgumentException("Indexed or mapped properties are not supported on" + " objects of type Map: " + propertyName);`
	`795`	`+ throw new IllegalArgumentException("Indexed or mapped properties are not supported on objects of type Map: " + propertyName);`
`796`	`796`	`}`
`797`	`797`
`798`	`798`	`return bean.get(propertyName);`
`@@ -1399,7 +1399,7 @@ public void setMappedProperty(final Object bean, final String name, final String`
`1399`	`1399`	`Method mappedWriteMethod = ((MappedPropertyDescriptor) descriptor).getMappedWriteMethod();`
`1400`	`1400`	`mappedWriteMethod = MethodUtils.getAccessibleMethod(bean.getClass(), mappedWriteMethod);`
`1401`	`1401`	`if (mappedWriteMethod == null) {`
`1402`		`- throw new NoSuchMethodException("Property '" + name + "' has no mapped setter method" + "on bean class '" + bean.getClass() + "'");`
	`1402`	`+ throw new NoSuchMethodException("Property '" + name + "' has no mapped setter method on bean class '" + bean.getClass() + "'");`
`1403`	`1403`	`}`
`1404`	`1404`	`if (LOG.isTraceEnabled()) {`
`1405`	`1405`	`final String valueClassName = value == null ? "<null>" : value.getClass().getName();`
`@@ -1537,7 +1537,7 @@ protected void setPropertyOfMapBean(final Map<String, Object> bean, String prope`
`1537`	`1537`	`}`
`1538`	`1538`
`1539`	`1539`	`if (resolver.isIndexed(propertyName) \|\| resolver.isMapped(propertyName)) {`
`1540`		`- throw new IllegalArgumentException("Indexed or mapped properties are not supported on" + " objects of type Map: " + propertyName);`
	`1540`	`+ throw new IllegalArgumentException("Indexed or mapped properties are not supported on objects of type Map: " + propertyName);`
`1541`	`1541`	`}`
`1542`	`1542`
`1543`	`1543`	`bean.put(propertyName, value);`