kusalk commented on code in PR #1243:
URL: https://github.com/apache/struts/pull/1243#discussion_r1983334831


##########
core/src/test/java/org/apache/struts2/interceptor/parameter/StrutsParameterAnnotationTest.java:
##########
@@ -489,14 +487,14 @@ public Map<String, Pojo> getPublicPojoMapDepthTwo() {
         }
     }
 
-    static class ModelAction implements ModelDriven<Pojo> {
+    public static class ModelAction implements ModelDriven<Pojo> {

Review Comment:
   The visibility of the Action class or any of its superclasses can affect the 
result of `Introspector#getBeanInfo`. This is why our unit tests didn't 
previously catch the issue observed by the reporter.
   
   The new solution bypasses the `Introspector` code entirely for `ModelDriven` 
Actions now anyway.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to