From 7c80047c8249222747ed3731a49efacc24123080 Mon Sep 17 00:00:00 2001
From: Bob Evans <bizob2828@yahoo.com>
Date: Tue, 14 Apr 2015 15:43:09 -0400
Subject: [PATCH] removed default of true on track opens/clicks

Conflicts:
	test/test_transmissions.py
	tox.ini
---
 docs/resources/transmissions.rst            | 8 ++++++--
 examples/transmissions/send_transmission.py | 4 ++--
 sparkpost/transmissions.py                  | 4 ++--
 test/test_transmissions.py                  | 2 --
 4 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/docs/resources/transmissions.rst b/docs/resources/transmissions.rst
index b4e3c4f..564ec5e 100644
--- a/docs/resources/transmissions.rst
+++ b/docs/resources/transmissions.rst
@@ -13,7 +13,9 @@ Here at SparkPost, our messages are known as transmissions. Let's use the underl
         recipients=['someone@somedomain.com'],
         html='<p>Hello world</p>',
         from_email='test@sparkpostbox.com',
-        subject='Hello from python-sparkpost'
+        subject='Hello from python-sparkpost',
+        track_opens=True,
+        track_clicks=True
     )
 
     print response
@@ -46,7 +48,9 @@ Using inline templates and/or recipients
         text="Hello world",
         html='<p>Hello world</p>',
         from_email='test@sparkpostbox.com',
-        subject='Hello from python-sparkpost'
+        subject='Hello from python-sparkpost',
+        track_opens=True,
+        track_clicks=True
     )
 
 
diff --git a/examples/transmissions/send_transmission.py b/examples/transmissions/send_transmission.py
index f1ea4c2..2c34717 100644
--- a/examples/transmissions/send_transmission.py
+++ b/examples/transmissions/send_transmission.py
@@ -21,8 +21,8 @@
   custom_headers = {
     'X-CUSTOM-HEADER': 'foo bar'
   },
-  track_opens = False,
-  track_clicks = False,
+  track_opens = True,
+  track_clicks = True,
   campaign = 'sdk example',
   metadata = {
     'key': 'value',
diff --git a/sparkpost/transmissions.py b/sparkpost/transmissions.py
index 564c96d..9e3c5b0 100644
--- a/sparkpost/transmissions.py
+++ b/sparkpost/transmissions.py
@@ -26,8 +26,8 @@ def _translate_keys(self, **kwargs):
         model['metadata'] = kwargs.get('metadata')
         model['substitution_data'] = kwargs.get('substitution_data')
 
-        model['options']['open_tracking'] = kwargs.get('track_opens', True)
-        model['options']['click_tracking'] = kwargs.get('track_clicks', True)
+        model['options']['open_tracking'] = kwargs.get('track_opens')
+        model['options']['click_tracking'] = kwargs.get('track_clicks')
         model['options']['sandbox'] = kwargs.get('use_sandbox')
 
         model['content']['use_draft_template'] = \
diff --git a/test/test_transmissions.py b/test/test_transmissions.py
index 5b2b69d..2a81e3d 100644
--- a/test/test_transmissions.py
+++ b/test/test_transmissions.py
@@ -10,8 +10,6 @@ def test_translate_keys_with_list():
     t = Transmissions('uri', 'key')
     results = t._translate_keys(recipient_list='test')
     assert results['return_path'] == 'default@sparkpostmail.com'
-    assert results['options']['open_tracking'] is True
-    assert results['options']['click_tracking'] is True
     assert results['content']['use_draft_template'] is False
     assert results['recipients'] == {'list_id': 'test'}