qgis · Jul 14, 2006
diff --git a/‎src/gui/qgsattributetable.cpp
Lines changed: 17 additions & 13 deletions b/‎src/gui/qgsattributetable.cpp
Lines changed: 17 additions & 13 deletions
diff --git a/‎src/gui/qgsattributetable.h
Lines changed: 17 additions & 3 deletions b/‎src/gui/qgsattributetable.h
Lines changed: 17 additions & 3 deletions
@@ -369,6 +369,8 @@ void QgsAttributeTable::deleteAttribute(const QString& name)
     mEdited=true;
 }
 
+
+/* Deprecated: See QgisApp::editCopy() instead */
 void QgsAttributeTable::copySelectedRows()
 {
   // Copy selected rows to the clipboard
@@ -413,21 +415,23 @@ void QgsAttributeTable::copySelectedRows()
 
 bool QgsAttributeTable::commitChanges(QgsVectorLayer* layer)
 {
-    bool returnvalue=true;
-    if(layer)
-    {
-	if(!layer->commitAttributeChanges(mDeletedAttributes, mAddedAttributes, mChangedValues))
-	{
-	    returnvalue=false;
-	}
-    }
-    else
-    {
-	returnvalue=false;
-    }
+  bool isSuccessful = FALSE;
+
+  if(layer)
+  {
+    isSuccessful = layer->commitAttributeChanges(
+                            mDeletedAttributes,
+                            mAddedAttributes,
+                            mChangedValues);
+  }
+
+  if (isSuccessful)
+  {
     mEdited=false;
     clearEditingStructures();
-    return returnvalue;
+  }
+
+  return isSuccessful;
 }
 
 bool QgsAttributeTable::rollBack(QgsVectorLayer* layer)
 
@@ -70,11 +70,25 @@ class QgsAttributeTable:public Q3Table
     /**Deletes an attribute (but does not commit it)
       @param name attribute name*/
     void deleteAttribute(const QString& name);
-    /**Copies the selected rows to the clipboard */
+
+    /** Copies the selected rows to the clipboard 
+        Deprecated: See QgisApp::editCopy() instead */
     void copySelectedRows();
-    /**Delegates to QgsVectorLayer to decide, which changes
-      belong to not commited features or to commited ones*/
+
+    /**
+      Attempts to commit any changes to disk.  Returns the result of the attempt.
+      If a commit fails, the in-memory changes are left alone.
+
+      This allows editing to continue if the commit failed on e.g. a
+      disallowed value in a Postgres database - the user can re-edit and try
+      again.
+
+      Delegates to QgsVectorLayer to decide, which changes
+      belong to not commited features or to commited ones.
+
+     */
     bool commitChanges(QgsVectorLayer* layer);
+
     /**Discard all changes and restore
       the state before editing was started*/
     bool rollBack(QgsVectorLayer* layer);