[SHELL32] - CDefViewBckgrndMenu: Shell_MergeMenus doesn't handle items added by shell...

svn path=/trunk/; revision=73407

diff --git a/reactos/dll/win32/shell32/CDefViewBckgrndMenu.cpp b/reactos/dll/win32/shell32/CDefViewBckgrndMenu.cpp
index 1e1bacd..14de3de 100644 (file)
--- a/reactos/dll/win32/shell32/CDefViewBckgrndMenu.cpp
+++ b/reactos/dll/win32/shell32/CDefViewBckgrndMenu.cpp
@@ -170,16 +170,25 @@ CDefViewBckgrndMenu::QueryContextMenu(HMENU hMenu, UINT indexMenu, UINT idCmdFir
         hr = m_psf->CreateViewObject(NULL, IID_PPV_ARG(IContextMenu, &m_folderCM));
         if (SUCCEEDED(hr))
         {
-            hr = m_folderCM->QueryContextMenu(hMenuPart, 0, idCmdFirst, idCmdLast, uFlags);
+            InsertMenuA(hMenu, indexMenu++, MF_BYPOSITION | MF_SEPARATOR, 0, NULL);
+            UINT SeparatorIndex = indexMenu;
+            UINT count = GetMenuItemCount(hMenu);
+
+            hr = m_folderCM->QueryContextMenu(hMenu, indexMenu, idCmdFirst, idCmdLast, uFlags);
             if (SUCCEEDED(hr))
             {
-                Shell_MergeMenus(hMenu, hMenuPart, indexMenu, 0, 0xFFFF, MM_DONTREMOVESEPS | MM_SUBMENUSHAVEIDS);
-                DestroyMenu(hMenuPart);
+                //Shell_MergeMenus(hMenu, hMenuPart, indexMenu, 0, UINT_MAX, MM_ADDSEPARATOR| MM_DONTREMOVESEPS | MM_SUBMENUSHAVEIDS);
+                //DestroyMenu(hMenuPart);
             }
             else
             {
                 WARN("QueryContextMenu failed!\n");
             }
+
+            /* If no item was added after the separator, remove it */
+            if (count == GetMenuItemCount(hMenu))
+                DeleteMenu(hMenu, SeparatorIndex, MF_BYPOSITION);
+
         }
         else
         {