-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Must not modify PSI inside save listener #53
Comments
Stack Trace in IDE Fatal Errors Window: |
Thank you for filling the issue. Does that happen often? Does the plugin still work? Also can you post exact version (build # in Help > About)? I'll look into it. |
Thanks for investigating. I have only started using IntelliJ and save actions today, however I can see 3 instances of the error in the IntelliJ console log. I think the plugin is working, not exactly sure, I will investigate further the next time it happens. Exact version information: |
Having this problem since I installed the new version of IntelliJ IDEA: 2016.2Build #IU-162.1121.32 |
Same issue after the latest IntelliJ update. Looks like the plugin does something (e.g. reformats the code and rearranges methods), but causes an exception to be thrown too. :) |
The processors are independent (reformat, quick fixes, rearrange, etc.) and it seems it's only the rearrange processor that crashes (the others are finishing). I'll check it out |
Keep having this problem. Especially when creating a new file |
I'm getting it when it would appear to be making fields final. It happens sometimes.
IntelliJ IDEA 2016.2 |
In my case it also happens when I am creating a new class. Class is created with double package declaration (one after another), after that IDE becomes unstable (e.g. weird syntax highlighting bug). |
@dubreuia Listen to this : and the problem had disappeared on both version Ultimate and Community. |
Also having the issue on saving and creating new files. This basically makes the plugin unusable for me... |
@AbdElraoufSabri Happy to hear that it disappeared, might be your java version... Are you sure it's not your configuration? It looks like the problem only happens if you have the "rearrange code" option activated. @MALPI An easy workaround is to deactivate "rearrange code", the rest of the plugin should still work. I'll check out this issue ASAP. |
@dubreuia Actually the problem happened again but you were right, once I disabled rearrange code the problem disappeared |
I got the same problem, but rearrange code is one of the reasons why I use this plugin so I would like to see a fix for this. Error: http://pastebin.com/Xr6R5VFF |
I need to find time for this one, I didn't expect Jetbrains would break the plugin. Don't hesitate if you can submit a pull request. |
Same as @junje |
Same here, only sometimes. I use your plugin for about 2 years. I use rearrange option. IntelliJ IDEA (Community Edition) 2017.1 EAP Plugin version : 0.12 |
I also hit this rather frequently by now. Not using code rearrange, what I have enabled: |
I still can't reproduce this one. Can you post a full text stack trace @gendalfas? (or anybody else) I'd prefer understanding why this happens than putting a big try / catch |
Will do so once back at work. I have noticed that the stack traces actually differ somewhat midway, so that actually might be of help finding the actual cause. |
There's a possible solution in #40 |
Still the same problem, very annoying when this happens (and auto reformatting is the reason I am using SaveActions). |
I'm trying a fix in version 0.15 that I'll release this weekend. Can't reproduce so I hope it works. @BullCheat Cool d'avoir un bon feedback sur note site :-). Check notre blog tech et notre github si ça t'intéresse https://github.com/lesfurets / https://beastie.lesfurets.com/ |
https://media.giphy.com/media/niEnUBEHbY448/giphy.gif?response_id=5925df761649a9c40ca12ecd
(yes it works) |
@dubreuia I'm also having this issue, so there will be a possible fix for it on version 0.15 being released on this weekend? |
I'm testing it right now I should be able to release it today. The problem is: I can't reproduce so I'm just hoping I'm fixing the problem. |
Not fixed |
I can confirm. |
Crashes frequently for me. IntelliJ IDEA 2017.1.3 Must not modify PSI inside save listener |
You should probable reopen as i still have the issue. ans here is the stack: |
This issue will haunt me forever |
@charlouze What plugin version you used? |
@AbdElraoufSabri The very last one ! (0.18) |
Hey @dubreuia unfortunately I am not able to reproduce the issue but I remember how I got mine:
IntelliJ IDEA 2017.2.2 |
Hi @dubreuia, IntelliJ IDEA 2017.2.2 IDEA Project uses springboot, lombok, gradle etc... I appear to be able to re-create the issue any time.
|
I've corrected the problem, attached 0.20-RC should fix it. I'll test it a bit before releasing (if someone could help I'd appreciate that). |
Tested on a couple of classes, seems to fix the issue. Can't reproduce using @MattGurneyAMP steps. +1 LGTM IntelliJ IDEA 2017.2 |
Released v0.20 |
Getting the error "Must not modify PSI inside save listener" when saving a .java file.
IDEA: 2016.2
Save Actions: 0.11
Save Actions Settings:
The text was updated successfully, but these errors were encountered: