KDE Bug Tracking System
Home
Report New Wish or Bug
Query Existing Reports
First
Last
Prev
Next
No search results available
Search page
Bug
138165
:
AND and OR formulas don't support ranges
P
roduct
:
kspread
Co
m
ponent
:
general
Status
:
RESOLVED
Resolution
:
FIXED
Target
:
---
Version
:
unspecified
Pr
i
ority
:
NOR
Severity
:
normal
V
otes
:
0
Description
:
Opened:
2006-11-30 15:34
Last Changed:
2006-12-01 21:33:47
Version: 1.6.1 (using KDE KDE 3.5.5) Installed from: Debian testing/unstable Packages OS: Linux At least the AND+OR formulas should also support ranges/arrays like AND(A1:A5;B1:B5). This isn't only more comfortable but also needed to provide better OpenFormula-compatibility.
Comment
#1
Sebastian Sauer 2006-11-30 18:43:06
Created an attachment (id=18736)
[details]
Patch that implements ranges for AND+OR This patch against the KOffice 1.6 branch implements support for ranges/arrays for the AND and the OR formula functions. Testing doesn't show any regressions. Following cases are now valid; AND(A1) AND(A1,A2) AND(A1:A5) AND(A1:A5;B1:B10) OR(A1) OR(A1,A2) OR(A1:A5) OR(A1:A5;B1:B10) Not committed yet cause it may needed to also extend other logical functions to support ranges too. Would be nice if someone tests OOo and provides a list of formula functions that may need to support ranges as well.
Comment
#2
Sebastian Sauer 2006-11-30 18:45:51
before I forget it; the patch does not include the changed xml-file which also needs to be extended (e.g. the samples should also include such range-definitions and the argument-descriptions need to be changed as well).
Comment
#3
Matthias 2006-11-30 20:50:06
This is a duplicate of
bug 131809
(which i reported).
Comment
#4
Sebastian Sauer 2006-11-30 23:54:23
***
Bug 131809
has been marked as a duplicate of this bug. ***
Comment
#5
Sebastian Sauer 2006-12-01 00:03:56
TODO I'll look at next days; * check+maybe implement it also for e.g. IF, NOT, TRUE, FALSE, NAND, NOR, XOR * we really could need a complete function-review here to check if there are more functions where OOo allows to define ranges too. Propably it may also an idea to have those parts more generic for 2.0 ( maybe related to
http://lists.kde.org/?l=koffice-devel&m=116484095414360&w=2
).
Comment
#6
Tomas Mecir 2006-12-01 11:40:26
30 Nov 2006 23:03:57 -0000, Sebastian Sauer <
mail@dipe.org
>:
> * we really could need a complete function-review here to check if there are more functions where OOo allows to define ranges too. Propably it may also an idea to have those parts more generic for 2.0 ( maybe related to
http://lists.kde.org/?l=koffice-devel&m=116484095414360&w=2
).
OOo or not, we shall go for compatibility with ODF functions - which OOo is going for as well.
Comment
#7
Sebastian Sauer 2006-12-01 15:51:12
> OOo or not, we shall go for compatibility with ODF functions - which OOo is going for as well.
True. I looked at the latest OpenFormula-draft (
http://www.oasis-open.org/committees/download.php/21372/openformula-20061129.odt
) and it does not mention that e.g. the AND()-function supports arrays but it also does not mention that it does not. So, IMHO the spec should be extended to explicit allow arrays for logical functions like AND() cause it just makes sense. The alternate would be to fill a bugreport against OOo but I guess they would mark it as invalid/won't fix cause it may break existing documents that worked before. So, the only alternate I see here is to let KSpread behave like OOo and to extend the specs...
Comment
#8
Sebastian Sauer 2006-12-01 16:57:03
Comparing the logical functions, AND, OR, NAND, NOR and XOR are affected. The specs are only defining AND, OR and XOR ( NAND==NOT(AND()) && NOR=NOT(OR()) ) where OOo does not implement XOR ( filled now as
http://www.openoffice.org/issues/show_bug.cgi?id=72182
).
Comment
#9
Sebastian Sauer 2006-12-01 17:54:28
Created an attachment (id=18744)
[details]
Patch that implements ranges for AND+OR+NAND+NOR+XOR This patch against the KOffice 1.6 branch implements support for ranges/arrays for the AND, OR, NAND, NOR and XOR formula functions. Also it does try to simplify the code at NAND+NOR by reusing the reverse of there AND+NR counterparts. As usual tested and so far no regressions did show up.
Comment
#10
Sebastian Sauer 2006-12-01 21:33:47
The patch got committed now to the 1.6-branch and trunk. Thanks for the feedback, Matthias.
P
latform
:
Debian testing
O
S
:
Linux
K
eywords
:
People
Reporter
:
Sebastian Sauer
Assigned To
:
Sebastian Sauer
CC
:
mail dipe org
matthias_berndt gmx de
Related actions
View Bug Activity
Format For Printing
XML
Clone This Bug
Note
You need to
log in
before you can comment on or make changes to this bug.
Attachments
Patch that implements ranges for AND+OR
(2.02 KB, patch)
2006-11-30 18:43
,
Sebastian Sauer
Details
Patch that implements ranges for AND+OR+NAND+NOR+XOR
(3.74 KB, patch)
2006-12-01 17:54
,
Sebastian Sauer
Details
Hide Obsolete
(1) |
View All
Add an attachment
(proposed patch, testcase, etc.)
Depends on
:
B
locks
:
Show dependency tree
-
Show dependency graph
First
Last
Prev
Next
No search results available
Search page
Actions
Reports
Requests
Reports
Bugs reported today
Bugs reported in the last 3 days
Bug reports with patches
Weekly Bug statistics
The most hated bugs
The most severe bugs
The most frequently reported bugs
The most wanted features
Junior Jobs
Report ownership counts and charts
My Account
New Account
Log In