Added modify to resource.py in ResourceBase.

Author: Paul Breaux

f5/bigip/resource.py

@@ -438,12 +438,53 @@ def __init__(self, container):
         """
         super(ResourceBase, self).__init__(container)
 
-    def _update(self, **kwargs):
-        """wrapped with update, override that in a subclass to customize"""
+    def _modify(self, **kwargs):
+        """Wrapped with modify, override in a subclass to customize."""
+
+        requests_params, patch_uri, session, read_only = \
+            self._prepare_put_or_patch(kwargs)
+        self._check_for_boolean_pair_reduction(kwargs)
+        data_dict = self._pop_read_only_attrs(kwargs, read_only)
+        data_dict.update(kwargs)
+        response = session.patch(patch_uri, json=data_dict, **requests_params)
+        self._local_update(response.json())
+
+    def modify(self, **kwargs):
+        """Modify the configuration of the resource on device based on kwargs
+
+        """
+
+        self._modify(**kwargs)
+
+    def _check_for_boolean_pair_reduction(self, kwargs):
+        """Check if boolean pairs should be reduced in this resource."""
+
+        if 'reduction_forcing_pairs' in self._meta_data:
+            for key1, key2 in self._meta_data['reduction_forcing_pairs']:
+                kwargs = self._reduce_boolean_pair(kwargs, key1, key2)
+        return kwargs
+
+    def _prepare_put_or_patch(self, kwargs):
+        """Retrieve the appropriate request items for put or patch calls."""
+
         requests_params = self._handle_requests_params(kwargs)
         update_uri = self._meta_data['uri']
         session = self._meta_data['bigip']._meta_data['icr_session']
         read_only = self._meta_data.get('read_only_attributes', [])
+        return requests_params, update_uri, session, read_only
+
+    def _pop_read_only_attrs(self, kwargs, read_only):
+        """Remove any read-only attributes from kwargs"""
+
+        for attr in read_only:
+            kwargs.pop(attr, '')
+        return kwargs
+
+    def _update(self, **kwargs):
+        """wrapped with update, override that in a subclass to customize"""
+
+        requests_params, update_uri, session, read_only = \
+            self._prepare_put_or_patch(kwargs)
 
         # Get the current state of the object on BIG-IP® and check the
         # generation Use pop here because we don't want force in the data_dict
@@ -452,10 +493,7 @@ def _update(self, **kwargs):
             # generation has a known server-side error
             self._check_generation()
 
-        # Reduce any boolean pairs as specified by the meta_data entry below
-        if 'reduction_forcing_pairs' in self._meta_data:
-            for key1, key2 in self._meta_data['reduction_forcing_pairs']:
-                kwargs = self._reduce_boolean_pair(kwargs, key1, key2)
+        kwargs = self._check_for_boolean_pair_reduction(kwargs)
 
         # Save the meta data so we can add it back into self after we
         # load the new object.
@@ -474,8 +512,7 @@ def _update(self, **kwargs):
         # the data dict with the attributes.  If they pass in read-only attrs
         # in the method call we are going to let BIG-IP® let them know about it
         # when it fails
-        for attr in read_only:
-            data_dict.pop(attr, '')
+        data_dict = self._pop_read_only_attrs(data_dict, read_only)
 
         data_dict.update(kwargs)