You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "SeasonPanPan (via GitHub)" <gi...@apache.org> on 2023/02/23 12:34:48 UTC
[GitHub] [commons-lang] SeasonPanPan opened a new pull request, #1033: Fix: [LANG-1694]'MethodUtils.getMatchingMethod' fails with "Found multiple candidates"
SeasonPanPan opened a new pull request, #1033:
URL: https://github.com/apache/commons-lang/pull/1033
JIRA: [https://issues.apache.org/jira/browse/LANG-1694](https://issues.apache.org/jira/browse/LANG-1694)
MethodUtils.getMatchingMethod() fails with "Found multiple candidates for method" message if the method is **an override of an abstract method** and parameter types do not exactly match the declared types.
The behavior of this method has changed between versions 3.11 and 3.12.0. Previously, the method would pick the first method that matches the signature and replace it only if a method with a more specific signature is found. Maybe it's a compatibility issue.
Fix: Keep only one candidate to return directly, if there are more than two candidates, continue to compare whether the declaration classes of the first two methods are the same, if not, also consider the first method the best match(Class itself).
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] codecov-commenter commented on pull request #1033: Fix: [LANG-1694]'MethodUtils.getMatchingMethod' fails with "Found multiple candidates"
Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #1033:
URL: https://github.com/apache/commons-lang/pull/1033#issuecomment-1441709626
# [Codecov](https://codecov.io/gh/apache/commons-lang/pull/1033?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#1033](https://codecov.io/gh/apache/commons-lang/pull/1033?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (acd978c) into [master](https://codecov.io/gh/apache/commons-lang/commit/469013a4f5a5cb666b35d72122690bb7f355c0b5?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (469013a) will **increase** coverage by `0.00%`.
> The diff coverage is `100.00%`.
```diff
@@ Coverage Diff @@
## master #1033 +/- ##
=========================================
Coverage 92.07% 92.07%
- Complexity 7477 7478 +1
=========================================
Files 194 194
Lines 15716 15717 +1
Branches 2897 2897
=========================================
+ Hits 14470 14471 +1
Misses 674 674
Partials 572 572
```
| [Impacted Files](https://codecov.io/gh/apache/commons-lang/pull/1033?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
|---|---|---|
| [.../org/apache/commons/lang3/reflect/MethodUtils.java](https://codecov.io/gh/apache/commons-lang/pull/1033?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2NvbW1vbnMvbGFuZzMvcmVmbGVjdC9NZXRob2RVdGlscy5qYXZh) | `90.28% <100.00%> (+0.03%)` | :arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
--
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: notifications-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] SeasonPanPan commented on pull request #1033: Fix: [LANG-1694]'MethodUtils.getMatchingMethod' fails with "Found multiple candidates"
Posted by "SeasonPanPan (via GitHub)" <gi...@apache.org>.
SeasonPanPan commented on PR #1033:
URL: https://github.com/apache/commons-lang/pull/1033#issuecomment-1448336367
> I'll take a look this weekend.
Thank u.
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] garydgregory commented on pull request #1033: Fix: [LANG-1694]'MethodUtils.getMatchingMethod' fails with "Found multiple candidates"
Posted by "garydgregory (via GitHub)" <gi...@apache.org>.
garydgregory commented on PR #1033:
URL: https://github.com/apache/commons-lang/pull/1033#issuecomment-1448326340
I'll take a look this weekend.
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] SeasonPanPan commented on pull request #1033: Fix: [LANG-1694]'MethodUtils.getMatchingMethod' fails with "Found multiple candidates"
Posted by "SeasonPanPan (via GitHub)" <gi...@apache.org>.
SeasonPanPan commented on PR #1033:
URL: https://github.com/apache/commons-lang/pull/1033#issuecomment-1448235790
@garydgregory @esivakumar18 @kinow Do you have time to review my code?
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-lang] garydgregory merged pull request #1033: [LANG-1694] MethodUtils.getMatchingMethod() fails with "Found multiple candidates"
Posted by "garydgregory (via GitHub)" <gi...@apache.org>.
garydgregory merged PR #1033:
URL: https://github.com/apache/commons-lang/pull/1033
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org