Skip to content

Method as variable inside class fail to count parameters #18438

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
Closed
Polandia94 opened this issue Jan 10, 2025 · 0 comments · Fixed by #19233
Closed

Method as variable inside class fail to count parameters #18438

Polandia94 opened this issue Jan 10, 2025 · 0 comments · Fixed by #19233
Labels
bug mypy got something wrong

Comments

@Polandia94
Copy link

Bug Report

if a varible inside a class is a method of other class, when you call it has problems counting the parameters

(A clear and concise description of what the bug is.)

To Reproduce

https://mypy-play.net/?mypy=latest&python=3.12&gist=75f641be99916a76f9c9113fe99cd78d

Expected Behavior

The program run correctly ( on runtime or in pyright works fine)

Actual Behavior

mypy counts the second method as another self.
@Polandia94 Polandia94 added the bug mypy got something wrong label Jan 10, 2025
@Polandia94 Polandia94 mentioned this issue Jan 10, 2025
Closed
@ilevkivskyi ilevkivskyi mentioned this issue Jun 4, 2025
Merged
JukkaL pushed a commit that referenced this issue Jun 11, 2025
@ilevkivskyi @JukkaL
Handle assignment of bound methods in class bodies (#19233)
c20fd78 
Fixes #18438
Fixes #19146

Surprisingly, a very small change is sufficient to replicate Python
runtime behavior for all the important cases (see `checkmember.py`). I
also replace the `bound_args` argument of `CallableType`, that was
mostly unused, with a flag (as suggested by @JukkaL) and make sure it is
properly set/preserved everywhere.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug mypy got something wrong
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Handle assignment of bound methods in class bodies ilevkivskyi/mypy
1 participant
@Polandia94