protected-access and @classmethod cls

[alvarocaceres]

Steps to reproduce

class C(object):
  _Foo = 12

  @classmethod
  def Enable(cls):
    x = cls()
    print x._Foo  # protected-access warning

Current behavior

W: 7,10: Access to a protected member _Foo of a client class (protected-access)

Expected behavior

Seems like this should be allowed, for consistency with regular methods

pylint --version output

No config file found, using default configuration
pylint 2.0.0, 
astroid 1.5.0
Python 2.7.6 (default, Jun 22 2015, 17:58:13) 
[GCC 4.8.2]

[Paebbels]

I'm using Landscape.io, which uses pylint to check my Python code. I reported there another example for a falsely reported access to a protected member. See Landscape falsely reports an access to protected members for more details.

Note my presented example uses type hints, which are not evaluated by pylint as it seems.

[mkielg11]

I get this error with pylint version 2.5.3 on Python 3.8.0, is this in the pipeline of getting fixed?

[hippo91]

@mkielg11 can you give us a snippet of your issue please?
Thanks.

[mkielg11]

@mkielg11 can you give us a snippet of your issue please?
Thanks.

Well, yes, sorry @hippo91 for long reply time. The example is, however, exactly as above:

class TestClass:
    _test_var = 0

    @classmethod
    def test_class_method(cls):
        instance = cls()
        instance._test_var = 40
        return instance

Result:
W0212: Access to a protected member _test_var of a client class (protected-access)

> pylint --version
pylint 2.5.3
astroid 2.4.2
Python 3.8.5

[hippo91]

@mkielg11 thanks for your snippet. I can reproduce it. Unfortunately nobody has been interested in fixing this bug.
What about trying to fix this?

[mkielg11]

@mkielg11 thanks for your snippet. I can reproduce it. Unfortunately nobody has been interested in fixing this bug.
What about trying to fix this?

You mean that I could propose a fix? I guess I may be able to, however, I have no insight in the pylint source code, so I think I'll pass on that.

[irgeek]

This is one bug that irks me enough I'd like to have a crack at solving it. I haven't worked with the codebase before, though, so I'm not sure where to start. Is there a getting-started guide or something similar that would help me in figuring out where to start fixing this?

[Pierre-Sassoulas]

Sure thing @irgeek thank you for willing to contribute, I assigned you to the issue. You can find the developper doc here: http://pylint.pycqa.org/en/latest/development_guide/contribute.html