If.. Else.. statement don't work.

[tristan208]

Right of the top, I'm totally n00b at this.

This is a script to maximize Mailwasher if it is in the tray, or else close it to tray. It closes it perfectly, but I can't get it to open up.

If I make a script with just the Else statement, it works like a charm to, but together they don't work.

Anyway, here is my simple code:

If WinExists("MailWasher") Then
    WinClose("MailWasher")
Else 
    Send( "!+m" )
EndIf

Any good ideas on how to make this work properly?

(Alt+Shit+m is a shortcut in MailWasher to maximize it.)

Edited February 22, 2008 by tristan208

[comradeb14ck]

The WinClose() function does not maximize a window. It actually closes it (terminating the program in most cases). What you want is

WinSetState("MailWasher", "", @SW_MAXIMIZE)

That will maximize the window if that program is open.

[Aassdd]

The above code is used to check if the program exists in the tray? If yes, it's not working because WinExists check for programs in the taskbar.

[tristan208]

The WinClose() function does not maximize a window. It actually closes it (terminating the program in most cases). What you want is

WinSetState("MailWasher", "", @SW_MAXIMIZE)

That will maximize the window if that program is open.

I think you misunderstood me. It's usually minimized to tray, and I want it to maximize from tray, but only if it isn't maximized. If it is maximized I want it to close. Split up these two statements work a charm, but combined in an If.. Else.. I can't get it to work.

[GEOSoft]

Right of the top, I'm totally n00b at this.

This is a script to maximize Mailwasher if it is in the tray, or else close it to tray. It closes it perfectly, but I can't get it to open up.

If I make a script with just the Else statement, it works like a charm to, but together they don't work.

Anyway, here is my simple code:

If WinExists("MailWasher") Then
    WinClose("MailWasher")
Else 
    Send( "!+m" )
EndIf

Any good ideas on how to make this work properly?

(Alt+Shit+m is a shortcut in MailWasher to maximize it.)

Because if the window exists then you are closing it.

All you should need is

If WinExists("MailWasher") AND WinGetState("MailWasher") < 32 Then WinSetState("MailWasher", "", @SW_Maximize)

Also make sure that you have looked at Opt ("WinTitleMatchMode") and the special information about Win Titles in the help file.

[Aassdd]

George, what is "< 32", I don't understand it.

[GEOSoft]

George, what is "< 32", I don't understand it.

It means if the state of the window is less than (<) 32.

From the help file for WinGetState()

32 = Window is maximized

So what we are saying here is if the Window is not Maximized then Maximize it.

[tristan208]

I'm obviously not very good at explaining my needs, because it seems like you all misunderstood me.

MailWasher always resides in the tray, and when new mail arrives it starts blinking. What I want is the possibility

to open it up from the tray, which can easily be done by pressing shift+alt+m. But if the Mailwasher window is already

open, I want the script to close it back to tray.

The reason for this, is that I have a 'Mail' button on my mouse, but MailWasher isn't supported by the mouse software,

so I wanted to write my own little thing to get the job done.

And now I'm confused.

The first part of the statement checks if MailWasher exists, and if it does, it closes it to tray. And that works.

But if MailWasher doesn't exist, shouldn't it run the Else statement?

Edited February 22, 2008 by tristan208

[GEOSoft]

I'm obviously not very good at explaining my needs, because it seems like you all misunderstood me.

MailWasher always resides in the tray, and when new mail arrives it starts blinking. What I want is the possibility

to open it up from the tray, which can easily be done by pressing shift+alt+m. But if the Mailwasher window is already

open, I want the script to close it back to tray.

The reason for this, is that I have a 'Mail' button on my mouse, but MailWasher isn't supported by the mouse software,

so I wanted to write my own little thing to get the job done.

Try this but I'm just going off the top of my head here and it's already been pointed out that my memory is not great.

If WinExists("MailWasher") Then
   If NOT WinGetState("MailWasher") >= 32 Then
      WinSetState("MailWasher", "", @SW_Maximize);; You could use your send instead if you really insist.
   Else
      WinClose("MailWasher");;  Watch out that this doesn't exit MailWasher entirely.  If it does, I have a func to fix it.
   EndIf
EndIf

[tristan208]

Try this but I'm just going off the top of my head here and it's already been pointed out that my memory is not great.

If WinExists("MailWasher") Then
   If NOT WinGetState("MailWasher") >= 32 Then
      WinSetState("MailWasher", "", @SW_Maximize);; You could use your send instead if you really insist.
   Else
      WinClose("MailWasher");;  Watch out that this doesn't exit MailWasher entirely.  If it does, I have a func to fix it.
   EndIf
EndIf

Just tried your code, it didn't work.

I can't get it to open up from tray, unless I put the Send at the very beginning.

The WinSetState doesn't work when MailWasher is in the tray, that's why I want the Send to send the shortcut to MailWasher.

And I don't want it to close entirely, just put it back to the tray. (This part works flawlesly.)

Edited February 22, 2008 by tristan208

[tristan208]

Finally got it.

$state = WinGetState("MailWasher", "")

If Not BitAnd($state, 2) Then
    Send("+!m")
EndIf

If BitAnd($state, 2) Then
    WinClose("MailWasher")
EndIf

[FreeFry]

My guess is that when the window is closed, it just hides itself, it doesn't destroy the window, therefore your first if statement is always true.

Edit:

Therefore your original code should be:

If WinExists("MailWasher") And BitAND(WinGetState("MailWasher"), 2) Then
    WinClose("MailWasher")
Else
    Send( "!+m" )
EndIf

Edited February 22, 2008 by FreeFry