First of all, for simple SQL queries, DBCommand() is a better choice. DBConnOpen and cohorts are designed for instances where you'd need to execute multiple SQL commmands and you want t keep the connection open during that time.
You will also, as already stated, have to put the values inside quotes in order to have properly formatted SQL statements.
Finally, with the advent of KiXtart 4.50 is is a bad coding habit to have macros inside strings 
_________________________
There are two types of vessels, submarines and targets.
