fixing Confluence.remove_page_attachment_keep_version() function to not delete the whole attachment (#367)

Author: simonrupf

atlassian/confluence.py

@@ -520,7 +520,7 @@ def remove_page_attachment_keep_version(self, page_id, filename, keep_last_versi
         attachment_versions = self.get_attachment_history(attachment.get("id"))
         while len(attachment_versions) > keep_last_versions:
             remove_version_attachment_number = attachment_versions[keep_last_versions].get('number')
-            self.delete_attachment(page_id=page_id, filename=filename, version=remove_version_attachment_number)
+            self.delete_attachment_by_id(attachment_id=attachment.get("id"), version=remove_version_attachment_number)
             log.info(
                 "Removed oldest version for {}, now versions equal more than {}".format(attachment.get('title'),
                                                                                         len(attachment_versions)))