bug in designer (w/ fix)

Creating a new menu item wih ID 57616 should result in a special MRU control, but it won't because in CXTPControls::Add in XTPControls.cpp the code does not match the code found in CXTPControls::AddMenuItem

AddMenuItem has:

                if (nID == XTP_ID_WINDOWLIST) pControl = (CXTPControl*)CXTPControlWindowList::CreateObject();
                else if (nID == XTP_ID_WORKSPACE_ACTIONS) pControl = (CXTPControl*)CXTPControlWorkspaceActions::CreateObject() ;
                else if (nID == ID_VIEW_TOOLBAR) pControl = (CXTPControl*)CXTPControlToolbars::CreateObject();
                else if (nID == ID_OLE_VERB_FIRST && strCaption == _T("<<OLE VERBS GO HERE>>")) pControl = (CXTPControl*)CXTPControlOleItems::CreateObject();
                else if (nID == ID_FILE_MRU_FILE1) pControl = (CXTPControl*)CXTPControlRecentFileList::CreateObject();
                else pControl = (CXTPControl*)CXTPControlButton::CreateObject();

but Add only has:

            if (nId == XTP_ID_WINDOWLIST) pControl = (CXTPControl*)CXTPControlWindowList::CreateObject();
            else if (nId == XTP_ID_WORKSPACE_ACTIONS) pControl = (CXTPControl*)CXTPControlWorkspaceActions::CreateObject() ;
            else if (nId == ID_VIEW_TOOLBAR) pControl = (CXTPControl*)CXTPControlToolbars::CreateObject();
            else
            pControl = (CXTPControl*)CXTPControlButton::CreateObject();

so copying the basic code regarding ID_FILE_MRU_FILE1 should fix this problem.

well, its sort of a fix, since it causes the app to lock up when you click on the File menu after that...


Edited by Ark42

Can a fix like this go into the next version? I added XTP_ID_FILE_MRU to my copy and recompiled and it works fine now, I can create a MRU menu via the designer editor and it works, but only with this modification.