[jbosstools-issues] [JBoss JIRA] (JBIDE-21791) Forge addon - Changing the current file may cause a "file has changed in file vs memory" issue in Eclipse

[Lincoln Baxter III (JIRA)]

    [ https://issues.jboss.org/browse/JBIDE-21791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13173666#comment-13173666 ] 

Lincoln Baxter III commented on JBIDE-21791:
--------------------------------------------

If a user has unsaved files, I think a decent middle-ground would be to ask the user if they want to save all files first. If they say yes, save them. If they say no, continue and let them be annoyed by "file has changed" messages. But even that is annoying, because you could have a completely unrelated unsaved file open and that would trigger this behavior. 

I'd also recommend making this a configurable option, so the user can mark a check-box for: "[x] Remember my choice." so they don't have to be bugged by this over and over again if they don't want to.

> Forge addon - Changing the current file may cause a "file has changed in file vs memory" issue in Eclipse
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: JBIDE-21791
>                 URL: https://issues.jboss.org/browse/JBIDE-21791
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: forge
>    Affects Versions: 4.3.1.Beta2
>            Reporter: Claus Ibsen
>            Assignee: George Gastaldi
>             Fix For: 4.3.1.CR1
>
>         Attachments: eclipse-add.png
>
>
> Similar problem as FORGE-2605 but this time its in Eclipse.
> See the attached screenshot, where I have edited the file camel-context.xml, and then run my forge command, which would insert a Camel endpoint at the cursor position. But eclipse has detected a file conflict as in that dialog.
> I wonder if the fix should be like in IDEA to save the file before the command is run.



--
This message was sent by Atlassian JIRA
(v6.4.11#64026)