game crashes when reading DATETIME NULL values

[Fusselwurm]

[Fusselwurm]

my guess: null pointer exception at

intercept-database/src/res.cpp

Line 45 in 1194bf5

case mariadb::value::date_time: row.emplace_back(res->get_date_time(i).str()); break;

intercept-database/src/res.cpp

Line 82 in 1194bf5

case mariadb::value::date_time: row.emplace_back(res->get_date_time(i).str()); break;

(trying to call str() on a null value)

[dedmen]

What should resultArray contain? nil/"NULL" ?
I guess "NULL" is most user friendly

[Fusselwurm]

I see that nil is really ugly as it can result in code being ignored instead of erroring out.
But "NULL" as a string is equally wrong - going from a distinct non-value and making a magic string of it while valid values are also strings is... ugh...

How about objNull? It's not a scalar, but it's a distinct value that doesnt have the drawbacks of nil

[dedmen]

Actually the database returns me a string for datetime. So it gives me "NULL" so forwarding that out seems to make most sense.

[Fusselwurm]

o.O huh. ok then.

[Fusselwurm]

in a perfect world, we'd have a date datatype in sqf, but we dont, sooo ok

[dedmen]

I could build one. But it seems a little out of scope.

[dedmen]

._. dbResultToArray docs say (null values from the database will be returned as nil entries in the array)

but if I do that, dbResultToParsedArray will return different things? oof.

[dedmen]

We need to handle null values anyway.
Idea was to create a new dbNull type along with things like objNull and such.

isNull will return true ofc. Also want _value isEqualTo "" to return true if trivially doable because some people want to use empty string for null values.
Though isEqualTo variant is hard to do, and might be confusing to people who actually want to check for empty string.

[dedmen]

new dbNull type was implemented in 1.5
dbNull == "" returns false by default, but can be enabled in config file if needed
I heard extDB returned "" for null values so that should make transition easier.

But isEqualTo to compare with empty string is not supported, Arma engine limitations.