Skip to content

Commit c080a8b

Browse files
committed
Replacing deprecated consulo.ide.impl.idea.openapi.util.text.StringUtil with consulo.util.lang.StringUtil. Some refactoring/localizing.
1 parent ae1bb77 commit c080a8b

File tree

8 files changed

+962
-1035
lines changed

8 files changed

+962
-1035
lines changed

python-impl/src/main/java/com/jetbrains/python/impl/buildout/config/psi/impl/BuildoutCfgValueLine.java

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -16,42 +16,42 @@
1616

1717
package com.jetbrains.python.impl.buildout.config.psi.impl;
1818

19-
import com.google.common.collect.Lists;
2019
import com.jetbrains.python.impl.buildout.config.psi.BuildoutPsiUtil;
2120
import com.jetbrains.python.impl.buildout.config.ref.BuildoutPartReference;
22-
import consulo.ide.impl.idea.openapi.util.text.StringUtil;
2321
import consulo.language.ast.ASTNode;
2422
import consulo.language.psi.PsiReference;
2523
import consulo.util.lang.Pair;
26-
24+
import consulo.util.lang.StringUtil;
2725
import jakarta.annotation.Nonnull;
26+
27+
import java.util.ArrayList;
2828
import java.util.List;
2929

3030
/**
3131
* @author traff
3232
*/
3333
public class BuildoutCfgValueLine extends BuildoutCfgPsiElement {
34-
public BuildoutCfgValueLine(@Nonnull final ASTNode node) {
35-
super(node);
36-
}
37-
38-
@Override
39-
public String toString() {
40-
return "BuildoutCfgValue:" + getNode().getElementType().toString();
41-
}
34+
public BuildoutCfgValueLine(@Nonnull final ASTNode node) {
35+
super(node);
36+
}
4237

43-
@Nonnull
44-
@Override
45-
public PsiReference[] getReferences() {
46-
if (BuildoutPsiUtil.isInBuildoutSection(this) && BuildoutPsiUtil.isAssignedTo(this, "parts")) {
47-
List<BuildoutPartReference> refs = Lists.newArrayList();
48-
List<Pair<String, Integer>> names = StringUtil.getWordsWithOffset(getText());
49-
for (Pair<String, Integer> name : names) {
50-
refs.add(new BuildoutPartReference(this, name.getFirst(), name.getSecond()));
51-
}
52-
return refs.toArray(new BuildoutPartReference[refs.size()]);
38+
@Override
39+
public String toString() {
40+
return "BuildoutCfgValue:" + getNode().getElementType().toString();
5341
}
5442

55-
return PsiReference.EMPTY_ARRAY;
56-
}
43+
@Nonnull
44+
@Override
45+
public PsiReference[] getReferences() {
46+
if (BuildoutPsiUtil.isInBuildoutSection(this) && BuildoutPsiUtil.isAssignedTo(this, "parts")) {
47+
List<BuildoutPartReference> refs = new ArrayList<>();
48+
List<Pair<String, Integer>> names = StringUtil.getWordsWithOffset(getText());
49+
for (Pair<String, Integer> name : names) {
50+
refs.add(new BuildoutPartReference(this, name.getFirst(), name.getSecond()));
51+
}
52+
return refs.toArray(new BuildoutPartReference[refs.size()]);
53+
}
54+
55+
return PsiReference.EMPTY_ARRAY;
56+
}
5757
}

python-impl/src/main/java/com/jetbrains/python/impl/codeInsight/imports/ImportCandidateHolder.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ else if (myImportable instanceof PyClass) {
151151
ContainerUtil.mapNotNull(((PyClass)myImportable).getSuperClasses(null), cls -> PyUtil.isObjectClass(cls) ? null : cls.getName());
152152
if (!supers.isEmpty()) {
153153
sb.append("(");
154-
consulo.ide.impl.idea.openapi.util.text.StringUtil.join(supers, ", ", sb);
154+
StringUtil.join(supers, ", ", sb);
155155
sb.append(")");
156156
}
157157
}

python-impl/src/main/java/com/jetbrains/python/impl/codeInsight/intentions/PyStringConcatenationToFormatIntention.java

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,21 +23,23 @@
2323
import com.jetbrains.python.psi.*;
2424
import com.jetbrains.python.psi.types.PyType;
2525
import com.jetbrains.python.psi.types.TypeEvalContext;
26+
import consulo.annotation.access.RequiredWriteAction;
2627
import consulo.codeEditor.Editor;
27-
import consulo.ide.impl.idea.util.NotNullFunction;
2828
import consulo.language.psi.PsiElement;
2929
import consulo.language.psi.PsiFile;
3030
import consulo.language.psi.util.PsiTreeUtil;
3131
import consulo.language.util.IncorrectOperationException;
3232
import consulo.project.Project;
3333
import consulo.python.impl.localize.PyLocalize;
34+
import consulo.util.lang.Couple;
3435
import consulo.util.lang.Pair;
3536
import consulo.util.lang.StringUtil;
3637
import jakarta.annotation.Nonnull;
3738

3839
import java.util.ArrayList;
3940
import java.util.Collection;
4041
import java.util.List;
42+
import java.util.function.Function;
4143

4244
/**
4345
* @author Alexey.Ivanov
@@ -126,6 +128,8 @@ private static Collection<PyElementType> getOperators(@Nonnull PyBinaryExpressio
126128
return res;
127129
}
128130

131+
@Override
132+
@RequiredWriteAction
129133
public void doInvoke(@Nonnull Project project, Editor editor, PsiFile file) throws IncorrectOperationException {
130134
PsiElement element =
131135
PsiTreeUtil.getTopmostParentOfType(file.findElementAt(editor.getCaretModel().getOffset()), PyBinaryExpression.class);
@@ -136,18 +140,18 @@ public void doInvoke(@Nonnull Project project, Editor editor, PsiFile file) thro
136140
final LanguageLevel languageLevel = LanguageLevel.forElement(element);
137141
final boolean useFormatMethod = languageLevel.isAtLeast(LanguageLevel.PYTHON27);
138142

139-
NotNullFunction<String, String> escaper = consulo.ide.impl.idea.openapi.util.text.StringUtil.escaper(false, "\"\'\\");
143+
Function<String, String> escaper = StringUtil.escaper(false, "\"\'\\");
140144
StringBuilder stringLiteral = new StringBuilder();
141145
List<String> parameters = new ArrayList<>();
142-
Pair<String, String> quotes = Pair.create("\"", "\"");
146+
Pair<String, String> quotes = Couple.of("\"", "\"");
143147
boolean quotesDetected = false;
144148
final TypeEvalContext context = TypeEvalContext.userInitiated(file.getProject(), file);
145149
int paramCount = 0;
146150
boolean isUnicode = false;
147151
final PyClassTypeImpl unicodeType = PyBuiltinCache.getInstance(element).getObjectType("unicode");
148152

149153
for (PyExpression expression : getSimpleExpressions((PyBinaryExpression) element)) {
150-
if (expression instanceof PyStringLiteralExpression) {
154+
if (expression instanceof PyStringLiteralExpression stringLiteral1) {
151155
final PyType type = context.getType(expression);
152156
if (type != null && type.equals(unicodeType)) {
153157
isUnicode = true;
@@ -156,7 +160,7 @@ public void doInvoke(@Nonnull Project project, Editor editor, PsiFile file) thro
156160
quotes = PyStringLiteralUtil.getQuotes(expression.getText());
157161
quotesDetected = true;
158162
}
159-
String value = ((PyStringLiteralExpression) expression).getStringValue();
163+
String value = stringLiteral1.getStringValue();
160164
if (!useFormatMethod) {
161165
value = value.replace("%", "%%");
162166
}
@@ -169,7 +173,7 @@ public void doInvoke(@Nonnull Project project, Editor editor, PsiFile file) thro
169173
}
170174
}
171175
if (quotes == null) {
172-
quotes = Pair.create("\"", "\"");
176+
quotes = Couple.of("\"", "\"");
173177
}
174178
stringLiteral.insert(0, quotes.getFirst());
175179
if (isUnicode && !quotes.getFirst().toLowerCase().contains("u")) {

0 commit comments

Comments
 (0)