Merge pull request #741 from tableau/fix_740

Chris Shin · web-flow · commit 6550ae1da1c4 · 2020-11-20T09:44:18.000-08:00
Fixes data_acceleration field always in workbook update payload
diff --git a/tableauserverclient/server/request_factory.py b/tableauserverclient/server/request_factory.py
@@ -593,13 +593,12 @@ def update_req(self, workbook_item):
         if workbook_item.owner_id:
             owner_element = ET.SubElement(workbook_element, 'owner')
             owner_element.attrib['id'] = workbook_item.owner_id
-        if workbook_item.data_acceleration_config is not None and \
-                'acceleration_enabled' in workbook_item.data_acceleration_config:
+        if workbook_item.data_acceleration_config['acceleration_enabled'] is not None:
             data_acceleration_config = workbook_item.data_acceleration_config
             data_acceleration_element = ET.SubElement(workbook_element, 'dataAccelerationConfig')
             data_acceleration_element.attrib['accelerationEnabled'] = str(data_acceleration_config
                                                                           ["acceleration_enabled"]).lower()
-            if "accelerate_now" in data_acceleration_config:
+            if data_acceleration_config['accelerate_now'] is not None:
                 data_acceleration_element.attrib['accelerateNow'] = str(data_acceleration_config
                                                                         ["accelerate_now"]).lower()
 
