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

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::FunctionProviderIm

Re: hints asked for fdo#59727

2013-05-15 Thread Eike Rathke
Hi Winfried, On Wednesday, 2013-05-15 11:23:56 +0200, Winfried Donkers wrote: > > 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? Keep things as they are. Tr

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 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::CreateStringF

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 execut

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 > FormulaCompiler::CreateStr

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

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 > proble

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 > "com.sun.star.sheet.addi