[ros-diffs] [cfinck] 37178: Merge r37177 This should be the very final now, no more bugs accepted ; -)

Author: cfinck
Date: Mon Nov  3 16:01:11 2008
New Revision: 37178

URL: http://svn.reactos.org/svn/reactos?rev=37178&view=rev
Log:
Merge r37177
This should be the very final now, no more bugs accepted ;-)

Modified:
    branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shfldr_cpanel.c
    branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shlview.c

Modified: branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shfldr_cpanel.c
URL:
http://svn.reactos.org/svn/reactos/branches/ros-branch-0_3_7/reactos/dll/wi…
==============================================================================
--- branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shfldr_cpanel.c [iso-8859-1]
(original)
+++ branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shfldr_cpanel.c [iso-8859-1] Mon
Nov  3 16:01:11 2008
@@ -1212,7 +1212,7 @@
     if (LoadStringW(shell32_hInstance, IDS_OPEN, szBuffer,
sizeof(szBuffer)/sizeof(WCHAR)))
     {
         szBuffer[(sizeof(szBuffer)/sizeof(WCHAR))-1] = L'\0';
-        _InsertMenuItemW(hMenu, indexMenu++, TRUE, idCmdFirst + Count, MFT_STRING,
szBuffer, MFS_DEFAULT);
+        _InsertMenuItemW(hMenu, indexMenu++, TRUE, IDS_OPEN, MFT_STRING, szBuffer,
MFS_DEFAULT); //FIXME identifier
         Count++;
     }
 
@@ -1224,7 +1224,7 @@
         }
         szBuffer[(sizeof(szBuffer)/sizeof(WCHAR))-1] = L'\0';
 
-        _InsertMenuItemW(hMenu, indexMenu++, TRUE, idCmdFirst + Count, MFT_STRING,
szBuffer, MFS_ENABLED);
+        _InsertMenuItemW(hMenu, indexMenu++, TRUE, IDS_CREATELINK, MFT_STRING, szBuffer,
MFS_ENABLED); //FIXME identifier
         Count++;
     }
     return MAKE_HRESULT(SEVERITY_SUCCESS, 0, Count);
@@ -1251,7 +1251,7 @@
 
     TRACE("(%p)->(invcom=%p verb=%p wnd=%p)\n",This,lpcmi,lpcmi->lpVerb,
lpcmi->hwnd);
 
-    if (lpcmi->lpVerb == MAKEINTRESOURCEA(1))
+    if (lpcmi->lpVerb == MAKEINTRESOURCEA(IDS_OPEN)) //FIXME
     {
        ZeroMemory(&sei, sizeof(sei));
        sei.cbSize = sizeof(sei);
@@ -1264,7 +1264,7 @@
        if (sei.hInstApp <= (HINSTANCE)32)
           return E_FAIL;
     }
-    else if (lpcmi->lpVerb == MAKEINTRESOURCEA(2))
+    else if (lpcmi->lpVerb == MAKEINTRESOURCEA(IDS_CREATELINK)) //FIXME
     {
         if (!SHGetSpecialFolderPathW(NULL, szPath, CSIDL_DESKTOPDIRECTORY, FALSE))
              return E_FAIL;

Modified: branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shlview.c
URL:
http://svn.reactos.org/svn/reactos/branches/ros-branch-0_3_7/reactos/dll/wi…
==============================================================================
--- branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shlview.c [iso-8859-1] (original)
+++ branches/ros-branch-0_3_7/reactos/dll/win32/shell32/shlview.c [iso-8859-1] Mon Nov  3
16:01:11 2008
@@ -887,10 +887,12 @@
 					ici.lpVerb = MAKEINTRESOURCEA( def );
 					ici.hwnd = This->hWnd;
 
-					IContextMenu_InvokeCommand(cm, (LPCMINVOKECOMMANDINFO) &ici );
-					IContextMenu_Release(cm);
-					DestroyMenu( hmenu );
-					return S_OK;
+					if (IContextMenu_InvokeCommand(cm, (LPCMINVOKECOMMANDINFO) &ici ) == S_OK)
+					{
+						IContextMenu_Release(cm);
+						DestroyMenu( hmenu );
+						return S_OK;
+					}
 				}
 				
 			}