--- a/flys-client/src/main/java/de/intevation/flys/client/client/ui/chart/ChartThemePanel.java	Wed May 25 14:50:37 2011 +0000
+++ b/flys-client/src/main/java/de/intevation/flys/client/client/ui/chart/ChartThemePanel.java	Wed May 25 14:58:42 2011 +0000
@@ -1,10 +1,10 @@
 package de.intevation.flys.client.client.ui.chart;
 
-import java.util.List;
-
 import com.google.gwt.core.client.GWT;
+import com.google.gwt.user.client.rpc.AsyncCallback;
 
 import com.smartgwt.client.types.ListGridFieldType;
+import com.smartgwt.client.util.SC;
 import com.smartgwt.client.widgets.Canvas;
 import com.smartgwt.client.widgets.grid.events.EditCompleteEvent;
 import com.smartgwt.client.widgets.grid.events.EditCompleteHandler;
@@ -14,13 +14,15 @@
 import com.smartgwt.client.widgets.layout.VLayout;
 
 import de.intevation.flys.client.shared.model.Collection;
-import de.intevation.flys.client.shared.model.Facet;
 import de.intevation.flys.client.shared.model.FacetRecord;
 import de.intevation.flys.client.shared.model.OutputMode;
 import de.intevation.flys.client.shared.model.Theme;
 import de.intevation.flys.client.shared.model.ThemeList;
 
+import de.intevation.flys.client.client.Config;
 import de.intevation.flys.client.client.FLYSConstants;
+import de.intevation.flys.client.client.services.CollectionAttributeService;
+import de.intevation.flys.client.client.services.CollectionAttributeServiceAsync;
 
 
 /**
@@ -31,6 +33,12 @@
     /** The interface that provides i18n messages. */
     private FLYSConstants MSG = GWT.create(FLYSConstants.class);
 
+
+    /** The service that is used to modify collection attributes.*/
+    protected CollectionAttributeServiceAsync updater =
+        GWT.create(CollectionAttributeService.class);
+
+
     public static final String GRID_FIELD_ACTIVE = "active";
     public static final String GRID_FIELD_NAME   = "name";
 
@@ -99,6 +107,19 @@
 
 
     /**
+     * Replace the current collection with a new one. <b>NOTE: this operation
+     * triggers updateGrid() which modifies the themes in the grid.</b>
+     *
+     * @param collection The new collection object.
+     */
+    protected void setCollection(Collection collection) {
+        this.collection = collection;
+
+        updateGrid();
+    }
+
+
+    /**
      * A method that removes all records from theme grid.
      */
     protected void clearGrid() {
@@ -119,6 +140,8 @@
      * data.
      */
     protected void updateGrid() {
+        GWT.log("ChartThemePanel.updateGrid");
+
         clearGrid();
 
         ThemeList themeList = collection.getThemeList(mode.getName());
@@ -153,7 +176,72 @@
         int         row = event.getRowNum();
         FacetRecord rec = (FacetRecord) list.getRecord(row);
 
-        // TODO Save modified facets
+        updateThemeList(rec.getTheme());
+    }
+
+
+    /**
+     * Update the theme list of the current collection with a modified theme. If
+     * a theme is really modified, the CollectionAttributeService is triggered
+     * to save the changes to the artifact server.
+     *
+     * @param theme The modified theme.
+     */
+    protected void updateThemeList(Theme theme) {
+        GWT.log("Update theme: " + theme.getFacet());
+
+        ThemeList themeList = collection.getThemeList(mode.getName());
+
+        String a = theme.getArtifact();
+        String f = theme.getFacet();
+
+        int num = themeList != null ? themeList.getThemeCount() : 0;
+
+        boolean updateRequired = false;
+
+        for (int i = 1; i <= num; i++) {
+            Theme old = themeList.getThemeAt(i);
+
+            if (f.equals(old.getFacet()) && a.equals(old.getArtifact())) {
+                themeList.removeTheme(old);
+                themeList.addTheme(theme);
+
+                updateRequired = true;
+
+                break;
+            }
+        }
+
+        if (updateRequired) {
+            updateCollection();
+        }
+    }
+
+
+    /**
+     * This method triggers the CollectionAttributeService. Based on the current
+     * collectin settings, the attribute of the collection is modified or not.
+     * But in every case, we will get a new collection object - which might be
+     * the same as the current one.
+     */
+    public void updateCollection() {
+        final Config config = Config.getInstance();
+        final String url    = config.getServerUrl();
+        final String loc    = config.getLocale();
+
+        GWT.log("ChartThemePanel.updateCollection via RPC now");
+
+        updater.update(collection, url, loc, new AsyncCallback<Collection>() {
+            public void onFailure(Throwable caught) {
+                GWT.log("Could not update collection attributes.");
+                SC.warn(MSG.getString(caught.getMessage()));
+            }
+
+
+            public void onSuccess(Collection collection) {
+                setCollection(collection);
+            }
+        });
     }
 }
 // vim:set ts=4 sw=4 si et sta sts=4 fenc=utf8 :