MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-09-27 10:03:11 +07:00
Code Issues Packages Projects Releases Wiki Activity

[python] PY-85031 Use PyAstNamedParameter.isSelf when reporting legacy positional-only parameters

Browse Source 
It correctly takes into account the first argument of `__new__` that is also implicit,
and thus should be excluded when reporting positional-only parameters following
non-positional-only ones.

GitOrigin-RevId: abf09c6be5e49c21c12264d22bf63a51018fdcd6
This commit is contained in:
Mikhail Golubev
2025-10-28 15:32:15 +00:00
committed by intellij-monorepo-bot
parent e7fedc4208
commit aea9e8688f
3 changed files with 3 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
python/python-psi-impl/src/com/jetbrains/python/psi/impl/PyFunctionImpl.java
-9
View File
@@ -790,15 +790,6 @@ public class PyFunctionImpl extends PyBaseElementImpl<PyFunctionStub> implements
return getStubOrPsiChild(PyStubElementTypes.ANNOTATION);
}
/**
* is `function` a method or a classmethod
*/
public static boolean isMethod(PyFunction function) {
final var isMethod = ScopeUtil.getScopeOwner(function) instanceof PyClass;
final var modifier = function.getModifier();
return (isMethod && modifier == null) || modifier == CLASSMETHOD;
}
/**
* @param self should be this
*/
python/python-psi-impl/src/com/jetbrains/python/validation/PyParameterListAnnotatorVisitor.java
+1 -3
View File
@@ -19,7 +19,6 @@ import com.intellij.lang.annotation.HighlightSeverity;
import com.jetbrains.python.PyPsiBundle;
import com.jetbrains.python.psi.*;
import com.jetbrains.python.psi.impl.ParamHelper;
import com.jetbrains.python.psi.impl.PyFunctionImpl;
import org.jetbrains.annotations.NotNull;
import java.util.HashSet;
@@ -38,7 +37,6 @@ public class PyParameterListAnnotatorVisitor extends PyElementVisitor {
@Override
public void visitPyParameterList(final @NotNull PyParameterList paramlist) {
final LanguageLevel languageLevel = LanguageLevel.forElement(paramlist);
final var hasImplicit = paramlist.getParent() instanceof PyFunction function && PyFunctionImpl.isMethod(function);
ParamHelper.walkDownParamArray(
paramlist.getParameters(),
@@ -55,7 +53,7 @@ public class PyParameterListAnnotatorVisitor extends PyElementVisitor {
@Override
public void visitNamedParameter(PyNamedParameter parameter, boolean first, boolean last) {
final var name = parameter.getName();
if (!hadKeyword && name != null && !(hasImplicit && first) && !isPrivate(name)) {
if (!hadKeyword && name != null && !parameter.isSelf() && !isPrivate(name)) {
hadKeyword = true;
}
else if (hadKeyword && !hadPositionalContainer && !hadSingleStar && name != null && !hadSlash && isPrivate(name)) {
python/testData/highlighting/historicalPositionalOnlyParameters.py
+2 -1
View File
@@ -15,4 +15,5 @@ class A:
@staticmethod
def s1(__b): pass
@staticmethod
def s1(a, <warning descr="Positional-only parameter follows parameter that is not positional-only">__b</warning>): pass
def s1(a, <warning descr="Positional-only parameter follows parameter that is not positional-only">__b</warning>): pass
def __new__(cls, __b): pass