Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[java] LooseCoupling must ignore class literals: ArrayList.class #2464

Closed
rxmicro opened this issue May 3, 2020 · 1 comment · Fixed by #3177
Closed

[java] LooseCoupling must ignore class literals: ArrayList.class #2464

rxmicro opened this issue May 3, 2020 · 1 comment · Fixed by #3177
Labels
a:false-positive PMD flags a piece of code that is not problematic
Milestone
7.0.0

Comments

@rxmicro
Copy link

rxmicro commented May 3, 2020

Affects PMD Version:

6.23.0.

Description:

LooseCoupling must ignore class literals. For Example ArrayList.class

Code Sample demonstrating the issue:

void someMethod(Object o){
       if ( o.getClass() == ArrayList.class ) { // This case must be ignored by rule!
             // do something
       }
}

Running PMD through: Maven
@rxmicro rxmicro added the a:bug PMD crashes or fails to analyse a file. label May 3, 2020
@rxmicro rxmicro changed the title java: LooseCoupling must ignore class literals: ArrayList.class [java] LooseCoupling must ignore class literals: ArrayList.class May 27, 2020
@adangel adangel mentioned this issue Oct 10, 2020
Closed
@oowekyala oowekyala added a:false-positive PMD flags a piece of code that is not problematic and removed a:bug PMD crashes or fails to analyse a file. labels Jan 17, 2021
oowekyala added a commit to oowekyala/pmd that referenced this issue Mar 28, 2021
@oowekyala
Ignore class literals (pmd#2464)
9f087b6
@oowekyala oowekyala mentioned this issue Mar 28, 2021
Merged
4 tasks
@adangel adangel added this to the 7.0.0 milestone Apr 1, 2021
@adangel adangel linked a pull request Apr 2, 2021 that will close this issue
[java] Update rule LooseCoupling #3177
Merged
4 tasks
@adangel adangel mentioned this issue Jan 23, 2023
Closed
55 tasks
@adangel
Copy link
Member

adangel commented Apr 22, 2023

This has been fixed with PMD 7.0.0-rc1.

@adangel adangel closed this as completed Apr 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
a:false-positive PMD flags a piece of code that is not problematic
Projects
None yet
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

[java] Update rule LooseCoupling oowekyala/pmd
3 participants
@adangel @oowekyala @rxmicro