Remove CMS synchronization until a better implementation can be made

Author: RBech

code/Model/Categorypage.php

@@ -74,12 +74,9 @@ public function categoryAction()
     {
         $this->authenticate();
 
-        $pageparam = $this->getIntParam('page');
+        $page = $this->getIntParam('page');
         $limit = $this->getIntParam('limit');
 
-        $collection = new Varien_Data_Collection();
-        $paginator = new Clerk_Clerk_Model_Paginator($collection);
-
         $rootCategoryId = Mage::app()->getStore()->getRootCategoryId();
 
         $categories = Mage::getModel('catalog/category')
@@ -89,7 +86,9 @@ public function categoryAction()
             ->addAttributeToFilter('path', array('like' => "1/{$rootCategoryId}/%"))
             ->setOrder('entity_id', Varien_Db_Select::SQL_ASC)
             ->setPageSize($limit)
-            ->setCurPage($pageparam);
+            ->setCurPage($page + 1);
+
+        $items = [];
 
         foreach ($categories as $category) {
             //Get children categories
@@ -102,48 +101,14 @@ public function categoryAction()
                 'subcategories' => array_map('intval', $children),
             );
 
-            $item = new Varien_Object();
-            $item->setData($data);
-
-            $collection->addItem($item);
-        }
-
-        if (Mage::getStoreConfigFlag('clerk/general/sync_cms_pages')) {
-            $pages = Mage::getModel('cms/page')
-                ->getCollection()
-                ->addFieldToFilter('is_active', '1')
-                ->addStoreFilter(Mage::app()->getStore()->getId());
-
-            foreach ($pages as $page) {
-                $data = array(
-                    'id' => (int) $page->getId() + 10000,
-                    'name' => $page->getTitle(),
-                    'url' => Mage::helper('cms/page')->getPageUrl($page->getId()),
-                    'subcategories' => [],
-                );
-
-                $item = new Varien_Object();
-                $item->setData($data);
-
-                $collection->addItem($item);
-            }
+            $items[] = $data;
         }
 
-        $collection->setPageSize($limit);
-        $collection->setCurPage($pageparam + 1);
-
-        $this->getResponse()->setHeader('Total-Page-Count', $collection->getLastPageNumber() - 1);
+        $this->getResponse()->setHeader('Total-Page-Count', $categories->getLastPageNumber() - 1);
 
-        if ($pageparam > $collection->getLastPageNumber() - 1) {
+        if ($page > $categories->getLastPageNumber() - 1) {
             $this->getResponse()->setBody(json_encode([]));
         } else {
-            $iterator = $paginator->getIterator();
-
-            $items = [];
-            foreach ($iterator as $item) {
-                $items[] = $item->toArray();
-            }
-
             $this->getResponse()->setBody(json_encode($items));
         }
     }

code/Model/Paginator.php

@@ -2,7 +2,7 @@
 <config>
     <modules>
         <Clerk_Clerk>
-            <version>3.4.6</version>
+            <version>3.4.7</version>
         </Clerk_Clerk>
     </modules>
     <global>

code/controllers/ApiController.php

@@ -76,13 +76,6 @@
                             <show_in_store>1</show_in_store>
                             <source_model>adminhtml/system_config_source_yesno</source_model>
                         </disable_order_synchronization>
-                        <sync_cms_pages>
-                            <label>Send CMS pages</label>
-                            <frontend_type>select</frontend_type>
-                            <sort_order>50</sort_order>
-                            <show_in_store>1</show_in_store>
-                            <source_model>adminhtml/system_config_source_yesno</source_model>
-                        </sync_cms_pages>
                         <url>
                             <label>Import url</label>
                             <frontend_type>select</frontend_type>