Boolean Function to constrain another function

[Blue_Wings]

Hi there,
If possible i would like some help in the following matter:

I have a function with a button that send´s values from an excel cell in one sheet to another sheet, and i would like to have another function that checks if another 2 cells are empty and then pops up a msgbox giving a warning about the empty cells.
If the 2 cells are empty the warning is given and the button will not send anything.

At the moment i have something like this: (but the boolean seems not to work, something is wrong, because even with values on the cells CD11 and CE12 on sheet3 the button dont send anything).

Please Login or Register  to view this content.

Thanks.
Have a nice holidays everyone.

[sullivan372]

Might be a silly question but did you dim ARRIVAL anywhere? It looks like you are using it as a global, but you didn't include your declarations.

[Blue_Wings]

Yes i forgot to mention that, it was declared in anoter function, thanks for reminding me, now i have it on the code but it doesnt work anyway.
I Even writted a msgbox to test if it gets there but even the MsgBox "after boolean is true" dont appear, only the MsgBox "before boolean is true" pops up.

It means that the condicion "If ARRIVAL = True Then..." is not true... but it should be if the two cells CD11 and CE12 have values.

I tried to put even a code more simple like this, but it doesnt work (forcing the boolean ARRIVAL to be true, just to check if accepts the if condition from the other function):

Please Login or Register  to view this content.

[sullivan372]

So from what I can see of your code, the issue is what I was hinting at before
Unless you have this line:

Please Login or Register  to view this content.

then your boolean is only local to the specific function. So it is actually two different instances of a boolean variable called ARRIVAL, one in each of your subs.
If my understanding is correct, then there are a couple ways to fix it. You can put in the above line of code, or similarly:

Please Login or Register  to view this content.

-Either of these two lines would work, but they need to be placed outside your sub
With either of these lines, your Check_ARRIVAL_Code function will set the ARRIVAL to true or false, and then the btn sub would check the value of that variable. If you want to implement this, make sure you get rid of the dim ARRIVAL as Boolean line because this will create a local instance and ignore the global.

The alternate fix, and maybe the better one depending on the rest of your code, is to have your function return the true or false value, rather than setting a variable. Your code would then look like this:

Please Login or Register  to view this content.

Let me know if this helps.

[sullivan372]

Also note that I changed your first sub to public, because I don't think you can call a private macro from a button. The second sub is a function to allow for a return value.

[Blue_Wings]

Thanks, it helped, but i had the vba code (Global ARRIVAL As Boolean) in the sheet, and to work needs to be in the Module, i didnt know that.

Thanks once again.