Merge remote-tracking branch 'origin/master'

This commit is contained in:
Weblate 2018-02-22 10:49:53 +01:00
commit 41f4b70bb1
4 changed files with 36 additions and 32 deletions

View File

@ -50,29 +50,30 @@ import com.google.refine.expr.EvalError;
import com.google.refine.expr.util.CalendarParser;
import com.google.refine.expr.util.CalendarParserException;
import com.google.refine.grel.Function;
import com.google.refine.grel.ControlFunctionRegistry;
import com.google.refine.util.ParsingUtilities;
public class ToDate implements Function {
@Override
public Object call(Properties bindings, Object[] args) {
if (args.length == 0) {
// missing value, can this happen?
return null;
}
Object arg0 = args[0];
String o1;
if (args.length == 0) {
return new EvalError(ControlFunctionRegistry.getFunctionName(this) + " expects at least one argument");
} else {
Object arg0 = args[0];
if (arg0 instanceof Date) {
return arg0;
} else if (arg0 instanceof Calendar) {
return ((Calendar) arg0).getTime();
} else if (arg0 instanceof Long) {
o1 = ((Long) arg0).toString(); // treat integers as years
} else if (arg0 instanceof String) {
} else if (arg0 instanceof String && arg0.toString().trim().length() > 0) {
o1 = (String) arg0;
} else {
// ignore cell values that aren't strings
return new EvalError("Not a String - cannot parse to date");
// ignore cell values that aren't Date, Calendar, Long or String
return new EvalError("Unable to parse as date");
}
}
// "o, boolean month_first (optional)"
@ -98,13 +99,12 @@ public class ToDate implements Function {
// } catch (DatatypeConfigurationException e2) {
// }
}
return new EvalError("Cannot parse to date");
return new EvalError("Unable to parse as date");
}
}
} else if (args.length>=2) {
// "o, format1, format2 (optional), ..."
Locale locale = Locale.getDefault();
if (args.length>=2) {
for (int i=1;i<args.length;i++) {
if (!(args[i] instanceof String)) {
// skip formats that aren't strings
@ -156,9 +156,9 @@ public class ToDate implements Function {
}
}
return new EvalError("Unable to parse as date");
} else {
return new EvalError("Unable to parse as date");
}
return null;
}

View File

@ -39,6 +39,7 @@ import org.json.JSONException;
import org.json.JSONWriter;
import com.google.refine.expr.EvalError;
import com.google.refine.grel.ControlFunctionRegistry;
import com.google.refine.grel.Function;
public class ToNumber implements Function {
@ -58,13 +59,16 @@ public class ToNumber implements Function {
try {
return Double.parseDouble(s);
} catch (NumberFormatException e) {
return new EvalError("Cannot parse to number");
return new EvalError("Unable to parse as number");
}
} else {
return new EvalError("Unable to parse as number");
}
}
} else {
return new EvalError(ControlFunctionRegistry.getFunctionName(this) + " expects one non-null argument");
}
}
return null;
}
@Override
public void write(JSONWriter writer, Properties options)

View File

@ -43,6 +43,7 @@ import org.json.JSONException;
import org.json.JSONWriter;
import com.google.refine.expr.EvalError;
import com.google.refine.grel.ControlFunctionRegistry;
import com.google.refine.grel.Function;
import com.google.refine.util.StringUtils;
@ -68,7 +69,7 @@ public class ToString implements Function {
}
}
}
return new EvalError("ToString accepts an object and an optional second argument containing a date format string");
return new EvalError(ControlFunctionRegistry.getFunctionName(this) + " accepts an object and an optional second argument containing a date format string");
}

View File

@ -98,10 +98,9 @@ public class ToFromConversionTests extends RefineTest {
@Test
public void testToNumber() {
// Assert.assertTrue(invoke("toNumber") instanceof EvalError);
Assert.assertNull(invoke("toNumber"));
// Assert.assertTrue(invoke("toNumber", (Object) null) instanceof EvalError);
Assert.assertNull(invoke("toNumber", (Object) null));
Assert.assertTrue(invoke("toNumber") instanceof EvalError);
Assert.assertTrue(invoke("toNumber", (Object) null) instanceof EvalError);
Assert.assertTrue(invoke("toNumber", "") instanceof EvalError);
Assert.assertTrue(invoke("toNumber", "string") instanceof EvalError);
Assert.assertEquals(invoke("toNumber", "0.0"), 0.0);
Assert.assertEquals(invoke("toNumber", "123"), Long.valueOf(123));
@ -128,9 +127,9 @@ public class ToFromConversionTests extends RefineTest {
@Test
public void testToDate() throws CalendarParserException {
// Assert.assertTrue(invoke("toDate") instanceof EvalError);
Assert.assertNull(invoke("toDate"));
Assert.assertTrue(invoke("toDate") instanceof EvalError);
Assert.assertTrue(invoke("toDate", (Object) null) instanceof EvalError);
Assert.assertTrue(invoke("toDate", "") instanceof EvalError);
Assert.assertTrue(invoke("toDate", 1.0) instanceof EvalError);
Assert.assertTrue(invoke("toDate", "2012-03-01","xxx") instanceof EvalError); // bad format string
Assert.assertTrue(invoke("toDate", "2012-03-01") instanceof GregorianCalendar);