[[ Bug 14541 ]] Fix faulty logic in submenu pick

livecodeian · livecodeian · commit 6cd9f4a9ec43 · 2015-03-26T15:10:19.000Z
diff --git a/engine/src/button.cpp b/engine/src/button.cpp
@@ -3815,7 +3815,7 @@ void MCButton::docascade(MCStringRef p_pick)
 	
 	if (pptr != this)
 	{
-		if (m_menu_handler != nil && !m_menu_handler->OnMenuPick(pptr, *t_pick, nil))
+		if (m_menu_handler == nil || !m_menu_handler->OnMenuPick(pptr, *t_pick, nil))
 		{
 			MCParameter *param = new MCParameter;
 			param->setvalueref_argument(*t_pick);

Original file line number	Diff line number	Diff line change
`@@ -3815,7 +3815,7 @@ void MCButton::docascade(MCStringRef p_pick)`
`3815`	`3815`
`3816`	`3816`	`if (pptr != this)`
`3817`	`3817`	`{`
`3818`		`- if (m_menu_handler != nil && !m_menu_handler->OnMenuPick(pptr, *t_pick, nil))`
	`3818`	`+ if (m_menu_handler == nil \|\| !m_menu_handler->OnMenuPick(pptr, *t_pick, nil))`
`3819`	`3819`	`{`
`3820`	`3820`	`MCParameter *param = new MCParameter;`
`3821`	`3821`	`param->setvalueref_argument(*t_pick);`