Fix disabling of ungrouped-imports

[DanielNoord]

• Add yourself to CONTRIBUTORS if you are a new contributor.
• Add a ChangeLog entry describing what your PR does.
• If it's a new feature, or an important bug fix, add a What's New entry in
  doc/whatsnew/<current release.rst>.
• Write a good description on what the PR does.

Type of Changes

	Type
✓	🐛 Bug fix

Description

Needed for a PR that stops using the deprecated cachedproperty decorator from astroid.

[coveralls]

Pull Request Test Coverage Report for Build 1974038123

• 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 94.025%

---

Totals
Change from base Build 1974036227:	0.0%
Covered Lines:	15029
Relevant Lines:	15984

---

💛 - Coveralls

[Pierre-Sassoulas]

👍

[jacobtylerwalls]

Cool. So it looks like #2366 wasn't truly fixed, as the regression test added in #5063 didn't repro -- it just lacked the final line from the original report! Is it worth referencing the original issue in the changelog or not?

[DanielNoord]

@jacobtylerwalls I totally forgot I added that method. Anyway, I think that this solution is better (instead of using the method added in #5063). I think I did not fully look into ungrouped-imports when I did that issue and only focused on the message in the OP.

I could add the issue, but since it was "fixed" in 2.12 might be a bit weird. @Pierre-Sassoulas You decide 😄

[jacobtylerwalls]

I could add the issue, but since it was "fixed" in 2.12 might be a bit weird.

There are several cases where we just say "refers to" or "refs #..." right? best of both worlds.

[Pierre-Sassoulas]

I think we don't need to create issue when the problem is fixed (We can Refs #The PR so those wanting detail can find them).