| Summary: | Errors in functions/*xml description files | ||
|---|---|---|---|
| Product: | [Applications] calligrasheets | Reporter: | Burkhard Lück <lueck> |
| Component: | functions | Assignee: | Calligra Sheets (KSpread) Bugs <calligra-sheets-bugs-null> |
| Status: | CONFIRMED --- | ||
| Severity: | normal | CC: | inge, sebsauer |
| Priority: | NOR | ||
| Version: | Git | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: |
List of all errors found generating functions.docbook
List of types not handled by FunctionDescription.cpp |
||
Git commit 9c4c3bef739950056898bd98cf375be8d8774f0d by Burkhard Lück. Committed on 18/01/2012 at 22:34. Pushed by lueck into branch 'master'. fix Fix wrong function type descriptions in tables/functions/*xml files changed all int/IntType/Integer to the proper Type string Int + Bool to Boolean REVIEW:103710 M +2 -2 tables/functions/engineering.xml M +11 -13 tables/functions/financial.xml M +3 -2 tables/functions/math.xml M +2 -4 tables/functions/reference.xml M +3 -2 tables/functions/statistical.xml M +2 -1 tables/functions/text.xml http://commits.kde.org/calligra/9c4c3bef739950056898bd98cf375be8d8774f0d I assume this bug can be closed now? Created attachment 68157 [details]
List of types not handled by FunctionDescription.cpp
Due to the code in FunctionDescription.cpp the types 'Boolean', 'Int', 'String', 'Any' have a correct description on the Help tab in the functions dialog. Any other function type in the xml files has the type description "A floating point value..." in the functions dialog but that is in some cases (e.g. second parameter of CELL) wrong. Attached as comment #3 a list of functions with types != 'Boolean', 'Int', 'String', 'Any', 'Float', 'FLOAT', 'Double', which are all displayed as Float. From my pov these issues need to be addressed before closing this bug: 1) Use only <Type>Float</Type> for floating point values, not also 'FLOAT' and 'Double'. 2) Handle type 'Float' explicitely in FunctionDescription.cpp, that prevents silent failures like reported here. 3) Either use the raw type string from the xml files to display it in the funktions dialog or add all missing categories (Reference, Array, String/Numeric, Reference/Array, Error, Date) with more verbose typ descriptions like for the already handled types. Git commit a50925079090201cbb0f4f1f9819e1ad6ff59edd by Burkhard Lück. Committed on 24/05/2012 at 14:36. Pushed by lueck into branch 'master'. Fix wrong function type description for 'Date' REVIEW:104954 M +6 -0 sheets/FunctionDescription.cpp M +1 -1 sheets/FunctionDescription.h http://commits.kde.org/calligra/a50925079090201cbb0f4f1f9819e1ad6ff59edd Status update of this bug report: The following list shows all of types not handled by FunctionDescription.cpp and therefore have the type description "A floating point value..." in the functions dialog, which is obviusly wrong: ./information.xml:27: <Type>Reference</Type> ./information.xml:223: <Type>Reference</Type> ./information.xml:308: <Type>Error</Type> /reference.xml:212: <Type>Reference</Type> ./reference.xml:227: <Type>Reference</Type> ./reference.xml:239: <Type>String/Numeric</Type> ./reference.xml:243: <Type>String/Numeric</Type> ./reference.xml:247: <Type>String/Numeric</Type> ./reference.xml:258: <Type>String/Numeric</Type> ./reference.xml:261: <Type>String/Numeric</Type> ./reference.xml:265: <Type>Array</Type> ./reference.xml:284: <Type>String/Numeric</Type> ./reference.xml:287: <Type>String/Numeric</Type> ./reference.xml:291: <Type>Array</Type> ./reference.xml:310: <Type>String/Numeric</Type> ./reference.xml:313: <Type>Reference</Type> ./reference.xml:317: <Type>Reference</Type> ./reference.xml:321: <Type>Reference</Type> ./reference.xml:325: <Type>Reference</Type> ./reference.xml:329: <Type>Reference</Type> ./reference.xml:342: <Type>String/Numeric</Type> ./reference.xml:346: <Type>Reference/Array</Type> ./reference.xml:360: <Type>Reference</Type> ./reference.xml:363: <Type>Reference</Type> Thank you for the bug report. As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists. If this bug is no longer persisting or relevant please change the status to resolved. |
Created attachment 67459 [details] List of all errors found generating functions.docbook FunctionDescription.cpp has "Boolean", "Int", "String", "Any", everything else is treated as Float -> that is wrong. There is also Reference, Array, Bool, String/Numeric, Reference/Array, Error, FLOAT, int, IntType, Integer, Date, Double -> all teated as Float More errors: Summary of Errors in 1090 Functions: error_missing_parameter: 1 error_missing_syntax: 1 error_missing_example: 37 error_missing_related: 209 error_missing_helptxt: 1 error_args_syntax_comment: 85 error_optional_syntax_comments: 16 error_type_description_not_supported: 163 dead links: COUPDAYBS PRICEDISC PRICE ODDFPRICE