RE: hints asked for fdo#59727

2013-05-17 Thread Winfried Donkers
Hi Eike, The correct place would be a duplicate of saFuncTableOdf[], say saFuncTableOOoLO[], just to keep things cleanly separated because the functions are only known to LO and not defined in OpenFormula. The saFuncTableOOoLO also needs to be added to FunctionProviderImpl::FunctionProviderImpl()

Re: hints asked for fdo#59727

2013-05-17 Thread Eike Rathke
Hi Winfried, On Friday, 2013-05-17 16:27:31 +0200, Winfried Donkers wrote: If I understand you correctly, WEEKS in an ods should translate to 'org.openoffice.weeks' when saving tp xlsx, which should be translated to WEEKS when saving to ods. Am I correct? Yes. Respectively use uppercase

RE: hints asked for fdo#59727

2013-05-15 Thread Winfried Donkers
Hi Eike, They are implemented in scaddins/source/datefunc/ I wouldn't have looked for ROT13 there... Is that a location to be maintained or should it be moved to scaddins/source/analysis? It seems they are missing from sc/source/filter/oox/formulabase.cxx so adding them there should solve

RE: hints asked for fdo#59727

2013-05-14 Thread Winfried Donkers
Hi Eike, There are more problems with the add-in functions, see attachments with bug 59727. I'd start by setting a breakpoint in formula/source/core/api/FormulaCompiler.cxx FormulaCompiler::CreateStringFromToken() for case svExternal and step through to see what is actually executed and

Re: hints asked for fdo#59727

2013-05-14 Thread Eike Rathke
Hi Winfried, On Tuesday, 2013-05-14 13:02:04 +0200, Winfried Donkers wrote: There are more problems with the add-in functions, see attachments with bug 59727. I'd start by setting a breakpoint in formula/source/core/api/FormulaCompiler.cxx FormulaCompiler::CreateStringFromToken()

Re: hints asked for fdo#59727

2013-05-07 Thread Eike Rathke
Hi Winfried, On Thursday, 2013-05-02 12:24:42 +0200, Winfried Donkers wrote: Only if the .xlsx was saved with Calc 4.0, then the file contains com.sun.star.sheet.addin.DateFunctions.getDiffMonths, an original file containing MONTHS() also does not work though, the MONTHS is not

RE: hints asked for fdo#59727

2013-05-07 Thread Winfried Donkers
Hi Eike, There are more problems with the add-in functions, see attachments with bug 59727. I assumed so.. There are several places where things could go wrong.. [...] I'd start by setting a breakpoint in formula/source/core/api/FormulaCompiler.cxx

RE: hints asked for fdo#59727

2013-05-02 Thread Winfried Donkers
Hi Eike, Only if the .xlsx was saved with Calc 4.0, then the file contains com.sun.star.sheet.addin.DateFunctions.getDiffMonths, an original file containing MONTHS() also does not work though, the MONTHS is not recognized hence displayed as lowercase months. Seems to be a specific problem

Re: hints asked for fdo#59727

2013-04-26 Thread Eike Rathke
Hi Winfried, On Thursday, 2013-04-25 08:31:39 +0200, Winfried Donkers wrote: Bug fdo59727 is about addin function MONTHS() that is not interpreted correctly when read from an xlsx file. That is, an xlsx file with the function MONTHS(...) opened with calc shows

hints asked for fdo#59727

2013-04-25 Thread Winfried Donkers
Hi, Bug fdo59727 is about addin function MONTHS() that is not interpreted correctly when read from an xlsx file. That is, an xlsx file with the function MONTHS(...) opened with calc shows com.sun.star.sheet.addin.datefunctions.getdiffmonths(...) and understandably results to #NAME? . The