if not working correctly

[axy108]

Hi All,

I am trying to automate some websites but I am facing an issue trying to validate if the login screen is been shown or not.

Local $userName = 'user'
Local $password = 'password'
Local $oIE = _IECreate()
WinSetState("Blank Page", "", @SW_MAXIMIZE)
_IENavigate($oIE, "http://url.com")
_IELoadWait($oIE)
_IEGetObjByName($oIE, "ssousername")
AutoItSetOption("SendKeyDelay", 50)
If @error = 0 Then
enterUserNameAndPassword($userName, $password)
EndIf
Local $oNew = _IEGetObjById($oIE, "New")
_IEAction($oNew, "click")
_IELoadWait($oIE)


func enterUserNameAndPassword( $userName, $password)
Local $oUserId = _IEGetObjByName($oIE, "ssousername")
AutoItSetOption("SendKeyDelay", 50)
_IEAction($oUserId, "focus")
Send($userName)
Send("{TAB}")
Send($password)
Local $oLogin = _IEGetObjById($oIE, "loginButton")
_IEAction($oLogin, "click")
_IELoadWait($oIE)
EndFunc

I just wan to execute the enterUserNameAndPassword function if the get username object was successed and if not I want to continue with the next steps. But when I run the script the lines Send("{TAB}") and Send($password) are being executed and I do not know why. Does anybody know what is going on?

Really appreciate your help!

Thanks,

axy108.

[BrewManNH]

You're checking for an error here:

AutoItSetOption("SendKeyDelay", 50)
If @error = 0 Then

Which only checks to see if AutoItSetOption was able to set the option correctly. In this case it's going to work, so @error will equal 0, so it is going to run the function.

You need to check @error directly after the function that you want to check the errors from, because it is reset after every function, so it's not going to work correctly the way you have it.

[jdelaney]

All that work to use the _IE* functions, and then you don't use them to populate the fields?

This function will save a lot of headaches, rather than the 'send' commands which may be consumed by other apps: _IEFormElementSetValue

Edited January 14, 2013 by jdelaney

[axy108]

Thanks BrewManNH I didn't know that the error check worked that way I have been working with AutoIt only for a couple of weeks now. The script is working now .

And thanks jdelaney, I also didn't know about the function you mentioned but I looked for it in the help and now I am using it and you are right it's a lot much easier.

Thanks again to both of you.