Is there a known problem with @askuser() or am I using this incorrectly?

I have a command button that when pressed is used to make the ship to data the same as home address data. I wanted to protect against overwriting shipping data by accident so I added the feature of checking if ship to elements are blank. If data is found  it asks if you want to replace it.


var vshipblank as string

vshipblank = shipstreet + Shipcity + shipstate + shipzip
// writeln(vshipblank)


If vshipblank <> ""
then
     if @askuser("There is existing data in the ship to fields", "do you want to replace","existing ship to data with home address")



ShipStreet = homestreet
shipcity = homecity
shipstate = homestate
shipzip = homezip

The problem is that if all the ship to elements are empty it does not execute the first task of  ShipStreet = homestreet, but it does all the rest.

If ship to elements has data and you choose to replace it all data is correctly transferred.

So it is either my code or ???

Bob,

The reason why your code is behaving that way is because you don't have curly braces {} around the lines following the If @Askuser. Because of this, it doesn't know that all four lines are part of the conditional, and it's only including the first line it encounters as depending on the condition.

Quote:

Your original code is the equivalent of this:

If vshipblank <> "" Then
{
    If @askuser("There is existing data in the ship to fields", "do you want to replace","existing ship to data with home address")
    {
		    ShipStreet = homestreet
    }
}
// The following lines will always run because they
// are not enclosed by any condition.
shipcity = homecity
shipstate = homestate
shipzip = homezip
 

Does that make it clearer? 

Quote:


It's too bad you're way out in California. Do you ever get out this way (Ohio or East Coast)?