Add missing WXK constants for the control keys

[wxWidgets.git] / src / osx / menuitem_osx.cpp

diff --git a/src/osx/menuitem_osx.cpp b/src/osx/menuitem_osx.cpp
index 3d83c083567993982042c354227807833758b3b5..93ef5052867213a07960e6308f5bf7d1c922bb84 100644 (file)
--- a/src/osx/menuitem_osx.cpp
+++ b/src/osx/menuitem_osx.cpp
@@ -41,13 +41,14 @@ wxMenuItem::wxMenuItem(wxMenu *pParentMenu,
 
     // In other languages there is no difference in naming the Exit/Quit menu item between MacOS and Windows guidelines
     // therefore these item must not be translated
 
     // In other languages there is no difference in naming the Exit/Quit menu item between MacOS and Windows guidelines
     // therefore these item must not be translated

-    if ( wxStripMenuCodes(m_text).Upper() == wxT("EXIT") )
-        m_text = wxT("Quit\tCtrl+Q") ;
+    if (pParentMenu != NULL && !pParentMenu->GetNoEventsMode())
+        if ( wxStripMenuCodes(m_text).Upper() == wxT("EXIT") )
+            m_text = wxT("Quit\tCtrl+Q") ;

 
     m_radioGroup.start = -1;
     m_isRadioGroupStart = false;
 
 
     m_radioGroup.start = -1;
     m_isRadioGroupStart = false;
 

-    wxString text = wxStripMenuCodes(m_text);
+    wxString text = wxStripMenuCodes(m_text, (pParentMenu != NULL && pParentMenu->GetNoEventsMode()) ? wxStrip_Accel : wxStrip_All);

     if (text.IsEmpty() && !IsSeparator())
     {
         wxASSERT_MSG(wxIsStockID(GetId()), wxT("A non-stock menu item with an empty label?"));
     if (text.IsEmpty() && !IsSeparator())
     {
         wxASSERT_MSG(wxIsStockID(GetId()), wxT("A non-stock menu item with an empty label?"));


@@ -175,9 +176,9 @@ void wxMenuItem::UpdateItemBitmap()
     if ( !m_parentMenu )
         return;
 
     if ( !m_parentMenu )
         return;
 

-    if ( m_bitmap.Ok() )
+    if ( m_bitmap.IsOk() )

     {
     {

-        m_peer->SetBitmap( m_bitmap );
+        GetPeer()->SetBitmap( m_bitmap );

     }
 }
 
     }
 }
 


@@ -190,11 +191,11 @@ void wxMenuItem::UpdateItemStatus()
         return ;
 
     if ( IsCheckable() && IsChecked() )
         return ;
 
     if ( IsCheckable() && IsChecked() )

-        m_peer->Check( true );
+        GetPeer()->Check( true );

     else
     else

-        m_peer->Check( false );
+        GetPeer()->Check( false );

 
 

-    m_peer->Enable( IsEnabled() );
+    GetPeer()->Enable( IsEnabled() );

 }
 
 void wxMenuItem::UpdateItemText()
 }
 
 void wxMenuItem::UpdateItemText()


@@ -202,7 +203,7 @@ void wxMenuItem::UpdateItemText()
     if ( !m_parentMenu )
         return ;
 
     if ( !m_parentMenu )
         return ;
 

-    wxString text = wxStripMenuCodes(m_text);
+    wxString text = wxStripMenuCodes(m_text, m_parentMenu != NULL && m_parentMenu->GetNoEventsMode() ? wxStrip_Accel : wxStrip_All);

     if (text.IsEmpty() && !IsSeparator())
     {
         wxASSERT_MSG(wxIsStockID(GetId()), wxT("A non-stock menu item with an empty label?"));
     if (text.IsEmpty() && !IsSeparator())
     {
         wxASSERT_MSG(wxIsStockID(GetId()), wxT("A non-stock menu item with an empty label?"));


@@ -210,7 +211,7 @@ void wxMenuItem::UpdateItemText()
     }
 
     wxAcceleratorEntry *entry = wxAcceleratorEntry::Create( m_text ) ;
     }
 
     wxAcceleratorEntry *entry = wxAcceleratorEntry::Create( m_text ) ;

-    m_peer->SetLabel( text, entry );
+    GetPeer()->SetLabel( text, entry );

     delete entry ;
 }
 
     delete entry ;
 }