Review Request: Fix wrong function type descriptions in tables/functions/*xml files
Sebastian Sauer
mail at dipe.org
Wed Jan 18 05:40:40 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103710/#review9909
-----------------------------------------------------------
Ship it!
Ship It!
- Sebastian Sauer
On Jan. 16, 2012, 7:23 p.m., Burkhard Lück wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103710/
> -----------------------------------------------------------
>
> (Updated Jan. 16, 2012, 7:23 p.m.)
>
>
> Review request for Calligra, Marijn Kruisselbrink and Sebastian Sauer.
>
>
> Description
> -------
>
> FunctionDescription.cpp handles Types "Boolean", "Int", "String", "Any", everything else
> is treated as Float type-> that is wrong.
>
> There is also Reference, Array, Bool, String/Numeric, Reference/Array, Error,
> FLOAT, int, IntType, Integer, Date, Double -> all treated as Float and displayed with
> that type in functions dialog.
>
> 1) changed all int/IntType/Integer to the proper Type string "Int" + Bool to "Boolean"
>
> 2) removed sebsauers comments, this doesn't break string freeze, the msgs are already in tables.pot
>
> 2) added some comments about issues to fixe for 2.5, breaks strings freeze now.
>
> Not all issues mentioned in #290631 are fixed in this patch,
> the types Reference, Array, String/Numeric, Reference/Array, Error, Date, Double are still wrong.
>
>
> This addresses bug 290631.
> http://bugs.kde.org/show_bug.cgi?id=290631
>
>
> Diffs
> -----
>
> tables/functions/engineering.xml a9c5281
> tables/functions/financial.xml e54b400
> tables/functions/math.xml b495861
> tables/functions/reference.xml 2446021
> tables/functions/statistical.xml 1463231
> tables/functions/text.xml 6933622
>
> Diff: http://git.reviewboard.kde.org/r/103710/diff/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Burkhard Lück
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120118/c68aca43/attachment.htm>
More information about the calligra-devel
mailing list