146818 2007-06-15 12:16:25 +0000 Error when multiplying, dividing, subtracting or adding two Floating Point fields 2008-01-10 17:42:59 +0000 1 1 2 Applications KEXI Queries 1.1.3 (KOffice 1.6.3) Ubuntu Linux CLOSED FIXED NOR normal --- 1 r.c.drew staniek 0 oldest_to_newest 531649 0 r.c.drew 2007-06-15 12:16:25 +0000 Version: 1.1.3 (using KDE KDE 3.5.6) Installed from: Ubuntu Packages OS: Linux In the following: FP = Floating Point, Int = Integer If you set up a query to multiply FP x Int, result is FP. This works fine. However, the result for FP x Int gives a square with a question mark inside. Whether the FP number is single or double precision, the same problem occurs. This has been confirmed by Jarosław Staniek (current project maintainer) who asked me to file the bug. The same problem exists when adding, subtracting or dividing two FP numbers. Clearly, Kexi has great potential, and much great work has been done, but it is unusable as a database if any FP numbers are involved! 531650 1 r.c.drew 2007-06-15 12:28:38 +0000 Sorry, I made a typo: I should have written "However, the result for FP x FP gives a square with a question mark inside". 569832 2 staniek 2008-01-10 17:41:01 +0000 SVN commit 759457 by staniek: KexiDB fix the bug #146818: handling expressions containing binary operators like *, /, -, + where both arguments are of floating-point type (with single or double precision). Previously the result was marked as boolean, now it is floating-point. CCMAIL:146818-done@bugs.kde.org CCMAIL:kexi@kde.org M +2 -6 expression.cpp --- branches/koffice/1.6/koffice/kexi/kexidb/expression.cpp #759456:759457 @@ -400,14 +400,10 @@ if (ltInt && rtInt) return KexiDB::maximumForIntegerTypes(lt, rt); - if (Field::isFPNumericType(lt) && rtInt) + if (Field::isFPNumericType(lt) && (rtInt || lt==rt)) return lt; - if (Field::isFPNumericType(rt) && ltInt) + if (Field::isFPNumericType(rt) && (ltInt || lt==rt)) return rt; - if ((lt==Field::Double || lt==Field::Float) && rtInt) - return lt; - if ((rt==Field::Double || rt==Field::Float) && ltInt) - return rt; return Field::Boolean; }