When saving a Calc document to OOXML, functions that need changes in arguments (e.g. addition of arguments tha are optional in calc and mandatory in Excel) are no longer adjusted to Excel specifications. Example: GAMMADIST(1,2,5,0) should be written as GAMMADIST(1,2,5,1), but is written as GAMMADIST(1,2,5), which will produce an error in Excel. First occurrence version number is a bit of a guess, it at least occurs with version 4.2.5.
Was always the case, setting Version to Master-old; and Component to Spreadsheet.
I'll give it a try, using formula/source/core/api/FormulaCompiler.cxx FormulaCompiler::CreateStringFromTokenArray() to handle OOXML-functions (FormulaGrammar::isOOXML( meGrammar ) being true) in a similar way PODF-functions are handled. Don't expect instant fixes, my available time is limited.
Winfried Donkers committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4feaf96f50fa89eccb4e7a638220099e9a8482f1 fdo#81596 export to OOXML that need argument changes It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3dd241c52d6db424bd4696ff8f223976542e411b sensible indenting, fdo#81596 related It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=14aabee5a73e57e26b2062b1ebf00c3f1dff1250 simplify RewriteMissing() and switch in Add...(), fdo#81596 related It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8f031861f590ba914321816657a003375d93ef5d write TRUE() for 2nd IF() parameter, fdo#81596 related It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Let's call this fixed.
*** Bug 89042 has been marked as a duplicate of this bug. ***
Will this patch get backported to 4.4, 4.3?
For 4-4 I'll try, for 4-3 probably not as it is quite invasive and 4.3.7 would be the last release from the 4-3 branch.
Pending review at https://gerrit.libreoffice.org/14388 for 4-4
Cherry-picked into 4-4: http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-4-4&id=1706760769018662a50b494b2e4a0ca85c93ca42
Cherry-picked into 4-4-1: http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-4-4-1&id=37bf959783d34fe7cfdd0b2541056fd746e7584c