pyreverse: remove duplicated aggregations

[pylint-bot]

Originally reported by: Anonymous

---

i had some strange cases where i had two the same aggregations for a class. They should be removed.

---

• Bitbucket: https://bitbucket.org/logilab/pylint/issue/83

[pylint-bot]

Original comment by space one (BitBucket: spaceone, GitHub: @spaceone?):

---

if it helps: it happened when i had protected class members.
e.g. (untested)

#!python

class Foo(object):
    @property
    def bar(self):
        return self.__bar

    @bar.setter
    def bar(self, bar):
        self.__bar = bar

    def __init__(self):
        self.__bar = Bar()

[pylint-bot]

Original comment by Emile Anclin (BitBucket: anclin, GitHub: @Anclin?):

---

With the given example, I couldn't reproduce a behavior where the method "bar" appears two times in the diagram. This is true for the current state and earlier releases, going as far back as astng 0.24 / Pylint 0.26...

By default, Pyreverse calls for method research "node.values()" which returns the first node of each name in node.locals (I don't know why it is/should be the first one though).

I think, if there is a duplicated attribute, it must be in a more complicated situation...

[pylint-bot]

Original comment by Sylvain Thénault (BitBucket: sthenault, GitHub: @sthenault?):

---

invalid or deprecated. Feel free to reopen if you may provide a way to reproduce this with recent pylint version.

[VasLem]

I have a similar issue, using pylint 2.14.5 and Python 3.8 on this minimal example:

class Block:
    def __init__(self):
        self._cacheDir = None
    @property
    def cacheDir(self):
        if self._cacheDir is None:
            self.cacheDir = cacheDir
        return self._cacheDir

    @cacheDir.setter
    def cacheDir(self, value: str):
        self._cacheDir = value

I get this output:

Is there a way to stop the duplicates from showing up or is this the expected behavior?