Confluence: add cloud support for delete_attachment_by_id and get_attachment_history (#1473)

Added the same fix done for get_content_history_by_version_number to
attachment functions for cloud servers.

Co-authored-by: mroelandts <[email protected]>

Author: mroelandts

atlassian/confluence.py

@@ -1449,9 +1449,11 @@ def delete_attachment_by_id(self, attachment_id, version):
         :param version: file version
         :return:
         """
-        return self.delete(
-            "rest/experimental/content/{id}/version/{versionId}".format(id=attachment_id, versionId=version)
-        )
+        if self.cloud:
+            url = "rest/api/content/{id}/version/{versionId}".format(id=attachment_id, versionId=version)
+        else:
+            url = "rest/experimental/content/{id}/version/{versionId}".format(id=attachment_id, versionId=version)
+        return self.delete(url)
 
     def remove_page_attachment_keep_version(self, page_id, filename, keep_last_versions):
         """
@@ -1488,7 +1490,10 @@ def get_attachment_history(self, attachment_id, limit=200, start=0):
         :return
         """
         params = {"limit": limit, "start": start}
-        url = "rest/experimental/content/{}/version".format(attachment_id)
+        if self.cloud:
+            url = "rest/api/content/{id}/version".format(id=attachment_id)
+        else:
+            url = "rest/experimental/content/{id}/version".format(id=attachment_id)
         return (self.get(url, params=params) or {}).get("results")
 
     # @todo prepare more attachments info