Moving a menu item tree can potentially corrupt the order values at the level where the root menu item is inserted

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: 7.1.X, 7.2.X, 7.3.X, Master
• Fix Version/s: 7.1.10 DXP FP23, 7.1.10.6 SP6, 7.1.X, 7.2.10 DXP FP11, 7.2.10.4 DXP SP4, 7.2.X, 7.3.10 DXP FP1, 7.3.10.1 DXP SP1, 7.3.6 CE GA7, 7.3.X, 7.4.13 DXP GA1, Master
• Component/s: Navigation Menus
• Labels:
  • backend_test
  • ee-ts
  • liferay-dxp-71-sp6
  • liferay-dxp-72-sp4
  • liferay-dxp-73-sp1
  • liferay-fixpack-dxp-1-7310
  • liferay-fixpack-dxp-11-7210
  • liferay-fixpack-dxp-23-7110
• Branch Version/s:
  7.3.x, 7.2.x, 7.1.x
• Backported to Branch:
  Committed
• Fix Priority: 3
• Git Pull Request: https://github.com/liferay-echo/liferay-portal/pull/3519

Reproduction Steps:

1. Setup Liferay DXP 7.1
2. Create a new Site
3. Create the following page hierarchy
   1. Home
   2. Search
   3. P
      1. P1
         1. P2
4. Create a new navigation menu nm1
5. Add Home and Search as navigation menu items to nm1 in a row
6. Add P, P1 and P2 as navigation menu items to nm1 in a row
7. Check the order

Expected results:

• Navigation items follow the page hierarchy
• SELECT order_ FROM SiteNavigationMenuItem ORDER BY siteNavigationMenuItemId returns 0, 1, 2, 0, 0

Actual results:

• Navigation items may or not follow the page hierarchy. If not, the steps can be repeated or renaming one of the first-level items may lead to an unexpected order.
• SELECT order_ FROM SiteNavigationMenuItem ORDER BY siteNavigationMenuItemId returns 0, 0, 0, 0, 0