https://bugs.documentfoundation.org/show_bug.cgi?id=89872

Eike Rathke <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|medium                      |high
             Status|NEW                         |ASSIGNED
                 CC|                            |winfrieddonkers@libreoffice
                   |                            |.org
           Assignee|[email protected] |[email protected]
                   |desktop.org                 |
           Severity|normal                      |major

--- Comment #3 from Eike Rathke <[email protected]> ---
Tricky.. actually IF(range>0;range) in array context produces an array of
logical values, with the special case here the FALSE-path being omitted. Asking
only for ScMatrix::IsString() in ScInterpreter::GetNumberSequenceArray() is
wrong as it doesn't catch the empty cell / empty path conditions. This seems to
be a regression of 104a1e641554be2e789758ae67c0e24620df8035 in master and
b9088afc9f4f93548c668d81dd55e9bb63f91d80 in 4-4, which introduced a special
handling for the string case to fix bug 88547 that should not be triggered in
this case.

@Winfried: thinking on this further, I reckon the change was wrong and should
be reverted, sorry for accepting it first hand ;-)  The way how
GetNumberSequenceArray() is used (and was implemented) it indeed should ignore
all string and empty values like all spreadsheet functions do that expect a
number sequence as parameter. Actually this may spoil all sorts of things.. Bug
88547 needs to be fixed differently. I'll revert the corresponding changes and
reopen that bug.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to