Skip to content

Commit

Permalink
Generate java.time date and time types instead of joda types
Browse files Browse the repository at this point in the history
  • Loading branch information
Dan Schulte committed Mar 9, 2018
1 parent 1f57f00 commit 3aec111
Show file tree
Hide file tree
Showing 46 changed files with 1,651 additions and 1,894 deletions.
30 changes: 15 additions & 15 deletions src/JavaCodeGenerator.cs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ public class JavaCodeGenerator : CodeGenerator
"byte[]", "Byte[]",
"String",
"LocalDate",
"DateTime",
"OffsetDateTime",
"DateTimeRfc1123",
"Duration",
"Period",
Expand Down Expand Up @@ -1427,10 +1427,10 @@ private static IType ParseType(AutoRestIModelType autoRestIModelType, JavaSettin
result = ArrayType.ByteArray;
break;
case AutoRestKnownPrimaryType.Date:
result = ClassType.JodaLocalDate;
result = ClassType.LocalDate;
break;
case AutoRestKnownPrimaryType.DateTime:
result = ClassType.JodaDateTime;
result = ClassType.DateTime;
break;
case AutoRestKnownPrimaryType.DateTimeRfc1123:
result = ClassType.DateTimeRfc1123;
Expand Down Expand Up @@ -1461,7 +1461,7 @@ private static IType ParseType(AutoRestIModelType autoRestIModelType, JavaSettin
}
break;
case AutoRestKnownPrimaryType.TimeSpan:
result = ClassType.JodaPeriod;
result = ClassType.Duration;
break;
case AutoRestKnownPrimaryType.UnixTime:
result = PrimitiveType.UnixTimeLong;
Expand Down Expand Up @@ -2683,7 +2683,7 @@ public static JavaFile GetModelJavaFile(ServiceModel model, JavaSettings setting
string propertyConversion = null;
switch (sourceTypeName.ToLower())
{
case "datetime":
case "offsetdatetime":
switch (targetTypeName.ToLower())
{
case "datetimerfc1123":
Expand All @@ -2695,7 +2695,7 @@ public static JavaFile GetModelJavaFile(ServiceModel model, JavaSettings setting
case "datetimerfc1123":
switch (targetTypeName.ToLower())
{
case "datetime":
case "offsetdatetime":
propertyConversion = $"{expression}.dateTime()";
break;
}
Expand Down Expand Up @@ -2741,7 +2741,7 @@ public static JavaFile GetModelJavaFile(ServiceModel model, JavaSettings setting
{
switch (sourceTypeName.ToLower())
{
case "datetime":
case "offsetdatetime":
switch (targetTypeName.ToLower())
{
case "datetimerfc1123":
Expand All @@ -2753,7 +2753,7 @@ public static JavaFile GetModelJavaFile(ServiceModel model, JavaSettings setting
case "datetimerfc1123":
switch (targetTypeName.ToLower())
{
case "datetime":
case "offsetdatetime":
propertyConversion = $"{expression}.dateTime()";
break;
}
Expand Down Expand Up @@ -3031,7 +3031,7 @@ private static IType ConvertToClientType(IType modelType)
}
else if (modelType == ClassType.DateTimeRfc1123)
{
clientType = ClassType.JodaDateTime;
clientType = ClassType.DateTime;
}
else if (modelType == PrimitiveType.UnixTimeLong)
{
Expand Down Expand Up @@ -3179,7 +3179,7 @@ private static string AutoRestIModelTypeName(AutoRestIModelType autoRestModelTyp
result = "LocalDate";
break;
case AutoRestKnownPrimaryType.DateTime:
result = "DateTime";
result = "OffsetDateTime";
break;
case AutoRestKnownPrimaryType.DateTimeRfc1123:
result = "DateTimeRfc1123";
Expand Down Expand Up @@ -3302,14 +3302,14 @@ private static void ParameterConvertClientTypeToWireType(JavaBlock block, JavaSe
{
if (parameterIsRequired)
{
block.Line($"Long {target} = {source}.toDateTime(DateTimeZone.UTC).getMillis() / 1000;");
block.Line($"Long {target} = {source}.toInstant().getEpochSecond();");
}
else
{
block.Line($"Long {target} = null;");
block.If($"{source} != null", ifBlock =>
{
ifBlock.Line($"{target} = {source}.toDateTime(DateTimeZone.UTC).getMillis() / 1000;");
ifBlock.Line($"{target} = {source}.toInstant().getEpochSecond();");
});
}
}
Expand Down Expand Up @@ -4596,10 +4596,10 @@ private static IEnumerable<string> GetExpressionsToValidate(RestAPIMethod restAP
parameterType != ClassType.Double &&
parameterType != ClassType.BigDecimal &&
parameterType != ClassType.String &&
parameterType != ClassType.JodaDateTime &&
parameterType != ClassType.JodaLocalDate &&
parameterType != ClassType.DateTime &&
parameterType != ClassType.LocalDate &&
parameterType != ClassType.DateTimeRfc1123 &&
parameterType != ClassType.JodaPeriod &&
parameterType != ClassType.Duration &&
parameterType != ClassType.Boolean &&
parameterType != ClassType.ServiceClientCredentials &&
parameterType != ClassType.AzureTokenCredentials &&
Expand Down
10 changes: 5 additions & 5 deletions src/Model/ClassType.cs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ public class ClassType : IType
public static readonly ClassType Double = new ClassType("java.lang", "Double", defaultValueExpressionConverter: (string defaultValueExpression) => double.Parse(defaultValueExpression).ToString());
public static readonly ClassType String = new ClassType("java.lang", "String", defaultValueExpressionConverter: (string defaultValueExpression) => CodeNamer.Instance.QuoteValue(defaultValueExpression));
public static readonly ClassType Base64Url = new ClassType("com.microsoft.rest.v2", "Base64Url");
public static readonly ClassType JodaLocalDate = new ClassType("org.joda.time", "LocalDate", defaultValueExpressionConverter: (string defaultValueExpression) => $"LocalDate.parse(\"{defaultValueExpression}\")");
public static readonly ClassType JodaDateTime = new ClassType("org.joda.time", "DateTime", defaultValueExpressionConverter: (string defaultValueExpression) => $"DateTime.parse(\"{defaultValueExpression}\")");
public static readonly ClassType JodaPeriod = new ClassType("org.joda.time", "Period", defaultValueExpressionConverter: (string defaultValueExpression) => $"Period.parse(\"{defaultValueExpression}\")");
public static readonly ClassType DateTimeRfc1123 = new ClassType("com.microsoft.rest.v2", "DateTimeRfc1123", defaultValueExpressionConverter: (string defaultValueExpression) => $"DateTime.parse(\"{defaultValueExpression}\")");
public static readonly ClassType LocalDate = new ClassType("java.time", "LocalDate", defaultValueExpressionConverter: (string defaultValueExpression) => $"LocalDate.parse(\"{defaultValueExpression}\")");
public static readonly ClassType DateTime = new ClassType("java.time", "OffsetDateTime", defaultValueExpressionConverter: (string defaultValueExpression) => $"OffsetDateTime.parse(\"{defaultValueExpression}\")");
public static readonly ClassType Duration = new ClassType("java.time", "Duration", defaultValueExpressionConverter: (string defaultValueExpression) => $"Duration.parse(\"{defaultValueExpression}\")");
public static readonly ClassType DateTimeRfc1123 = new ClassType("com.microsoft.rest.v2", "DateTimeRfc1123", defaultValueExpressionConverter: (string defaultValueExpression) => $"new DateTimeRfc1123(\"{defaultValueExpression}\")");
public static readonly ClassType BigDecimal = new ClassType("java.math", "BigDecimal");
public static readonly ClassType UUID = new ClassType("java.util", "UUID");
public static readonly ClassType Object = new ClassType("java.lang", "Object");
Expand All @@ -29,7 +29,7 @@ public class ClassType : IType
public static readonly ClassType CloudException = new ClassType("com.microsoft.azure.v2", "CloudException");
public static readonly ClassType RestException = new ClassType("com.microsoft.azure.v2", "RestException");
public static readonly ClassType UnixTime = new ClassType("com.microsoft.rest.v2", "UnixTime");
public static readonly ClassType UnixTimeDateTime = new ClassType("org.joda.time", "DateTime", new[] { "org.joda.time.DateTimeZone" });
public static readonly ClassType UnixTimeDateTime = new ClassType("java.time", "OffsetDateTime");
public static readonly ClassType UnixTimeLong = new ClassType("java.lang", "Long");
public static readonly ClassType AzureEnvironment = new ClassType("com.microsoft.azure.v2", "AzureEnvironment");
public static readonly ClassType HttpPipeline = new ClassType("com.microsoft.rest.v2.http", "HttpPipeline");
Expand Down
Loading

0 comments on commit 3aec111

Please sign in to comment.