Strip whitespace from IDs, filter empty update patches, fix README ids[0] reference, use DataFrame index labels in errors

Author: Saurabh Badenkal

README.md

@@ -263,7 +263,7 @@ new_accounts["telephone1"] = ["555-0199", "555-0299"]
 client.dataframe.update("account", new_accounts, id_column="accountid")
 
 # Clear a field by setting clear_nulls=True (by default, NaN/None fields are skipped)
-df = pd.DataFrame([{"accountid": ids[0], "websiteurl": None}])
+df = pd.DataFrame([{"accountid": new_accounts["accountid"].iloc[0], "websiteurl": None}])
 client.dataframe.update("account", df, id_column="accountid", clear_nulls=True)
 
 # Delete records by passing a Series of GUIDs

src/PowerPlatform/Dataverse/operations/dataframe.py

@@ -245,13 +245,14 @@ def update(
         if id_column not in changes.columns:
             raise ValueError(f"id_column '{id_column}' not found in DataFrame columns")
 
-        ids = changes[id_column].tolist()
-        invalid = [i for i, v in enumerate(ids) if not isinstance(v, str) or not v.strip()]
+        raw_ids = changes[id_column].tolist()
+        invalid = [changes.index[i] for i, v in enumerate(raw_ids) if not isinstance(v, str) or not v.strip()]
         if invalid:
             raise ValueError(
                 f"id_column '{id_column}' contains invalid values at row index(es) {invalid}. "
                 "All IDs must be non-empty strings."
             )
+        ids = [v.strip() for v in raw_ids]
 
         change_columns = [column for column in changes.columns if column != id_column]
         if not change_columns:
@@ -260,10 +261,18 @@ def update(
             )
         change_list = dataframe_to_records(changes[change_columns], na_as_null=clear_nulls)
 
-        if len(ids) == 1:
-            self._client.records.update(table, ids[0], change_list[0])
+        # Filter out rows where all change values were NaN/None (empty dicts)
+        paired = [(rid, patch) for rid, patch in zip(ids, change_list) if patch]
+        if not paired:
+            return
+        ids_filtered, change_filtered = zip(*paired)
+        ids_filtered = list(ids_filtered)
+        change_filtered = list(change_filtered)
+
+        if len(ids_filtered) == 1:
+            self._client.records.update(table, ids_filtered[0], change_filtered[0])
         else:
-            self._client.records.update(table, ids, change_list)
+            self._client.records.update(table, ids_filtered, change_filtered)
 
     # ----------------------------------------------------------------- delete
 
@@ -299,15 +308,16 @@ def delete(
         if not isinstance(ids, pd.Series):
             raise TypeError("ids must be a pandas Series")
 
-        id_list = ids.tolist()
-        if not id_list:
+        raw_list = ids.tolist()
+        if not raw_list:
             return None
 
-        invalid = [i for i, v in enumerate(id_list) if not isinstance(v, str) or not v.strip()]
+        invalid = [ids.index[i] for i, v in enumerate(raw_list) if not isinstance(v, str) or not v.strip()]
         if invalid:
             raise ValueError(
                 f"ids Series contains invalid values at index(es) {invalid}. " "All IDs must be non-empty strings."
             )
+        id_list = [v.strip() for v in raw_list]
 
         if len(id_list) == 1:
             self._client.records.delete(table, id_list[0])