2004-03-10 Umadevi S (sumadevi@novell.com)
[mono.git] / mcs / class / System.Data / System.Data / DataRowCollection.cs
index 5b25b50b3fc926539bce8c81f0ed5787aebc3f66..1debc20ca4260867b88ff1a61fa4c01d64b1c999 100644 (file)
@@ -62,14 +62,22 @@ namespace System.Data
                        if (row == null)
                                throw new ArgumentNullException("row", "'row' argument cannot be null.");
 
-                       if (list.IndexOf(row) != -1)
+                       if (row.Table != this.table)
+                               throw new ArgumentException ("This row already belongs to another table.");
+                       
+                       // If row id is not -1, we know that it is in the collection.
+                       if (row.RowID != -1)
                                throw new ArgumentException ("This row already belongs to this table.");
                        
-                       if (table.DataSet != null && table.DataSet.EnforceConstraints)
+
+                       if ((table.DataSet == null || table.DataSet.EnforceConstraints) && !table._duringDataLoad)
                                // we have to check that the new row doesn't colide with existing row
                                ValidateDataRowInternal(row);
                        
+                       row.HasParentCollection = true;
                        list.Add (row);
+                       // Set the row id.
+                       row.RowID = list.Count - 1;
                        row.AttachRow ();
                        row.Table.ChangedDataRow (row, DataRowAction.Add);
                }
@@ -90,6 +98,20 @@ namespace System.Data
                /// </summary>
                public void Clear () 
                {
+                       if (this.table.DataSet != null)
+                       {
+                               foreach (DataTable table in this.table.DataSet.Tables)
+                               {
+                                       foreach (Constraint c in table.Constraints)
+                                       {
+                                               if (c is ForeignKeyConstraint)
+                                               {
+                                                       if (((ForeignKeyConstraint) c).RelatedTable.Equals(this.table))
+                                                               throw new InvalidConstraintException("Cannot clear table Parent because ForeignKeyConstraint " + c.ConstraintName + " enforces Child.");
+                                               }
+                                       }
+                               }
+                       }
                        list.Clear ();
                }
 
@@ -133,18 +155,22 @@ namespace System.Data
                        
                        foreach (DataRow row in this) {
                                
-                               object primValue = row [primColumnName];
-                               if (key == null) {
-                                       if (primValue == null)
-                                               return row;
-                                       else 
-                                               continue;
-                               }
+                               if (row.RowState != DataRowState.Deleted)
+                               {
+                                       object primValue = row [primColumnName];
+                                       if (key == null) 
+                                       {
+                                               if (primValue == null)
+                                                       return row;
+                                               else 
+                                                       continue;
+                                       }
                                       
-                               newKey = Convert.ChangeType (key, Type.GetTypeCode(primValue.GetType ()));
+                                       newKey = Convert.ChangeType (key, Type.GetTypeCode(primValue.GetType ()));
 
-                               if (primValue.Equals (newKey))
-                                       return row;
+                                       if (primValue.Equals (newKey))
+                                               return row;
+                               }
                        }
                                                
                        // FIXME: is the correct value null?
@@ -170,28 +196,34 @@ namespace System.Data
                        
                        foreach (DataRow row in this) {
                                
-                               bool eq = true;
-                               for (int i = 0; i < keys.Length; i++) {
+                               if (row.RowState != DataRowState.Deleted)
+                               {
+                                       bool eq = true;
+                                       for (int i = 0; i < keys.Length; i++) 
+                                       {
                                        
-                                       object primValue = row [primColumnNames [i]];
-                                       object keyValue = keys [i];
-                                       if (keyValue == null) {
-                                               if (primValue == null)
-                                                       return row;
-                                               else 
-                                                       continue;
-                                       }
+                                               object primValue = row [primColumnNames [i]];
+                                               object keyValue = keys [i];
+                                               if (keyValue == null) 
+                                               {
+                                                       if (primValue == null)
+                                                               return row;
+                                                       else 
+                                                               continue;
+                                               }
                                                                       
-                                       newKey = Convert.ChangeType (keyValue, Type.GetTypeCode(primValue.GetType ()));
+                                               newKey = Convert.ChangeType (keyValue, Type.GetTypeCode(primValue.GetType ()));
 
-                                       if (!primValue.Equals (newKey)) {
-                                               eq = false;
-                                               break;
-                                       }                                               
-                               }
+                                               if (!primValue.Equals (newKey)) 
+                                               {
+                                                       eq = false;
+                                                       break;
+                                               }                                               
+                                       }
 
-                               if (eq)
-                                       return row;
+                                       if (eq)
+                                               return row;
+                               }
                        }
                                                
                        // FIXME: is the correct value null?
@@ -205,11 +237,43 @@ namespace System.Data
                {
                        if (pos < 0)
                                throw new IndexOutOfRangeException ("The row insert position " + pos + " is invalid.");
+                       
+                       if (row == null)
+                               throw new ArgumentNullException("row", "'row' argument cannot be null.");
+       
+                       if (row.Table != this.table)
+                               throw new ArgumentException ("This row already belongs to another table.");
+
+                       // If row id is not -1, we know that it is in the collection.
+                       if (row.RowID != -1)
+                               throw new ArgumentException ("This row already belongs to this table.");
+                       
+                       if ((table.DataSet == null || table.DataSet.EnforceConstraints) && !table._duringDataLoad)
+                               // we have to check that the new row doesn't colide with existing row
+                               ValidateDataRowInternal(row);
                                
                        if (pos >= list.Count)
                                list.Add (row);
                        else
                                list.Insert (pos, row);
+                               
+                       row.HasParentCollection = true;
+                       row.AttachRow ();
+                       row.Table.ChangedDataRow (row, DataRowAction.Add);
+               }
+
+               /// <summary>
+               /// Removes the specified DataRow from the internal list. Used by DataRow to commit the removing.
+               /// </summary>
+               internal void RemoveInternal (DataRow row) {
+                       if (row == null) {
+                               throw new IndexOutOfRangeException ("The given datarow is not in the current DataRowCollection.");
+                       }
+                       int index = list.IndexOf(row);
+                       if (index < 0) {
+                               throw new IndexOutOfRangeException ("The given datarow is not in the current DataRowCollection.");
+                       }
+                       list.RemoveAt(index);
                }
 
                /// <summary>
@@ -217,13 +281,16 @@ namespace System.Data
                /// </summary>
                public void Remove (DataRow row) 
                {
-                       // FIXME: This is the way the MS.NET handles this kind of situation. Could be better, but what can you do.
-                       if (row == null || list.IndexOf (row) == -1)
+                       if (row == null)
                                throw new IndexOutOfRangeException ("The given datarow is not in the current DataRowCollection.");
-
-                       list.Remove (row);
-                       row.DetachRow ();
-                       table.DeletedDataRow (row, DataRowAction.Delete);
+                       int index = list.IndexOf(row);
+                       if (index < 0)
+                               throw new IndexOutOfRangeException ("The given datarow is not in the current DataRowCollection.");
+                       row.Delete();
+                       // if the row was in added state it will be in Detached state after the
+                       // delete operation, so we have to check it.
+                       if (row.RowState != DataRowState.Detached)
+                               row.AcceptChanges();
                }
 
                /// <summary>
@@ -233,10 +300,9 @@ namespace System.Data
                {                       
                        if (index < 0 || index >= list.Count)
                                throw new IndexOutOfRangeException ("There is no row at position " + index + ".");
-
                        DataRow row = (DataRow)list [index];
-                       list.RemoveAt (index);                  
-                       table.DeletedDataRow (row, DataRowAction.Delete);
+                       row.Delete();
+                       row.AcceptChanges();
                }
 
                ///<summary>
@@ -246,17 +312,48 @@ namespace System.Data
                [MonoTODO]
                internal void ValidateDataRowInternal(DataRow row)
                {
-                       //FIXME: this validates constraints in the order they appear
-                       //in the collection. Most probably we need to do it in a 
-                       //specific order like unique/primary keys first, then Foreignkeys, etc
-                       foreach(Constraint constraint in table.Constraints)
-                       {
-                               constraint.AssertConstraint(row);
+                       //first check for null violations.
+                       row.CheckNullConstraints();
+                       // This validates constraints in the specific order : 
+                       // first unique/primary keys first, then Foreignkeys, etc
+                       ArrayList uniqueConstraintsDone = new ArrayList();
+                       ArrayList foreignKeyConstraintsDone = new ArrayList();
+                       try {
+                               foreach(Constraint constraint in table.Constraints.UniqueConstraints) {
+                                       constraint.AssertConstraint(row);
+                                       uniqueConstraintsDone.Add(constraint);
+                               }
+                       
+                               foreach(Constraint constraint in table.Constraints.ForeignKeyConstraints) {
+                                       constraint.AssertConstraint(row);
+                                       foreignKeyConstraintsDone.Add(constraint);
+                               }
+                       }
+                       // if one of the AssertConstraint failed - we need to "rollback" all the changes
+                       // caused by AssertCoinstraint calls already succeeded
+                       catch(ConstraintException e) {
+                               RollbackAsserts(row,foreignKeyConstraintsDone,uniqueConstraintsDone);
+                               throw e;
+                       }
+                       catch(InvalidConstraintException e) {   
+                               RollbackAsserts(row,foreignKeyConstraintsDone,uniqueConstraintsDone);
+                               throw e;
                        }
-
                }
-               
-       }
 
+               private void RollbackAsserts(DataRow row,ICollection foreignKeyConstraintsDone,
+                       ICollection uniqueConstraintsDone)
+               {
+                       // if any of constraints assert failed - 
+                       // we have to rollback all the asserts scceeded
+                       // on order reverse to thier original execution
+                       foreach(Constraint constraint in foreignKeyConstraintsDone) {
+                               constraint.RollbackAssert(row);
+                       }
 
+                       foreach(Constraint constraint in uniqueConstraintsDone) {
+                               constraint.RollbackAssert(row);
+                       }
+               }
+       }
 }