[Confluence] Add new move page methods #326 (#436)

* [Confluence] Evaluate the move page method

Author: gonchik

atlassian/confluence.py

@@ -416,6 +416,23 @@ def create_page(self, space, title, body, parent_id=None, type='page',
             data['ancestors'] = [{'type': type, 'id': parent_id}]
         return self.post(url, data=data)
 
+    def move_page(self, space_key, page_id, target_title, position="append"):
+        """
+        Move page method
+        :param space_key:
+        :param page_id:
+        :param target_title:
+        :param position: topLevel or append
+        :return:
+        """
+        url = "/pages/movepage.action"
+        params = {"spaceKey": space_key, "pageId": page_id}
+        if target_title:
+            params["targetTitle"] = target_title
+        if position:
+            params["position"] = position
+        return self.get(url, params=params, headers=self.no_check_headers)
+
     def get_all_spaces(self, start=0, limit=500, expand=None):
         """
         Get all spaces with provided limit
@@ -714,12 +731,12 @@ def is_page_content_is_already_updated(self, page_id, body, title=None):
         """
         if self.advanced_mode:
             confluence_content = (((self.get_page_by_id(page_id, expand='body.storage').json() or {})
-                    .get('body') or {})
-                    .get('storage') or {})
+                                   .get('body') or {})
+                                  .get('storage') or {})
         else:
             confluence_content = (((self.get_page_by_id(page_id, expand='body.storage') or {})
-                    .get('body') or {})
-                    .get('storage') or {})
+                                   .get('body') or {})
+                                  .get('storage') or {})
 
         if title:
             current_title = confluence_content.get('title', None)
@@ -793,8 +810,9 @@ def update_page(self, page_id, title, body, parent_id=None, type='page', represe
 
             return self.put('rest/api/content/{0}'.format(page_id), data=data)
 
-    def _insert_to_existing_page(self, page_id, title, insert_body, parent_id=None, type='page', representation='storage',
-                    minor_edit=False, top_of_page=False):
+    def _insert_to_existing_page(self, page_id, title, insert_body, parent_id=None, type='page',
+                                 representation='storage',
+                                 minor_edit=False, top_of_page=False):
         """
         Insert body to a page if already exist
         :param parent_id:
@@ -849,11 +867,12 @@ def append_page(self, page_id, title, append_body, parent_id=None, type='page',
         """
         log.info('Updating {type} "{title}"'.format(title=title, type=type))
 
-        return self._insert_to_existing_page(page_id, title, append_body, parent_id=parent_id, type=type, representation=representation,
-                    minor_edit=minor_edit, top_of_page=False)
+        return self._insert_to_existing_page(page_id, title, append_body, parent_id=parent_id, type=type,
+                                             representation=representation,
+                                             minor_edit=minor_edit, top_of_page=False)
 
     def prepend_page(self, page_id, title, prepend_body, parent_id=None, type='page', representation='storage',
-                    minor_edit=False):
+                     minor_edit=False):
         """
         Append body to page if already exist
         :param parent_id:
@@ -868,8 +887,9 @@ def prepend_page(self, page_id, title, prepend_body, parent_id=None, type='page'
         """
         log.info('Updating {type} "{title}"'.format(title=title, type=type))
 
-        return self._insert_to_existing_page(page_id, title, prepend_body, parent_id=parent_id, type=type, representation=representation,
-                    minor_edit=minor_edit, top_of_page=True)
+        return self._insert_to_existing_page(page_id, title, prepend_body, parent_id=parent_id, type=type,
+                                             representation=representation,
+                                             minor_edit=minor_edit, top_of_page=True)
 
     def update_or_create(self, parent_id, title, body, representation='storage', minor_edit=False):
         """

atlassian/rest_client.py

@@ -18,7 +18,7 @@ class AtlassianRestAPI(object):
                             'X-ExperimentalApi': 'opt-in'}
     form_token_headers = {'Content-Type': 'application/x-www-form-urlencoded; charset=UTF-8',
                           'X-Atlassian-Token': 'no-check'}
-
+    no_check_headers = {'X-Atlassian-Token': 'no-check'}
     response = None
 
     def __init__(self, url, username=None, password=None, timeout=60, api_root='rest/api', api_version='latest',

docs/confluence.rst

@@ -99,6 +99,9 @@ Page actions
     # Delete the page (content) property e.g. delete key of hash
     confluence.delete_page_property(page_id, page_property)
 
+    # Move page
+    confluence.move_page(space_key, page_id, target_title, position="append")
+
     # Get the page (content) property e.g. get key of hash
     confluence.get_page_property(page_id, page_property_key)