This bug is still present in build 515.
It not only affects static variables,
but also references to the inner class
itself, like so:
BEFORE:
class Outer {
class Inner {
protected static final String SEPARATOR = ".";
public void doStuff() {
String s = "A" + SEPARATOR + "B";
}
public boolean equals(Object object) {
Inner otherInner = (Inner)object;
//...
}
}
}
AFTER:
class Outer {
}
class Inner {
private static final String SEPARATOR = ".";
public void doStuff() {
String s = "A" + Outer.Inner.SEPARATOR + "B";
}
public boolean equals(Object object) {
Outer.Inner otherInner = (Outer.Inner)object;
//...
}
}
The "Outer.Inner." and "Outer." have been incorrectly added.
Regards.
-Thomas
> -----Original Message-----
> From: Valentin Kipiatkov [mailto:[EMAIL PROTECTED]]
> Sent: Friday, October 26, 2001 1:47 AM
> To: [EMAIL PROTECTED]
> Subject: Re: [Eap-list] Bug in "Move Inner to Upper Level" Refactoring
>
>
> This is fixed already in the builds #508 and #509. Please,
> switch to the
> latest build.
>
> Best regards,
> Valentin Kipiatkov
> -----------------------------------------------------------
> IntelliJ Software, http://www.intellij.com/
> "Develop with pleasure"
> -----------------------------------------------------------
>
> ----- Original Message -----
> From: "Vollmer, Thomas - CannonSA" <[EMAIL PROTECTED]>
> To: <[EMAIL PROTECTED]>
> Sent: Friday, October 26, 2001 5:12 AM
> Subject: [Eap-list] Bug in "Move Inner to Upper Level" Refactoring
>
>
> > BEFORE:
> > class Outer {
> > class Inner {
> > protected static final String SEPARATOR = ".";
> > public void doStuff() {
> > String s = "A" + SEPARATOR + "B";
> > }
> > }
> > }
> >
> > AFTER:
> > class Outer {
> > }
> >
> > class Inner {
> > private static final String SEPARATOR = ".";
> > public void doStuff() {
> > String s = "A" + Outer.Inner.SEPARATOR + "B";
> > }
> > }
> >
> > Notice the incorrect prefix "Outer.Inner." when referring to
> > the static variable SEPARATOR in method Inner.doStuff().
> > This prefix is automagically added when doing the refactoring.
> > Build 507.
> >
> > -Thomas
************************************
If this email is not intended for you, or you are not responsible for
the delivery of this message to the addressee, please note that this
message may contain ITT Privileged/Proprietary Information. In such
a case, you may not copy or deliver this message to anyone. You should
destroy this message and kindly notify the sender by reply email.
Information contained in this message that does not relate to the
business of ITT is neither endorsed by nor attributable to ITT.
************************************
_______________________________________________
Eap-list mailing list
[EMAIL PROTECTED]
http://www.intellij.com/mailman/listinfo/eap-list
