If loop killing script

[DLynam]

Hello World,

 

I am automating some software installations, and on a small subset of our machines, the installer we run needs a Microsoft C++ Redistributable. I have set the following code as my if loop:

If WinExists("MagTek HID Secure Card Reader Authenticator API Setup") Then
   WinActivate("MagTek HID Secure Card Reader Authenticator API Setup")
   Sleep(2000)
   MouseClick('primary', 152, 220, 1, 0)
   WinWait("Microsoft Visual C++ 2010 X86 Redistributable Setup")
   MouseClick('primary', 52, 265, 1, 0)
   Sleep(2000)
   MouseClick('primary', 365, 446, 1, 0)
   WinWait("Microsoft Visual C++ 2010 x86 Redistributable Setup", "Installation Is Complete")
   WinActivate("Microsoft Visual C++ 2010 x86 Redistributable Setup", "Installation Is Complete")
   MouseClick('primary', 439, 446, 1, 0)
EndIf

This works if the condition is met, however when the window does not pop up, the code will not run any further. Previous to this, I have a WinWait setup to timeout after 10 seconds to see if the MagTek window shows up, and that works.

 

The whole script runs if I take this if loop out, however that defeats the purpose of having the if loop there.

 

Anybody able to see what I'm missing?

[JockoDundee]

1 hour ago, DLynam said:

Anybody able to see what I'm missing?

The actual loop.

1 hour ago, DLynam said:

The whole script runs if I take this if loop out, however that defeats the purpose of having the if loop there.

“If's” are not loops.

This, OTOH, loops forever:

While True 

   If WinExists("MagTek HID Secure Card Reader Authenticator API Setup") Then
      WinActivate("MagTek HID Secure Card Reader Authenticator API Setup")
      Sleep(2000)
      MouseClick('primary', 152, 220, 1, 0)
      WinWait("Microsoft Visual C++ 2010 X86 Redistributable Setup")
      MouseClick('primary', 52, 265, 1, 0)
      Sleep(2000)
      MouseClick('primary', 365, 446, 1, 0)
      WinWait("Microsoft Visual C++ 2010 x86 Redistributable Setup", "Installation Is Complete")
      WinActivate("Microsoft Visual C++ 2010 x86 Redistributable Setup", "Installation Is Complete")
      MouseClick('primary', 439, 446, 1, 0)
   EndIf
   
   Sleep(1000)
WEnd

Change the While True to whatever condition you want to exit the loop.

[Aelc]

well i would use something like

_WinExistAct("MagTek HID Secure Card Reader Authenticator API Setup")
If @error Then _Error()
MouseClick("primary", 152, 220, 1, 0)
_WinExistAct("Microsoft Visual C++ 2010 X86 Redistributable Setup")
If @error Then _Error()
MouseClick("primary", 52, 265, 1, 0)
Sleep(2000)
MouseClick("primary", 365, 446, 1, 0)
_WinExistAct("Microsoft Visual C++ 2010 X86 Redistributable Setup", "Installation Is Complete")
If @error Then _Error()
MouseClick('primary', 439, 446, 1, 0)


Func _Error()
    MsgBox(48, "error", "timeout reached")
    Exit
EndFunc   ;==>_Error

Func _WinExistAct($title, $text = "", $timeout = 10)
    $timeout = $timeout * 1000
    Local $timer = TimerInit()
    Do
        Sleep(50)
        Local $diff = TimerDiff($timer)
    Until WinExists($title, $text) Or $diff >= $timeout
    If $timer > $timeout Then Return SetError(-1, 0, -1)
    If Not WinActive($title, $text) Then WinActivate($title, $text)
    Return 0
EndFunc   ;==>_WinExistAct

and also i would use ControlClick and ControlGetText in a do until loop to check if controls exist when it's same title of the window, instead of using mouseclick and sleep - to be fast as possible...

like this you are always waiting 2 seconds. if the installer finished or not. maybe some people have slower or faster PCs and then this doesn't work anymore or it takes more time as necessary

Edited December 18, 2020 by Aelc

[JockoDundee]

7 hours ago, Aelc said:

well i would use something like

Fwiw, the second window

WinWait("Microsoft Visual C++ 2010 X86 Redistributable Setup")

does not get activated before the click is sent, in the OPs version.

Maybe it doesn’t matter, maybe it does.

Edited December 19, 2020 by JockoDundee

[DLynam]

On 12/18/2020 at 11:55 AM, JockoDundee said:

The actual loop.

“If's” are not loops.

This, OTOH, loops forever:

While True 

   If WinExists("MagTek HID Secure Card Reader Authenticator API Setup") Then
      WinActivate("MagTek HID Secure Card Reader Authenticator API Setup")
      Sleep(2000)
      MouseClick('primary', 152, 220, 1, 0)
      WinWait("Microsoft Visual C++ 2010 X86 Redistributable Setup")
      MouseClick('primary', 52, 265, 1, 0)
      Sleep(2000)
      MouseClick('primary', 365, 446, 1, 0)
      WinWait("Microsoft Visual C++ 2010 x86 Redistributable Setup", "Installation Is Complete")
      WinActivate("Microsoft Visual C++ 2010 x86 Redistributable Setup", "Installation Is Complete")
      MouseClick('primary', 439, 446, 1, 0)
   EndIf
   
   Sleep(1000)
WEnd

Change the While True to whatever condition you want to exit the loop.

Hey, thanks! This worked. I'm new to autoit (coming from a python background where if is a loop by itself) and didn't realize that's how this worked. 

 

Cheers!

[water]

Doesn't the software you install provide a silent installation option? This way you would not need to automate the GUI.

[DLynam]

Just now, water said:

Doesn't the software you install provide a silent installation option? This way you would not need to automate the GUI.

Only one does, we are dealing with a series of about a dozen or so installers, and only one is silent. I don't have the skill or time to create silent installers for all of them. If you have ideas, feel free to suggest, but I've been told to work with what I've got. `\('-')/`

[water]

Most of the installers when called with one of the following parameters ("/?", "-?", "/help" or "-help") will provide a list of available command line switches. One of them might allow a silent installation.

Which applications do you need to install?