Merge pull request DeBortoliWines#52 from BT-aestebanez/master

flotho · web-flow · commit f2f12961a6d6 · 2018-10-05T11:31:27.000+02:00
[FIX] Dates should not take into account the timezone

Great thanks for this great contribution.
We also experienced such issues and thanks for the compliment.

BTW, do you use the odoo step available in the Pentaho Kettle?
diff --git a/src/main/java/com/odoojava/api/Row.java b/src/main/java/com/odoojava/api/Row.java
@@ -130,6 +130,7 @@ public Object get(String fieldName){
 		
 		if (value instanceof String && fieldType == Field.FieldType.DATE){
 			DateFormat dfm = new SimpleDateFormat("yyyy-MM-dd");
+			dfm.setTimeZone(TimeZone.getTimeZone("UTC"));
 			try{
 				return dfm.parse(value.toString());
 			}
@@ -140,6 +141,7 @@ public Object get(String fieldName){
 		
 		if (value instanceof String && fieldType == Field.FieldType.DATETIME){
 			DateFormat dfm = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
+			dfm.setTimeZone(TimeZone.getTimeZone("UTC"));
 			try{
 				return dfm.parse(value.toString());
 			}

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,7 @@ public Object get(String fieldName){`
`130`	`130`
`131`	`131`	`if (value instanceof String && fieldType == Field.FieldType.DATE){`
`132`	`132`	`DateFormat dfm = new SimpleDateFormat("yyyy-MM-dd");`
	`133`	`+ dfm.setTimeZone(TimeZone.getTimeZone("UTC"));`
`133`	`134`	`try{`
`134`	`135`	`return dfm.parse(value.toString());`
`135`	`136`	`}`
`@@ -140,6 +141,7 @@ public Object get(String fieldName){`
`140`	`141`
`141`	`142`	`if (value instanceof String && fieldType == Field.FieldType.DATETIME){`
`142`	`143`	`DateFormat dfm = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");`
	`144`	`+ dfm.setTimeZone(TimeZone.getTimeZone("UTC"));`
`143`	`145`	`try{`
`144`	`146`	`return dfm.parse(value.toString());`
`145`	`147`	`}`