Added missing ISortableAction implementation

diff --git a/wcfsetup/install/files/lib/data/menu/item/MenuItemAction.class.php b/wcfsetup/install/files/lib/data/menu/item/MenuItemAction.class.php
index 1e820a031aa9499b48d7f4a6186547fd5e771d2a..265f57aab3586c7cb9d0df5566cdce72158fceb8 100644 (file)
--- a/wcfsetup/install/files/lib/data/menu/item/MenuItemAction.class.php
+++ b/wcfsetup/install/files/lib/data/menu/item/MenuItemAction.class.php
@@ -1,8 +1,12 @@
 <?php
 namespace wcf\data\menu\item;
 use wcf\data\AbstractDatabaseObjectAction;
+use wcf\data\ISortableAction;
 use wcf\data\IToggleAction;
+use wcf\data\menu\Menu;
 use wcf\system\exception\PermissionDeniedException;
+use wcf\system\exception\UserInputException;
+use wcf\system\WCF;
 
 /**
  * Executes menu item related actions.
@@ -15,7 +19,7 @@ use wcf\system\exception\PermissionDeniedException;
  * @category   Community Framework
  * @since      2.2
  */
-class MenuItemAction extends AbstractDatabaseObjectAction implements IToggleAction {
+class MenuItemAction extends AbstractDatabaseObjectAction implements ISortableAction, IToggleAction {
        /**
         * @inheritDoc
         */
@@ -62,4 +66,67 @@ class MenuItemAction extends AbstractDatabaseObjectAction implements IToggleActi
                        $object->update(array('isDisabled' => ($object->isDisabled) ? 0 : 1));
                }
        }
+       
+       /**
+        * @inheritDoc
+        */
+       public function validateUpdatePosition() {
+               WCF::getSession()->checkPermissions(['admin.content.cms.canManageMenu']);
+               
+               // validate menu id
+               $this->readInteger('menuID');
+               $menu = new Menu($this->parameters['menuID']);
+               if (!$menu->menuID) {
+                       throw new UserInputException('menuID');
+               }
+               
+               // validate structure
+               if (!isset($this->parameters['data']) || !isset($this->parameters['data']['structure']) || !is_array($this->parameters['data']['structure'])) {
+                       throw new UserInputException('structure');
+               }
+               
+               $menuItemIDs = [];
+               foreach ($this->parameters['data']['structure'] as $menuItems) {
+                       $menuItemIDs = array_merge($menuItemIDs, $menuItems);
+               }
+               
+               $menuItemList = new MenuItemList();
+               $menuItemList->getConditionBuilder()->add('menu_item.itemID IN (?)', [$menuItemIDs]);
+               $menuItemList->getConditionBuilder()->add('menu_item.menuID = ?', [$this->parameters['menuID']]);
+               $menuItemList->readObjects();
+               $menuItems = $menuItemList->getObjects();
+               
+               if (count($menuItems) != count($menuItemIDs)) {
+                       throw new UserInputException('structure');
+               }
+               
+               foreach ($this->parameters['data']['structure'] as $parentItemID => $children) {
+                       if ($parentItemID && !isset($menuItems[$parentItemID])) {
+                               throw new UserInputException('structure');
+                       }
+               }
+       }
+       
+       /**
+        * @inheritDoc
+        */
+       public function updatePosition() {
+               $sql = "UPDATE  wcf".WCF_N."_menu_item
+                       SET     parentItemID = ?,
+                               showOrder = ?
+                       WHERE   itemID = ?";
+               $statement = WCF::getDB()->prepareStatement($sql);
+               
+               WCF::getDB()->beginTransaction();
+               foreach ($this->parameters['data']['structure'] as $parentItemID => $children) {
+                       foreach ($children as $showOrder => $menuItemID) {
+                               $statement->execute([
+                                       ($parentItemID ?: null),
+                                       $showOrder + 1,
+                                       $menuItemID
+                               ]);
+                       }
+               }
+               WCF::getDB()->commitTransaction();
+       }
 }