Issue 111413 - [ODFF] implement secant functions
Summary: [ODFF] implement secant functions
Status: RESOLVED FIXED
Alias: None
Product: Calc
Classification: Application
Component: code (show other issues)
Version: DEV300m76
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-05 22:20 UTC by Regina Henschel
Modified: 2013-08-07 15:13 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: Easy


Attachments
set of patches to DEV300m76 (3.65 KB, application/x-compressed)
2010-05-05 22:21 UTC, Regina Henschel
no flags Details
testcase (10.04 KB, application/vnd.oasis.opendocument.spreadsheet)
2011-03-01 16:27 UTC, ooo
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description Regina Henschel 2010-05-05 22:20:50 UTC
ODF 1.2 defines the function SEC, CSC, SECH and CSCH. OOo should implement them.
Gnumeric knows this functions already.

I have made the necessary changes, but any filters.
Comment 1 Regina Henschel 2010-05-05 22:21:49 UTC
Created attachment 69323 [details]
set of patches to DEV300m76
Comment 2 ooo 2010-12-03 16:51:14 UTC
Applied with two changes:
* In the mean time the help ID system changed from numeric IDs (formerly for
  interpreter functions in sc/inc/scfuncs.hrc) to string IDs, now in
  sc/inc/helpids.h
* Implementation allowed the functions to have more than one parameter, of
  which only the last is evaluated. There's a special section in
  formula/inc/formula/compiler.hrc between SC_OPCODE_START_1_PAR and
  SC_OPCODE_STOP_1_PAR that lets the compiler check for exactly one parameter
  being passed for those functions. Moved SC_OPCODE_COSECANT,
  SC_OPCODE_SECANT, SC_OPCODE_COSECANT_HYP and SC_OPCODE_SECANT_HYP there.

Also added RTL_LOGFILE_CONTEXT_AUTHOR to interpreter functions, credit to whom
credit is due ;-)
Comment 3 ooo 2010-12-03 16:53:13 UTC
In cws dr78:

changeset ff05fcb4fe39
http://hg.services.openoffice.org/cws/dr78/changeset/ff05fcb4fe39
M formula/inc/formula/compiler.hrc
M formula/inc/formula/opcode.hxx
M formula/source/core/resource/core_resource.src
M sc/inc/helpids.h
M sc/source/core/inc/interpre.hxx
M sc/source/core/tool/interpr1.cxx
M sc/source/core/tool/interpr4.cxx
M sc/source/ui/src/scfuncs.src
M sc/util/hidother.src

You can observe the progress and possible integration date of CWS dr78 at
http://tools.services.openoffice.org/EIS2/cws.ShowCWS?Path=DEV300%2Fdr78
Comment 4 daniel.rentz 2010-12-08 15:16:45 UTC
added support for the new secant functions to the Excel XLS export filter.
Every occurance of SEC(x) will be replaced with (1/COS(x));
CSC(x) with (1/SIN(x));
SECH(x) with (1/COSH(x));
CSCH(x) with (1/SINH(x)).
Comment 5 ooo 2011-03-01 16:27:15 UTC
Created attachment 75958 [details]
testcase
Comment 6 ooo 2011-03-01 16:31:35 UTC
Reassigning to QA for verification.
Comment 7 kla 2011-03-09 13:51:08 UTC
Seen ok in cws -> verified
Comment 8 ooo 2011-03-10 12:20:39 UTC
grml.. CWS dr78 didn't make it before translation freeze, string changes would pollute translation data. Removed all translation relevant strings from SEC, CSC, SECH and CSCH. The core feature is still available, documents containing the functions can be loaded and saved, but the UI function names are non-translatable English names and function wizard resource is not available.

TODO: undo this change for the next release.

In cws dr78:

changeset 12adc8781f9c http://hg.services.openoffice.org/cws/dr78/changeset/12adc8781f9c
M formula/source/core/resource/core_resource.src
M sc/inc/helpids.h
M sc/source/ui/src/scfuncs.src
M sc/util/hidother.src
Comment 9 Pedro Giffuni 2013-02-08 05:20:32 UTC
(In reply to comment #8)
> grml.. CWS dr78 didn't make it before translation freeze, string changes
> would pollute translation data. Removed all translation relevant strings
> from SEC, CSC, SECH and CSCH. The core feature is still available, documents
> containing the functions can be loaded and saved, but the UI function names
> are non-translatable English names and function wizard resource is not
> available.
> 
> TODO: undo this change for the next release.
> 
> In cws dr78:
> 
> changeset 12adc8781f9c
> http://hg.services.openoffice.org/cws/dr78/changeset/12adc8781f9c
> M formula/source/core/resource/core_resource.src
> M sc/inc/helpids.h
> M sc/source/ui/src/scfuncs.src
> M sc/util/hidother.src

The link is this:
http://hg.services.openoffice.org/DEV300/rev/12adc8781f9c

We just have to put back the translation strings so our pootle guys do their job.
Looks like an easy task for a beginner.
Comment 10 jsc 2013-02-25 17:01:52 UTC
reintegrated the resource strings for localization
Comment 11 SVN Robot 2013-02-26 12:41:21 UTC
"jsc" committed SVN revision 1450153 into trunk:
#111413# reintegrate resource strings for translation
Comment 12 jsc 2013-02-26 14:33:55 UTC
reintegrate resource strings, they should be now contained in the next translation cycle