diff --git a/Kieker.WebGUI/src/main/java/kieker/webgui/common/FileManager.java b/Kieker.WebGUI/src/main/java/kieker/webgui/common/FileManager.java index ef2b968a64b21f2e8a4ac767720814b5a0a3e20b..401e351bc20ecb6ea1522b1c77681146af983bb6 100644 --- a/Kieker.WebGUI/src/main/java/kieker/webgui/common/FileManager.java +++ b/Kieker.WebGUI/src/main/java/kieker/webgui/common/FileManager.java @@ -26,6 +26,8 @@ import java.io.FileOutputStream; import java.io.IOException; import java.util.ArrayList; import java.util.List; +import javax.faces.application.FacesMessage; +import javax.faces.context.FacesContext; import kieker.analysis.AnalysisController; import kieker.analysis.model.analysisMetaModel.MIDependency; @@ -135,9 +137,7 @@ public final class FileManager { /* * Make sure that the directory for the project exists. */ - if (!dirProject.exists()) { - return false; - } else { + if (dirProject.exists()) { /* * Try to save the project. */ @@ -145,12 +145,15 @@ public final class FileManager { try { // TODO Copy before saving as the controller destroys at least the dependencies. final AnalysisController controller = new AnalysisController(project); - return controller.saveToFile(fileProject); + if (controller.saveToFile(fileProject)) { + FacesContext.getCurrentInstance().addMessage(null, new FacesMessage(FacesMessage.SEVERITY_INFO, "", "Project saved: " + project.getName())); + return true; + } } catch (final NullPointerException ex) { FileManager.LOG.error("Could not save project '" + projectName + "'.", ex); - return false; } } + return false; } /** diff --git a/Kieker.WebGUI/src/main/webapp/main.xhtml b/Kieker.WebGUI/src/main/webapp/main.xhtml index a828cae4670783a29df8cc912cc111ed83d202e3..3c32c824819f5f636c174cd119cdf8695b9f96a3 100644 --- a/Kieker.WebGUI/src/main/webapp/main.xhtml +++ b/Kieker.WebGUI/src/main/webapp/main.xhtml @@ -32,11 +32,11 @@ <!-- This is the submenu for the current project, for example if someone doesn't want to use the context menu within the browser. --> <p:submenu label="Current Project"> - <p:menuitem value="Save Project" ajax="true" action="#{availableProjectsBean.saveProject(selectedProjectBean.getSelectedProject)}" update=":projectsForm" /> + <p:menuitem value="Save Project" ajax="true" action="#{availableProjectsBean.saveProject(selectedProjectBean.getSelectedProject())}" update=":projectsForm" /> <p:menuitem value="Set as Main Project" ajax="true" action="#{selectedProjectBean.setMainProject(selectedProjectBean.getSelectedProject())}" update=":projectsForm" /> <p:separator /> - <p:menuitem value="Delete Project" ajax="true" action="#{availableProjectsBean.deleteProject(selectedProjectBean.getSelectedProject)}" update=":projectsForm" /> - <p:menuitem value="Reset Project" ajax="true" action="#{availableProjectsBean.resetProject(selectedProjectBean.getSelectedProject)}" update=":projectsForm" /> + <p:menuitem value="Delete Project" ajax="true" action="#{availableProjectsBean.deleteProject(selectedProjectBean.getSelectedProject())}" update=":projectsForm" /> + <p:menuitem value="Reset Project" ajax="true" action="#{availableProjectsBean.resetProject(selectedProjectBean.getSelectedProject())}" update=":projectsForm" /> <p:separator /> <p:menuitem value="Configure Dependencies" ajax="false" url="/Kieker.WebGUI/projectDependencies" /> </p:submenu>