diff --git a/shotgun_api3/shotgun.py b/shotgun_api3/shotgun.py
index cccc4a010..af5507e6a 100755
--- a/shotgun_api3/shotgun.py
+++ b/shotgun_api3/shotgun.py
@@ -1026,13 +1026,7 @@ def upload(self, entity_type, entity_id, path, field_name=None,
                     "not sure why.\nPath: %s\nUrl: %s\nError: %s" % (
                     path, url, str(result)))
 
-        # TODO:
-        # we changed the result string in the middle of 1.8 to return the id
-        # remove once everyone is > 1.8.3
-        r = str(result).split(":")
-        attachment_id = 0
-        if len(r) > 1:
-            attachment_id = int(str(result).split(":")[1].split("\n")[0])
+        attachment_id = int(str(result).split(":")[1].split("\n")[0])
         return attachment_id
 
     def download_attachment(self, attachment_id):