summaryrefslogtreecommitdiff
path: root/oox/source/docprop
diff options
context:
space:
mode:
authorLionel Elie Mamane <lionel@mamane.lu>2013-07-05 17:12:51 +0200
committerTor Lillqvist <tml@iki.fi>2013-08-09 08:47:42 +0000
commit689e4849b797a47030dd8e2a84ae7c97bfc38d3a (patch)
tree3f9bb096fb136e085137e4d7865c2b8381903614 /oox/source/docprop
parent91ec774c9fff46af6800e75315561e86167fe5d1 (diff)
Fix timezone && fractional second handling of GetDateTimeFromW3CDTF
1) The timezone correction was inverted. Imagine we are parsing "2004-03-02T14:18:20+02". Before correction, aOslDTime contains "2004-03-02 14:18:20". According to the comment, we want to convert that to UTC time, so we have to *subtract* 2 hours, not *add* two hours. From http://www.w3.org/TR/NOTE-datetime: A time zone offset of "+hh:mm" indicates (...) a local time zone which is "hh" hours and "mm" minutes *ahead* of UTC. So if it is 14:18:20 in the timezone two hours ahead of UTC, it is two hours *earlier* in UTC, namely 12:18:20, and we need to *subtract* two hours, not *add* two hours. 2) Handling of fractions of a second was buggy: It reads only one digit after the dot. This could be a valid implementation decision to handle only deciseconds. However: 1) It then multiplies that by 10^9 (10e8 == 10*10^8, not 10^8!), and sticks that in the NanoSeconds field... That is 10 times too big: 0.3s == 3*10^8 ns 2) If there were additional digits, it then looks for the timezone offset specificator (beginning with '+' or '-') at these additional digits; it does not skip them: nOptTime is set to 0+3+2, i.e. to 5. Change-Id: I4738dc069e37f29c8bbd9e689689a933027af840 Reviewed-on: https://gerrit.libreoffice.org/4743 Reviewed-by: Tor Lillqvist <tml@iki.fi> Tested-by: Tor Lillqvist <tml@iki.fi>
Diffstat (limited to 'oox/source/docprop')
-rw-r--r--oox/source/docprop/docprophandler.cxx56
1 files changed, 43 insertions, 13 deletions
diff --git a/oox/source/docprop/docprophandler.cxx b/oox/source/docprop/docprophandler.cxx
index b604e9bdf387..e9a35b2cff40 100644
--- a/oox/source/docprop/docprophandler.cxx
+++ b/oox/source/docprop/docprophandler.cxx
@@ -95,46 +95,76 @@ void OOXMLDocPropHandler::AddCustomProperty( const uno::Any& aAny )
util::DateTime OOXMLDocPropHandler::GetDateTimeFromW3CDTF( const OUString& aChars )
{
oslDateTime aOslDTime = { 0, 0, 0, 0, 0, 0, 0, 0 };
- sal_Int32 nLen = aChars.getLength();
+ const sal_Int32 nLen = aChars.getLength();
if ( nLen >= 4 )
{
aOslDTime.Year = (sal_Int16)aChars.copy( 0, 4 ).toInt32();
- if ( nLen >= 7 && aChars.getStr()[4] == (sal_Unicode)'-' )
+ if ( nLen >= 7 && aChars[4] == (sal_Unicode)'-' )
{
aOslDTime.Month = (sal_uInt16)aChars.copy( 5, 2 ).toInt32();
- if ( nLen >= 10 && aChars.getStr()[7] == (sal_Unicode)'-' )
+ if ( nLen >= 10 && aChars[7] == (sal_Unicode)'-' )
{
aOslDTime.Day = (sal_uInt16)aChars.copy( 8, 2 ).toInt32();
- if ( nLen >= 16 && aChars.getStr()[10] == (sal_Unicode)'T' && aChars.getStr()[13] == (sal_Unicode)':' )
+ if ( nLen >= 16 && aChars[10] == (sal_Unicode)'T' && aChars[13] == (sal_Unicode)':' )
{
aOslDTime.Hours = (sal_uInt16)aChars.copy( 11, 2 ).toInt32();
aOslDTime.Minutes = (sal_uInt16)aChars.copy( 14, 2 ).toInt32();
sal_Int32 nOptTime = 0;
- if ( nLen >= 19 && aChars.getStr()[16] == (sal_Unicode)':' )
+ if ( nLen >= 19 && aChars[16] == (sal_Unicode)':' )
{
aOslDTime.Seconds = (sal_uInt16)aChars.copy( 17, 2 ).toInt32();
nOptTime += 3;
- if ( nLen >= 21 && aChars.getStr()[19] == (sal_Unicode)'.' )
+ if ( nLen >= 20 && aChars[19] == (sal_Unicode)'.' )
{
- aOslDTime.NanoSeconds = (sal_uInt32)(aChars.copy( 20, 1 ).toInt32() * 10e8);
- nOptTime += 2;
+ nOptTime += 1;
+ sal_Int32 digitPos = 20;
+ while (nLen > digitPos && digitPos < 29)
+ {
+ sal_Unicode c = aChars[digitPos];
+ if ( c < '0' || c > '9')
+ break;
+ aOslDTime.NanoSeconds *= 10;
+ aOslDTime.NanoSeconds += c - '0';
+ ++digitPos;
+ }
+ if ( digitPos < 29 )
+ {
+ // read less digits than 9
+ // add correct exponent of 10
+ nOptTime += digitPos - 20;
+ for(; digitPos<29; ++digitPos)
+ {
+ aOslDTime.NanoSeconds *= 10;
+ }
+ }
+ else
+ {
+ //skip digits with more precision than we can handle
+ while(nLen > digitPos)
+ {
+ sal_Unicode c = aChars[digitPos];
+ if ( c < '0' || c > '9')
+ break;
+ ++digitPos;
+ }
+ nOptTime += digitPos - 20;
+ }
}
}
sal_Int32 nModif = 0;
if ( nLen >= 16 + nOptTime + 6 )
{
- if ( ( aChars.getStr()[16 + nOptTime] == (sal_Unicode)'+' || aChars.getStr()[16 + nOptTime] == (sal_Unicode)'-' )
- && aChars.getStr()[16 + nOptTime + 3] == (sal_Unicode)':' )
-
+ if ( ( aChars[16 + nOptTime] == (sal_Unicode)'+' || aChars[16 + nOptTime] == (sal_Unicode)'-' )
+ && aChars[16 + nOptTime + 3] == (sal_Unicode)':' )
{
nModif = aChars.copy( 16 + nOptTime + 1, 2 ).toInt32() * 3600;
nModif += aChars.copy( 16 + nOptTime + 4, 2 ).toInt32() * 60;
- if ( aChars.getStr()[16 + nOptTime] == (sal_Unicode)'-' )
+ if ( aChars[16 + nOptTime] == (sal_Unicode)'-' )
nModif *= -1;
}
}
@@ -145,7 +175,7 @@ util::DateTime OOXMLDocPropHandler::GetDateTimeFromW3CDTF( const OUString& aChar
TimeValue aTmp;
if ( osl_getTimeValueFromDateTime( &aOslDTime, &aTmp ) )
{
- aTmp.Seconds += nModif;
+ aTmp.Seconds -= nModif;
osl_getDateTimeFromTimeValue( &aTmp, &aOslDTime );
}
}