Summary: | export to OOXML format is broken for Calc functions that need argument changes | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Winfried Donkers <winfrieddonkers> |
Component: | Calc | Assignee: | Winfried Donkers <winfrieddonkers> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | erack, kami911 |
Priority: | medium | ||
Version: | Master old -3.6 | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | target:4.5.0 target:4.4.1 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 40835 |
Description
Winfried Donkers
2014-07-21 11:37:24 UTC
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 |