Add refresh opt-out on Item.modify_metadata()

jjjake · Apr 19, 2024 · c17a9cd · c17a9cd
1 parent ac05934
commit c17a9cd
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/internetarchive/api.py b/internetarchive/api.py
@@ -209,6 +209,7 @@ def modify_metadata(
         secret_key=secret_key,
         debug=debug,
         request_kwargs=request_kwargs,
+        refresh=False
     )
 
 

diff --git a/internetarchive/cli/ia_metadata.py b/internetarchive/cli/ia_metadata.py
@@ -91,7 +91,7 @@ def modify_metadata(item: item.Item, metadata: Mapping, args: Mapping) -> Respon
         r = item.modify_metadata(metadata, target=args['--target'], append=append,
                                  expect=expect, priority=args['--priority'],
                                  append_list=append_list, headers=args['--header'],
-                                 insert=insert, timeout=args['--timeout'])
+                                 insert=insert, timeout=args['--timeout'], refresh=False)
         assert isinstance(r, Response)  # mypy: modify_metadata() -> Request | Response
     except ItemLocateError as exc:
         print(f'{item.identifier} - error: {exc}', file=sys.stderr)

diff --git a/internetarchive/item.py b/internetarchive/item.py
@@ -779,7 +779,8 @@ def modify_metadata(self,
                         debug: bool = False,
                         headers: Mapping | None = None,
                         request_kwargs: Mapping | None = None,
-                        timeout: int | float | None = None) -> Request | Response:
+                        timeout: int | float | None = None,
+                        refresh: bool = True) -> Request | Response:
         """Modify the metadata of an existing item on Archive.org.
 
         Note: The Metadata Write API does not yet comply with the
@@ -801,6 +802,8 @@ def modify_metadata(self,
         :param append_list: Append values to an existing multi-value
                             metadata field. No duplicate values will be added.
 
+        :param refresh: Refresh the item metadata after the request.
+
         :returns: A Request if debug else a Response.
 
         Usage::
@@ -850,7 +853,8 @@ def modify_metadata(self,
             return prepared_request
         resp = self.session.send(prepared_request, **request_kwargs)
         # Re-initialize the Item object with the updated metadata.
-        self.refresh()
+        if refresh:
+            self.refresh()
         return resp
 
     # TODO: `list` parameter name shadows the Python builtin