Cannot Change Menu Item Checked State -

[orange]

I have this code to generate a menu item:

$menuControlID = GUICtrlCreateMenuItem("Example Menu", $SettingsMenu)
GUICtrlSetOnEvent(-1, "ExampleMenuFunction")
GUICtrlSetState(-1, $GUI_CHECKED)oÝ÷ Øz0z÷«¶ç$yÛ-j×8D97!jx"ج½êò~»¬¶¶­xáÞâë-j׬iº.¶g§º+^Ê'+ajÚ®¢×µ«^¡öªºfz{¨­ª®¢Ú®¢Ùè­zj®¢×(Úè·z-mèbuקý¿êº^J¥az»-jׯj¸nW¬j·¡Ë¦z{^vÚ0¢¹«Þ¶¸vë"¨Úèg­©Ýæ¬Áéejëh×6Func ExampleMenuFunction()
$TempHandle = ControlGetHandle("","",$menuControlID)
 if Guictrlgetstate($TempHandle) = $GUI_CHECKED then guictrlsetstate($TempHandle,$GUI_UNCHECKED)
 if Guictrlgetstate($TempHandle) = $GUI_UNCHECKED then guictrlsetstate($TempHandle,$GUI_CHECKED)
EndFuncoÝ÷ Ø­¶­v¬mÂäz+az²!j÷­®'z÷«ÊØbajÒ$!£§tz-ëm+,µ«^¶!jx"¯zÚâyÖÆP­®Tµ+rïÛlerÚåIëRµ«^
í®FzÑÚÙ^ý³h¶§ÇËhÂä¶!jx¶¬²Ö­xZ½ëj·!yÛayú+ºkÒjyèµø§u©ò¶§·¥jاÚ-+²Ø^­æ§ÊÞ¶­q©ÚÙ{aÂ-ÊÞjYr~»¬¶¶­xØ­±ç¦±«- Ø^­ë!¢é]mæ¬jezÊ%ºØ¨8ZKÀÊ¡ë'¢Ü(®F®¶­sbb33cµFV×ôæFÆRÒuT7G&ÄvWDæFÆRb33c¶ÖVçT6öçG&öÄB¢wV7G&Ç6WG7FFRb33cµFV×ôæFÆRÂb33c´uTõTä4T4´TB

Edited July 27, 2007 by orange

[GEOSoft]

Try changing this and see what happens

Func ExampleMenuFunction()
 if Guictrlgetstate($menuControlID ) = $GUI_CHECKED then guictrlsetstate($menuControlID ,$GUI_UNCHECKED)
 if Guictrlgetstate($menuControlID ) = $GUI_UNCHECKED then guictrlsetstate($menuControlID ,$GUI_CHECKED)
EndFunc

To

Func ExampleMenuFunction()
 MsgBox(0,"TEST",Guictrlgetstate($menuControlID ))
 if Guictrlgetstate($menuControlID ) = 65 then guictrlsetstate($menuControlID ,68)
 if Guictrlgetstate($menuControlID ) = 68 then guictrlsetstate($menuControlID ,65)
 MsgBox(0,"TEST",Guictrlgetstate($menuControlID ))
EndFunc

Just to see what happens.

Edited July 27, 2007 by GEOSoft

[orange]

Return:

Msgbox 1: 0

Msgbox 2: 0

;************************************

EDIT:

Also does not work:

if bitand(Guictrlgetstate($menuControlID),$GUI_CHECKED) = 1 then msgbox (0,0,"Checked")
    if bitand(Guictrlgetstate($menuControlID),$GUI_UNCHECKED) = 1 then msgbox (0,0,"Checked")oÝ÷ Ù8b±«­¢+Ù¥¹Éä¡Õ¥ÑɱÑÍÑÑ ÀÌØíµ¹Õ
½¹Ñɽ±%¤¤

returns 0x000000.

Shouldn't there be SOMETHING initalized? $GUI_Checked = 1, and $GUI_Unchecked = 4, so shouldn't "binary(Guictrlgetstate($menuControlID))" return something like: 0x000001 or 0x000100

Edited July 27, 2007 by orange

[GEOSoft]

Return:

Msgbox 1: 0

Msgbox 2: 0

;************************************

EDIT:

Also does not work:

if bitand(Guictrlgetstate($menuControlID),$GUI_CHECKED) = 1 then msgbox (0,0,"Checked")
    if bitand(Guictrlgetstate($menuControlID),$GUI_UNCHECKED) = 1 then msgbox (0,0,"Checked")oÝ÷ Ù8b±«­¢+Ù¥¹Éä¡Õ¥ÑɱÑÍÑÑ ÀÌØíµ¹Õ
½¹Ñɽ±%¤¤

returns 0x000000.

Shouldn't there be SOMETHING initalized? $GUI_Checked = 1, and $GUI_Unchecked = 4, so shouldn't "binary(Guictrlgetstate($menuControlID))" return something like: 0x000001 or 0x000100

I just remembered, for menu items it's not GUICtrlGetState(). Use GUICtrlRead() instead.

[orange]

When I try this:

MsgBox(0,"TEST",binary(guictrlread($menuControlID )))
    guisetstate($menuControlID ,68)
    MsgBox(0,"TEST",binary(guictrlread($menuControlID )))

I get two return values of 0x4100000

However, if I manually change the declaration of the Menu item to Unchecked, I get this:

0x4400000

The difference is important; using GUICTRLREAD, I can find out the checked state, however, as noted above, GUICtrlSetState does not set the state of a menu control.

Edit: Yup, only the name of the control.

Anybody got any more ideas?

I am now trying GuiCtrlSetData, but I doubt that that is going to change anything but the name of the menu.

Edited July 27, 2007 by orange

[GEOSoft]

When I try this:

MsgBox(0,"TEST",binary(guictrlread($menuControlID )))
    guisetstate($menuControlID ,68)
    MsgBox(0,"TEST",binary(guictrlread($menuControlID )))

I get two return values of 0x4100000

However, if I manually change the declaration of the Menu item to Unchecked, I get this:

0x4400000

The difference is important; using GUICTRLREAD, I can find out the checked state, however, as noted above, GUICtrlSetState does not set the state of a menu control.

Edit: Yup, only the name of the control.

Anybody got any more ideas?

I am now trying GuiCtrlSetData, but I doubt that that is going to change anything but the name of the menu.

I just looked at a piece of my working code and I'm definitly using GUICtrlSetState() and I'm using the numeric values instead of those damned constants.

[orange]

SOLVED:

Here is the finalized version of working code, I am not sure why this took so long, but I hope that this is useful to someone who needs it:

if guictrlread($menuControlID) = 65 then 
        guictrlsetstate($menuControlID ,68)
    elseif guictrlread($menuControlID) = 68 
         guictrlsetstate($menuControlID ,65)
    EndIf

Thanks again to GEOSoft who saved my day. I appreciate it.

[GEOSoft]

I'm not using GUICtrlSetOnEvent(). I just have this code in the idle Loop

expandcollapse popup

Case $Menu_Opts_Min To $Menu_Opts_Animate
            If $Msg = $Menu_Opts_Run_Locate Then
                NotAvail()
                Return
            EndIf
            If GUICtrlRead($Msg) = 65 Then
                GUICtrlSetState($Msg, 68)
                If $Msg = $Menu_Opts_Min Then
                    IniWrite($Ini,'Program Settings','MinimizeOnUse','4')
                    $Min = 4
                ElseIf $Msg = $Menu_Opts_Backups Then
                    IniWrite($Ini,'Program Settings','Save PreEdit','4')
                    $E_Back = 4
                ElseIf $Msg = $Menu_Opts_Cust Then
                    IniWrite($Ini,'Program Settings','CustFold','4')
                    $Cust_Fldrs = 4
                ElseIf $Msg = $Menu_Opts_Templ Then
                    IniWrite($Ini,'Program Settings','UseShell','4')
                    $Use_Templates = 4
                ElseIf $Msg = $Menu_Opts_Animate Then
                    IniWrite($Ini, 'Program Settings','Animate','4')
                    $Animate = 4
                EndIf
            Else
                GUICtrlSetState($Msg, 65)
                If $Msg = $Menu_Opts_Min Then
                    IniWrite($Ini,'Program Settings','MinimizeOnUse','1')
                    $Min = 1
                ElseIf $Msg = $Menu_Opts_Backups Then
                    IniWrite($Ini,'Program Settings','Save PreEdit','1')
                    $E_Back = 1
                ElseIf $Msg = $Menu_Opts_Cust Then
                    IniWrite($Ini,'Program Settings','CustFold','1')
                    $Cust_Fldrs = 1
                ElseIf $Msg = $Menu_Opts_Templ Then
                    IniWrite($Ini,'Program Settings','UseShell','1')
                    $Use_Templates = 1
                ElseIf $Msg = $Menu_Opts_Animate Then
                    IniWrite($Ini, 'Program Settings','Animate','1')
                    $Animate = 1
                EndIf
            EndIf

And that is pasted directly from the working code

EDIT: Just forget about the INI lines and string variables like $Animate because they don't apply to your code. This was just an example of how I do it.

Edited July 27, 2007 by GEOSoft

[MrCreatoR]

This should be mentioned in the help file (about state of menu items - 65 and 68) - i am also a while ago had the same problem, and it took me couple of hours to figure it out (by checking every possibility with Gui... tools - just like you guys did).

[GEOSoft]

This should be mentioned in the help file (about state of menu items - 65 and 68) - i am also a while ago had the same problem, and it took me couple of hours to figure it out (by checking every possibility with Gui... tools - just like you guys did).

It was worse at one time. AutoIt code was broken and instead of 68 you had to use 69. It was only when I asked a question in Developer Chat that someone noticed it. But actually for menu items it does make sense because they can't be set to hidden.

Therefore you have

Enabled = 64

Disabled = 128

Checked = 1

Unchecked = 4

Therefore

Enabled + checked = 65

Disabled + unchecked = 132