Skip to content

Commit

Permalink
BZ-1247154 - Fly-over notifications not shown when renaming Data Obje…
Browse files Browse the repository at this point in the history
…cts in standalone editor
  • Loading branch information
wmedvede committed Oct 2, 2015
1 parent 0ec171e commit 4f78627
Showing 1 changed file with 3 additions and 1 deletion.
Expand Up @@ -498,6 +498,7 @@ private RemoteCallback<Path> getRenameSuccessCallback() {
@Override
public void callback( final Path targetPath ) {
view.hideBusyIndicator();
notification.fire( new NotificationEvent( org.uberfire.ext.editor.commons.client.resources.i18n.CommonConstants.INSTANCE.ItemRenamedSuccessfully() ) );
}
};
}
Expand Down Expand Up @@ -573,7 +574,7 @@ public void execute() {
new Command() {
@Override
public void execute() {
saveOperationService.save(versionRecordManager.getPathToLatest(), getSaveCommand(newTypeInfo, versionRecordManager.getPathToLatest()));
saveOperationService.save(versionRecordManager.getPathToLatest(), getSaveCommand( newTypeInfo, versionRecordManager.getPathToLatest()));
}
},
Constants.INSTANCE.modelEditor_action_yes_refactor_file_name(),
Expand Down Expand Up @@ -715,6 +716,7 @@ public void execute() {
versionRecordManager.reloadVersions( currentPath );

} else {
notification.fire( new NotificationEvent( org.uberfire.ext.editor.commons.client.resources.i18n.CommonConstants.INSTANCE.ItemRenamedSuccessfully() ) );
//If the file was renamed as part of the file saving, don't do anything.
//A rename event will arrive, the same as for the "Rename" case.
//and the file will be automatically reloaded.
Expand Down

0 comments on commit 4f78627

Please sign in to comment.