Saving using :w doesn't trigger global save event about vrapper HOT 6 CLOSED

Are you able to get more info from the AnyEdit folks to see what exactly the "global save event" is? There's probably a way for me to manually fire that event if I know exactly what it is.

As for Vrapper, when you do :w we invoke AbstractTextEditor.doSave(). When you do :wall we invoke AbstractTextEditor.getSite().getWorkbenchWindow().getWorkbench().saveAllEditors(). I'm not really sure what to do if those methods don't fire global save events. I'm not aware of other save methods that we should be invoking instead.

I don't mind making this change to support AnyEdit if you can find what exactly needs to be changed.

from vrapper.

Hi,
I'm the AnyEdit guy :-)

I think that you have to execute something like this:

    IHandlerService handlerService = (IHandlerService) PlatformUI
            .getWorkbench().getService(IHandlerService.class);
    try {
        handlerService.executeCommand(IWorkbenchCommandConstants.FILE_SAVE, null);
    } catch (CommandException e) {
        // TODO Auto-generated catch block
    } 

instead of directly calling doSave(). All the required hooks will be then executed. AnyEdit works on IExecutionListener.preExecute() hook, which is triggered by the "save" command execution (e.g. if one just hits "Ctrl+S" or clicks on the "save" button).
I did not tried it but I guess this should work.
Regards,
Andrey

from vrapper.

Copy and pasting code is definitely something I'm capable of! Thanks for doing the hard work for me. :) The :w command still appears to work so it's all the same on my end. I'll just assume I need to make a similar change for the "save all" operation and use IWorkbenchCommandConstants.FILE_SAVE_ALL. Thanks again!

from vrapper.

Ok, I've updated the unstable update site with 0.21.20120527. This version contains the refactored save operations. Please install it alongside AnyEdit and let me know if Andrey's fix didn't actually work. Thanks!

from vrapper.

Thanks very much for the fast turnaround. I'll try it out when I get back
in an hour or two and let you know how it looks.

from vrapper.

That looks great, tested both :w and :wall and both seem to work as they expected. Thanks very much kerforbes and Andrey for the fast resolution to this issue.

from vrapper.