From ed0ebdb26f75f22b559dbf3f5e9288d26643e24b Mon Sep 17 00:00:00 2001
From: David Mason <drdmason@gmail.com>
Date: Thu, 5 Mar 2015 12:02:02 +1000
Subject: [PATCH] Ensure that a success message is shown when project about
 page is updated.

---
 .../main/java/org/zanata/action/ProjectHome.java    | 13 +++++++++++++
 zanata-war/src/main/resources/messages.properties   |  2 ++
 .../WEB-INF/layout/project/settings-tab.xhtml       |  2 +-
 3 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/zanata-war/src/main/java/org/zanata/action/ProjectHome.java b/zanata-war/src/main/java/org/zanata/action/ProjectHome.java
index 9151ba3a4d..543171ab1e 100644
--- a/zanata-war/src/main/java/org/zanata/action/ProjectHome.java
+++ b/zanata-war/src/main/java/org/zanata/action/ProjectHome.java
@@ -966,6 +966,19 @@ public List<ValidationAction.State> getValidationStates() {
         return Arrays.asList(ValidationAction.State.values());
     }
 
+    /**
+     * Update the about page to the entered value, and show a success message.
+     */
+    @Restrict("#{s:hasPermission(projectHome.instance, 'update')}")
+    public void updateAboutPage() {
+        String status = update();
+        if ("updated".equals(status)) {
+            FacesMessages.instance().add(StatusMessage.Severity.INFO, msgs.get("jsf.project.AboutPageUpdated"));
+        } else {
+            FacesMessages.instance().add(StatusMessage.Severity.ERROR, msgs.get("jsf.project.AboutPageUpdateFailed"));
+        }
+    }
+
     @Override
     protected void updatedMessage() {
         // Disable the default message from Seam
diff --git a/zanata-war/src/main/resources/messages.properties b/zanata-war/src/main/resources/messages.properties
index b0705c8ac8..29545a2420 100644
--- a/zanata-war/src/main/resources/messages.properties
+++ b/zanata-war/src/main/resources/messages.properties
@@ -290,6 +290,8 @@ jsf.languageSettings.LanguagesDisabled=Disabled languages: {0}.
 jsf.languageSettings.LanguageEnabled=Language "{0}" has been enabled.
 jsf.languageSettings.LanguagesEnabled=Enabled languages: {0}.
 jsf.project.LanguageUpdateFromGlobal=Updated languages from global settings.
+jsf.project.AboutPageUpdated=About page updated.
+jsf.project.AboutPageUpdateFailed=There was a problem while updating the about page.
 jsf.project.AddWebhook=Add webhook
 jsf.project.RemoveWebhook=Webhook {0} removed.
 jsf.project.AddNewWebhook=Webhook {0} added.
diff --git a/zanata-war/src/main/webapp/WEB-INF/layout/project/settings-tab.xhtml b/zanata-war/src/main/webapp/WEB-INF/layout/project/settings-tab.xhtml
index 308af34e8c..ca2fb46ba2 100644
--- a/zanata-war/src/main/webapp/WEB-INF/layout/project/settings-tab.xhtml
+++ b/zanata-war/src/main/webapp/WEB-INF/layout/project/settings-tab.xhtml
@@ -490,7 +490,7 @@
     </p>
 
     <a4j:commandLink value="#{msgs['jsf.project.saveNotes']}"
-      action="#{projectHome.update}" rendered="#{projectHome.managed}"
+      action="#{projectHome.updateAboutPage}" rendered="#{projectHome.managed}"
       render="project-about"
       styleClass="l--push-top-1 button button--primary"/>
   </h:form>