[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-10-17 Thread oc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User oc changed the following:

What|Old value |New value

  Status|VERIFIED  |CLOSED





--- Additional comments from [EMAIL PROTECTED] Fri Oct 17 09:22:32 + 
2008 ---
closed because fix available in builds OOO300_m9 and DEV300_m33

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-08-11 Thread er
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Mon Aug 11 11:16:08 + 
2008 ---
The number of days Excel returns in this constellation actually depends
on two factors:
- Whether it is a leap year.
- The method given as a 3rd argument to DAYS360.


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-08-11 Thread er
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Mon Aug 11 11:19:50 + 
2008 ---
Created an attachment (id=55697)
test case illustrating some Feb-28 behavior


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-08-07 Thread drking
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Thu Aug  7 21:40:11 + 
2008 ---
Adding a MS reference to this issue:
http://support.microsoft.com/kb/916004

When you use the DAYS360 function to calculate the number of days between two 
dates, an unexpected value is returned.

The article continues:
when you use the DAYS360 function with a start date of February 28 and with an 
end date of March 28, a value of 28 days is returned. You expect a value of 30 
days to be returned for every full month.

and I point out that you do actually expect 28 days, not 30. Possibly MS are 
aware of their bug, but their authors have not explained it properly.



-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-06-04 Thread oc
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User oc changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from [EMAIL PROTECTED] Wed Jun  4 12:35:26 + 
2008 ---
verified in internal build cws_odff3

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-05-09 Thread er
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User er changed the following:

What|Old value |New value

 Assigned to|er|oc





--- Additional comments from [EMAIL PROTECTED] Fri May  9 16:59:18 + 
2008 ---
Reassigning to QA for verification.


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-04-11 Thread er
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User er changed the following:

What|Old value |New value

  Status|NEW   |RESOLVED

  Resolution|  |FIXED

Subcomponent|code  |programming





--- Additional comments from [EMAIL PROTECTED] Fri Apr 11 19:29:30 + 
2008 ---
In cws odff03:

sc/source/core/tool/interpr2.cxx  1.35.20.1

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-09 Thread fst
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User fst changed the following:

What|Old value |New value

 Assigned to|spreadsheet   |er





--- Additional comments from [EMAIL PROTECTED] Wed Jan  9 15:17:17 + 
2008 ---
Hi Eike,

seems to be yours.

Frank

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-09 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Wed Jan  9 15:40:22 + 
2008 ---
Eike, 

I could give you a diff file for this, but I don't have a CVS acct to give you a
cvs diff, so why don't just leave this to you. It's interpr2.cxx:336, change as
above.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-09 Thread er
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Wed Jan  9 17:37:35 + 
2008 ---
No problem with a one line change like the one above. However, to
produce a  cvs diff -pu  you don't need a CVS account, anoncvs does as
well, check out sources using
cvs -d :pserver:[EMAIL PROTECTED]:/cvs ...

Are you sure that change doesn't break the other !bFlag case? I didn't
check yet.

Regarding
 parameter 3.  At the moment it is a boolean.  Why not allow an integer 
 argment 
 also.  Hence
   False = 0   = 30/360 Excel
   True = -1,1 = 30E/360
  2= 30/360 SIA compliant.

That wouldn't do. While zero is interpreted as False, any other value
than zero is interpreted as True. Overloading that in future versions
could break already existing documents that don't care if True is 1 or
some other value than 0, e.g. as a result of an expression.

ANother thing: there once was a reference document SMD_Fields_030802.pdf
publicly available at some financial services provider or so that
claimed the Excel method was called PSA 30 or NASD 30, and Excel was the
only application implementing that. Unfortunately the document isn't
available anymore. Do you happen to know if that is some official
name, or whether there are public references one could point to?

Thanks
  Eike


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-09 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Wed Jan  9 22:22:31 + 
2008 ---
Created an attachment (id=50766)
Ditto + 3 extra sheets givien 30E/360 test vectors + validation delta


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-09 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Wed Jan  9 22:39:59 + 
2008 ---
 However, to produce a  cvs diff -pu  you don't need a CVS account, 
 anoncvs does as well, check out sources using
 cvs -d :pserver:[EMAIL PROTECTED]:/cvs ...

Thanks, I should have thought about anoncvs. This gives enough to google up some
references.

 Are you sure that change doesn't break the other !bFlag case? I didn't check 
 yet.

No this guard condition is to do the flip *ONLY* if true.  It therefore leaves
the true case unchanged.  Trust but verify I've added the extra 30E/360 check
in the test spreadsheet to cover this one explicitly.  The TRUE option used to
and still works.

 ... parameter 3. ... 
 That wouldn't do. While zero is interpreted as False, any other value
 than zero is interpreted as True.

Fine, your the chief engineer for this module.

 ANother thing: there once was a reference document SMD_Fields_030802.pdf
 publicly available at some financial services provider or so that
 claimed the Excel method was called PSA 30 or NASD 30, and Excel was the
 only application implementing that. Unfortunately the document isn't
 available anymore. Do you happen to know if that is some official
 name, or whether there are public references one could point to?

No primary references.  These still seem to be in paper form  only.  Google
wiki 360 day calendar.  I updated the Wikipedia article, and in researching
ths came across two good secondary refs:
  http://www.financialcad.com/support/developerfunc/mathref/Daycount.htm
  http://www.sifma.org/services/publications/calculations-method-volume2.shtml

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-06 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Sun Jan  6 18:51:26 + 
2008 ---
Here's the patch which makes Calc 100% compatible with Excel on this one:

sc/source/core/tool/interpr2.cxx line 335

   if (nDate2  nDate1)

   if (bFlag  (nDate2  nDate1))

Tested on my OOo 2.2.1 sandpit, but I don't think module this was changed in 2.3
or 2.3.1 so this same patch should apply.


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-06 Thread kpalagin
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User kpalagin changed the following:

What|Old value |New value

  CC|'kpalagin'|'er,kpalagin'

  Issue type|TASK  |PATCH





--- Additional comments from [EMAIL PROTECTED] Sun Jan  6 20:22:35 + 
2008 ---
terrye,
I am not sure if you need to file Joint Copyright Agreement and attach .patch 
file for one-liner, but suggest doing so to speed-up acceptance of this change.
I am also CCing Eike Rathke, project lead for Calc.

Thanks for your effort!

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-06 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Mon Jan  7 00:09:52 + 
2008 ---
Already signed the JPA, as this isn't my first fix.  I'll submit the patch.

One for er to ponder, one way of retaining Excel compatability for this 
function, plus allowing a 30/360 SIA compliant option would be to over load 
parameter 3.  At the moment it is a boolean.  Why not allow an integer argment 
also.  Hence
  False = 0   = 30/360 Excel
  True = -1,1 = 30E/360
 2= 30/360 SIA compliant.

Clearly export to XLS should map arg 3 back to T/F so you lose compliance.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-03 Thread er
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User er changed the following:

What|Old value |New value

  CC|'er,kpalagin' |'kpalagin'

 Assigned to|spreadsheet   |er

  Ever confirmed|  |1

  Status|UNCONFIRMED   |NEW

  Issue type|DEFECT|TASK

  OS/Version|Windows XP|All

 Summary|Bug in DAYS360 Algo   |ODFF: DAYS360 compliance

Target milestone|---   |OOo 3.0





--- Additional comments from [EMAIL PROTECTED] Thu Jan  3 12:54:46 + 
2008 ---
Interesting. Needs further investigation, also relevant for ODFF spec. Might
also be interesting to compare with older Excel97 implementation to see if
anything changed.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-03 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934





--- Additional comments from [EMAIL PROTECTED] Thu Jan  3 14:04:23 + 
2008 ---
Created an attachment (id=50643)
Version 2 of test cases


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[sc-issues] [Issue 84934] ODFF: DAYS360 compliance

2008-01-03 Thread terrye
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84934


User terrye changed the following:

What|Old value |New value

 Assigned to|er|spreadsheet





--- Additional comments from [EMAIL PROTECTED] Thu Jan  3 13:59:37 + 
2008 ---
kpalagin, I have updated the ODS to V2 to do the extra comparison,  This
includes the following sheets which evaluate this function for a 36x36 test 
vector:

 ExcelEvaluated on Excel 2003
 Calc Evaluated on Calc 2.3.1
 USDays360Basic implementation of 30/360 (SIA) algo
 ExDays360Basic implementation of the Excel algo
 CalcAlgo360  Basic implementation of the Calc algo
 DiffExcelCalcExcel-Calc Delta
 DiffExcelUSDays360   Excel-30/360 Delta
 DiffCalcUSDays360Calc-30/360 Delta
 DiffExcelExDays360   Excel-Its Basic Implementation Delta(==0)
 DiffCalcCalcAlgo360  Calc-Its Basic Implementation Delta(==0)

Ignore my last post of Wed Jan 2 15:27:10.  I had a mind fart.

The algorithm is for two dates A,B where AB.  Excel gets dates wrong when
comparing last days of leap-years. In the defined range Calc and Excel agree, so
calc also gets it wrong.  Also Microsoft has preserve this same bug since Excel
97 to ensure calculation compatibility.

Where they DO differ is in the case where AB, in that neither error but 
instead adopt different conventions: Calc flips the algo about the origin
so DAYS360(A,B)=-DAYS360(B,A) and Excel just extrapolates (meaninglessly)
backwards so that the two randomly differ by up to 2 days.

The point is that users may use DAYS360 outside of its strict use in actuarial
calcs and win such cases we might get AB.  In such cases loading a working
(that is doing what the user thinks is OK) spreadsheet into Calc will give
different results.



-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]