> 2022年9月23日 14:51,Rémy Maucherat <r...@apache.org> 写道:
> 
> On Fri, Sep 23, 2022 at 5:43 AM Han Li <li...@apache.org> wrote:
>> 
>> 
>> 
>>> 2022年9月23日 05:17,r...@apache.org 写道:
>>> 
>>> This is an automated email from the ASF dual-hosted git repository.
>>> 
>>> remm pushed a commit to branch 9.0.x
>>> in repository https://gitbox.apache.org/repos/asf/tomcat.git
>>> 
>>> 
>>> The following commit(s) were added to refs/heads/9.0.x by this push:
>>>    new 7fda27272f Fix missing attributes
>>> 7fda27272f is described below
>>> 
>>> commit 7fda27272ff6cfb6ca920451a72f62e1c62a8ba3
>>> Author: remm <r...@apache.org>
>>> AuthorDate: Thu Sep 22 22:56:19 2022 +0200
>>> 
>>>   Fix missing attributes
>>> 
>>>   The code is very confusing, but the main attribute map can contain
>>>   either the get or set method as the value, so it must not be used. This
>>>   should probably be refactored some more ...
>> 
>> I will do this work today. ;)
> 
> I think it would be best to avoid any further unnecessary refactorings
> for now, until the next release is actually out :)
OK.

Han


> 
> Thanks !
> 
> Rémy
> 
>> Han
>> 
>>> ---
>>> .../modeler/modules/MbeansDescriptorsIntrospectionSource.java     | 2 +-
>>> webapps/docs/changelog.xml                                        | 8 
>>> ++++++++
>>> 2 files changed, 9 insertions(+), 1 deletion(-)
>>> 
>>> diff --git 
>>> a/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
>>>  
>>> b/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
>>> index 05f393d023..ffb0937066 100644
>>> --- 
>>> a/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
>>> +++ 
>>> b/java/org/apache/tomcat/util/modeler/modules/MbeansDescriptorsIntrospectionSource.java
>>> @@ -311,7 +311,7 @@ public class MbeansDescriptorsIntrospectionSource 
>>> extends ModelerSource
>>>                String name = attEntry.getKey();
>>>                AttributeInfo ai = new AttributeInfo();
>>>                ai.setName(name);
>>> -                Method gm = attEntry.getValue();
>>> +                Method gm = getAttMap.get(name);
>>>                if (gm != null) {
>>>                    ai.setGetMethod(gm.getName());
>>>                    Class<?> t = gm.getReturnType();
>>> diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
>>> index 7c79ceb65e..1d1a098173 100644
>>> --- a/webapps/docs/changelog.xml
>>> +++ b/webapps/docs/changelog.xml
>>> @@ -105,6 +105,14 @@
>>>  issues do not "pop up" wrt. others).
>>> -->
>>> <section name="Tomcat 9.0.67 (remm)" rtext="in development">
>>> +  <subsection name="Coyote">
>>> +    <changelog>
>>> +      <fix>
>>> +        Fix a regression in refactoring for Hashtables which caused mbeans 
>>> to
>>> +        lose many of their attributes. (remm)
>>> +      </fix>
>>> +    </changelog>
>>> +  </subsection>
>>> </section>
>>> <section name="Tomcat 9.0.66 (remm)" rtext="release in progress">
>>>  <subsection name="Catalina">
>>> 
>>> 
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>>> 
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
>> For additional commands, e-mail: dev-h...@tomcat.apache.org
>> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to