Commit d2bfe3d1 authored by Anna.Kozlova's avatar Anna.Kozlova
Browse files

ensure name when retrieving constructors from a class (IDEA-188068)

parent a43d6320
Branches unavailable Tags unavailable
Showing with 13 additions and 13 deletions
+13 -13
...@@ -70,7 +70,7 @@ public class PsiImplUtil { ...@@ -70,7 +70,7 @@ public class PsiImplUtil {
public static PsiMethod[] getConstructors(@NotNull PsiClass aClass) { public static PsiMethod[] getConstructors(@NotNull PsiClass aClass) {
List<PsiMethod> result = null; List<PsiMethod> result = null;
for (PsiMethod method : aClass.getMethods()) { for (PsiMethod method : aClass.getMethods()) {
if (method.isConstructor()) { if (method.isConstructor() && method.getName().equals(aClass.getName())) {
if (result == null) result = ContainerUtil.newSmartList(); if (result == null) result = ContainerUtil.newSmartList();
result.add(method); result.add(method);
} }
......
...@@ -503,8 +503,8 @@ public class PsiClassImpl extends JavaStubPsiElement<PsiClassStub<?>> implements ...@@ -503,8 +503,8 @@ public class PsiClassImpl extends JavaStubPsiElement<PsiClassStub<?>> implements
} }
// rename constructors // rename constructors
for (PsiMethod method : getConstructors()) { for (PsiMethod method : getMethods()) {
if (method.getName().equals(oldName)) { if (method.isConstructor() && method.getName().equals(oldName)) {
method.setName(newName); method.setName(newName);
} }
} }
......
// "Add constructor parameter" "true" // "Add constructor parameter" "true"
class A { class Foo {
private final String text; private final String text;
public Foo(String text) { public Foo(String text) {
......
// "Add constructor parameter" "true" // "Add constructor parameter" "true"
class A { class Foo {
private final String te<caret>xt; private final String te<caret>xt;
public Foo(String text) { public Foo(String text) {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int var; private final int var;
private Main() { private Main() {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int var; private final int var;
private Main() { private Main() {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int var; private final int var;
private Main(int var) { private Main(int var) {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int var; private final int var;
private Main() { private Main() {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int v<caret>ar; private final int v<caret>ar;
private Main() private Main()
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int v<caret>ar; private final int v<caret>ar;
private Main() { private Main() {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int v<caret>ar; private final int v<caret>ar;
private Main(int var) { private Main(int var) {
......
// "Initialize in constructor" "true" // "Initialize in constructor" "true"
class A { class Main {
private final int v<caret>ar; private final int v<caret>ar;
private Main() { private Main() {
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment