This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 61931 - Push down adds extra line to the pushed method
Summary: Push down adds extra line to the pushed method
Status: VERIFIED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Unsupported (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: Pavel Flaska
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-08 13:42 UTC by Roman Strobl
Modified: 2007-09-26 09:14 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Roman Strobl 2005-08-08 13:42:15 UTC 
[NetBeans 200508071800; WinXP; JDK 1.5.0_04]

Issue summary:
--------------
Push down adds an extra line on beginning of pushed method in target class.

Steps to reproduce:
-------------------
1. Create two classes, Parent.java and Child.java.
2. Add into Parent.java following code:
    public String pokus() {
        return "";        
    }
3. Execute push down refactoring on method pokus().
-> The following code is created in Child.java:
    public String pokus() {

        return "";        
    }
Comment 1 Martin Matula 2005-08-25 08:21:31 UTC 
Looks like an issue for Pavel :)
Comment 2 Pavel Flaska 2005-08-26 14:07:59 UTC 
Checking in
java/javacore/src/org/netbeans/modules/javacore/jmiimpl/javamodel/BehavioralFeatureImpl.java;
/cvs/java/javacore/src/org/netbeans/modules/javacore/jmiimpl/javamodel/BehavioralFeatureImpl.java,v
 <--  BehavioralFeatureImpl.java
new revision: 1.12; previous revision: 1.11
done
Checking in
refactoring/test/unit/data/goldenfiles/org/netbeans/test/refactoring/pullup/PullUpRefactoringTest/TestBaseClass.pass;
/cvs/refactoring/test/unit/data/goldenfiles/org/netbeans/test/refactoring/pullup/PullUpRefactoringTest/TestBaseClass.pass,v
 <--  TestBaseClass.pass
new revision: 1.2; previous revision: 1.1
done
Comment 3 Roman Strobl 2005-08-29 09:46:47 UTC 
It works fine now. Verified in 200508281800.
Comment 4 Quality Engineering 2007-09-20 10:49:04 UTC 
Reorganization of java component