Passing Input Variables

[tweaker]

I'm having a bit of difficulty with a gui login procedure for my app and I don't know why it isn't working.

Opt('MustDeclareVars', 1)

Login()

Func Login()
    Local $username, $password, $ip, $btn, $msg

    GUICreate(" Login", 320, 120, @DesktopWidth / 2 - 160, @DesktopHeight / 2 - 145, -1, 0x00000018); WS_EX_ACCEPTFILES
    $username = GUICtrlCreateInput("Username", 10, 5, 300, 20)
    $password = GUICtrlCreateInput("Password", 10, 35, 300, 20)
    $ip = GUICtrlCreateInput("IP",10,65,300,20)
    $btn = GUICtrlCreateButton("Ok", 40, 95, 60, 20)

    GUISetState()

    $msg = 0
    While $msg <> $GUI_EVENT_CLOSE
        $msg = GUIGetMsg()
        Select
            Case $msg = $btn
                ExitLoop
        EndSelect
    WEnd    
EndFunc

; Open database connection
DataInit()
OnAutoItExitRegister("DataExit")
if not DataConnect(1,"" & $ip & "","" & $username & "","" & $password & "","MtgoData",0,14430) then Exit(MsgBox(16,"No database connection",DataError()))

If I remove the gui login process and put the 3 values straight into the code it works perfectly.

if not DataConnect(1,"192.168.100.52","useraccess","pwdabc123","db",0,14430) then Exit(MsgBox(16,"No database connection",DataError()))

The error I'm getting:

quote_collection.au3 (44) : ==> Variable used without being declared.:

Does anyone have any suggestions on how to make it work properly?

Edited May 8, 2011 by tweaker

[GEOSoft]

Not sure why that is the error message. It probably has something to do with some code you didn't show.

There is however a potential problem in that you have declared $username, $password and $ip as local only which means they will not be valid outside the function login()

[AdmiralAlkex]

Also $username, $password and $ip is just ControlId's to the respective controls. If you want the text they contains use GUICtrlRead()

[pieeater]

if it helps you can use the login udf i made as a reference, however i used my login with ini files, not databases :/

links in my signiture

Edited May 8, 2011 by pieeater

[tweaker]

Also $username, $password and $ip is just ControlId's to the respective controls. If you want the text they contains use GUICtrlRead()

OK I changed the Local to Global as suggested above and added the GUICrtlRead() and so far its working great. Thanks for the suggestions!!

[GEOSoft]

Glad to hear it worked out.