> 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