Repository navigation
Auto-import was not triggered automatically in some cases #1098
Description
Environment
- Operating System: Win/Mac/Linux
- JDK version: 13/12/8
- Visual Studio Code version:1.39.2
- Java extension version:0.51.0
Steps To Reproduce
- Make sure java.actionsOnPaste.organizeImports in settings is true.
- New Foo.java with codes as below:
import java.io.File;
public class Foo {
public static void main(String[] args) {
File f =new File("A.txt");
}
}
Result:
-
If copy the main method to another empty java file
Auto-import was not triggered automatically.
-
If copy the all class to another empty java file
Auto-import was triggered automatically.
Activity
fbricon commentedon Oct 16, 2019
The feature is expected to work when pasting into an empty file. It seems you're trying to paste in a file that already has a class definition, so it's not empty.
Triggering autoimport when pasting in non-empty files might cause some issues eventually, that's why I'd prefer not to enable it at this time.
testforstephen commentedon Oct 16, 2019
Copying a method (even a code block) to an existing class is a higher frequency operation, so supporting copying a method is more helpful for me.
fbricon commentedon Oct 16, 2019
Fair enough. Let's try it then, I'll mark the feature as experimental.
leonardofel commentedon Oct 18, 2019
oh very nice feature, but has a little problem...
Is an automatically activated experimental feature really necessary?
fbricon commentedon Oct 21, 2019
@leonardofel can you please open a new ticket?
The best way to expose users to the new feature is to enable it automatically. I reckon this was a bit risky, but since this can be disabled, I'm not too worried. In any case, we'll either fix your problem or backpedal on the enable-by-default bit.
fbricon commentedon Oct 21, 2019
@leonardofel #1112 was opened about that issue
leonardofel commentedon Oct 21, 2019
I understand... this bug occurs when I paste during multicursor, anyway thx for the reply, waiting for future fix
fbricon commentedon Oct 22, 2019
@leonardofel you can install the latest CI build containing the fix, from https://download.jboss.org/jbosstools/jdt.ls/staging/?C=M;O=D