Skip to content

Commit f0d2ea2

Browse files
authored
Merge pull request #1033 from appwrite/fix-dotnet-and-dart-formatting
fix: dotnet & dart models
2 parents 2d02e13 + 3463325 commit f0d2ea2

File tree

2 files changed

+25
-27
lines changed

2 files changed

+25
-27
lines changed

templates/dart/lib/src/models/model.dart.twig

+5-4
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,12 @@ class {{ definition.name | caseUcfirst | overrideIdentifier }} implements Model
66
{% for property in definition.properties %}
77
/// {{ property.description }}
88
final {% if not property.required %}{{_self.sub_schema(property)}}? {{ property.name | escapeKeyword }}{% else %}{{_self.sub_schema(property)}} {{ property.name | escapeKeyword }}{% endif %};
9+
910
{% endfor %}
10-
{% if definition.additionalProperties %}
11+
{%~ if definition.additionalProperties %}
1112
final Map<String, dynamic> data;
12-
{% endif %}
1313

14+
{% endif %}
1415
{{ definition.name | caseUcfirst | overrideIdentifier}}({% if definition.properties | length or definition.additionalProperties %}{{ '{' }}{% endif %}
1516

1617
{% for property in definition.properties %}
@@ -33,7 +34,7 @@ class {{ definition.name | caseUcfirst | overrideIdentifier }} implements Model
3334
{%- endif -%}
3435
{%- else -%}
3536
{%- if property.type == 'array' -%}
36-
List<{{ property.items.type | caseUcfirst }}>.from(map['{{property.name | escapeDollarSign }}']?.map((x) => x{% if property.items.type == "string" %}.toString(){% endif %}) ?? [])
37+
List.from(map['{{property.name | escapeDollarSign }}'] ?? [])
3738
{%- else -%}
3839
map['{{property.name | escapeDollarSign }}']
3940
{%- if property.type == "number" -%}
@@ -76,4 +77,4 @@ class {{ definition.name | caseUcfirst | overrideIdentifier }} implements Model
7677
{% endfor %}
7778
{% endif %}
7879
{% endfor %}
79-
}
80+
}

templates/dotnet/Package/Models/Model.cs.twig

+20-23
Original file line numberDiff line numberDiff line change
@@ -40,31 +40,28 @@ namespace {{ spec.title | caseUcfirst }}.Models
4040

4141
public static {{ definition.name | caseUcfirst | overrideIdentifier }} From(Dictionary<string, object> map) => new {{ definition.name | caseUcfirst | overrideIdentifier }}(
4242
{%~ for property in definition.properties %}
43-
{{ property.name | caseCamel | escapeKeyword | removeDollarSign }}:
44-
{%- if property.sub_schema %}
45-
{%~ if property.type == 'array' %}
46-
((JArray)map["{{ property.name }}"])
47-
.ToObject<List<Dictionary<string, object>>>()
48-
.Select(it => {{ property.sub_schema | caseUcfirst | overrideIdentifier }}.From(map: it))
49-
.ToList()
50-
{%- else %}
51-
{{ property.sub_schema | caseUcfirst | overrideIdentifier }}.From(
52-
map: ((JObject)map["{{ property.name }}"])
53-
.ToObject<Dictionary<string, object>>()!
54-
)
43+
{{ property.name | caseCamel | escapeKeyword | removeDollarSign }}:{{' '}}
44+
{%- if property.sub_schema %}
45+
{%- if property.type == 'array' -%}
46+
((JArray)map["{{ property.name }}"]).ToObject<List<Dictionary<string, object>>>().Select(it => {{ property.sub_schema | caseUcfirst | overrideIdentifier }}.From(map: it)).ToList()
47+
{%- else -%}
48+
{{ property.sub_schema | caseUcfirst | overrideIdentifier }}.From(map: ((JObject)map["{{ property.name }}"]).ToObject<Dictionary<string, object>>()!)
5549
{%- endif %}
56-
{%~ else %}
57-
{%~ if property.type == 'array' %}
50+
{%- else %}
51+
{%- if property.type == 'array' -%}
5852
((JArray)map["{{ property.name }}"]).ToObject<{{ property | typeName }}>()
59-
{%~ else %}
60-
{%~ if property.type == "integer" or property.type == "number" %}
61-
{%~ if not property.required %}map["{{ property.name }}"] == null ? null : {% endif %} Convert.To{% if property.type == "integer" %}Int64{% else %}Double{% endif %}(map["{{ property.name }}"])
62-
{%~ else %}
63-
{%~ if property.type == "boolean" %}
53+
{%- else %}
54+
{%- if property.type == "integer" or property.type == "number" %}
55+
{%- if not property.required -%}map["{{ property.name }}"] == null ? null :{% endif %}Convert.To{% if property.type == "integer" %}Int64{% else %}Double{% endif %}(map["{{ property.name }}"])
56+
{%- else %}
57+
{%- if property.type == "boolean" -%}
6458
({{ property | typeName }}{% if not property.required %}?{% endif %})map["{{ property.name }}"]
65-
{%~ else %}
66-
{%~ if not property.required %}map.TryGetValue("{{ property.name }}", out var {{ property.name | caseCamel | escapeKeyword | removeDollarSign }}) ? {{ property.name | caseCamel | escapeKeyword | removeDollarSign }}?.ToString() : null
67-
{%~ else %} map["{{ property.name }}"].ToString(){% endif %}
59+
{%- else %}
60+
{%- if not property.required -%}
61+
map.TryGetValue("{{ property.name }}", out var {{ property.name | caseCamel | escapeKeyword | removeDollarSign }}) ? {{ property.name | caseCamel | escapeKeyword | removeDollarSign }}?.ToString() : null
62+
{%- else -%}
63+
map["{{ property.name }}"].ToString()
64+
{%- endif %}
6865
{%- endif %}
6966
{%~ endif %}
7067
{%~ endif %}
@@ -105,4 +102,4 @@ namespace {{ spec.title | caseUcfirst }}.Models
105102
{%~ endif %}
106103
{%~ endfor %}
107104
}
108-
}
105+
}

0 commit comments

Comments
 (0)