Skip to content

Commit 1edf935

Browse files
authored
Merge pull request #505 from appwrite/fix-flutter-sdk-alignment
Fix alignment and spacing issues for Flutter SDK
2 parents aef1fd4 + 556d759 commit 1edf935

File tree

5 files changed

+23
-24
lines changed

5 files changed

+23
-24
lines changed

src/SDK/Language/Dart.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ public function getFilters(): array
428428
new TwigFilter('dartComment', function ($value) {
429429
$value = explode("\n", $value);
430430
foreach ($value as $key => $line) {
431-
$value[$key] = " /// " . wordwrap($value[$key], 75, "\n /// ");
431+
$value[$key] = " /// " . wordwrap($value[$key], 75, "\n /// ");
432432
}
433433
return implode("\n", $value);
434434
}, ['is_safe' => ['html']]),

templates/flutter/base/requests/api.twig

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
{% import 'flutter/base/utils.twig' as utils %}
22
final Map<String, dynamic> params = {
3-
{{ utils.map_parameter(method.parameters.query) }}
4-
{{ utils.map_parameter(method.parameters.body) }}
3+
{{- utils.map_parameter(method.parameters.query) -}}
4+
{{~ utils.map_parameter(method.parameters.body) }}
55
};
66

77
final Map<String, String> headers = {
8-
{{ utils.map_headers(method.headers) }}
8+
{{~ utils.map_headers(method.headers) }}
99
};
1010

1111
final res = await client.call(HttpMethod.{{ method.method | caseLower }}, path: path, params: params, headers: headers);

templates/flutter/base/requests/file.twig

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
1-
{% import 'flutter/base/utils.twig' as utils %}
1+
{% import 'flutter/base/utils.twig' as utils %}
22
final Map<String, dynamic> params = {
3-
{{ utils.map_parameter(method.parameters.query) }}
4-
{{ utils.map_parameter(method.parameters.body) }}
3+
{{~ utils.map_parameter(method.parameters.query) }}
4+
{{~ utils.map_parameter(method.parameters.body) }}
55
};
66

77
final Map<String, String> headers = {
8-
{{ utils.map_headers(method.headers) }}
8+
{{~ utils.map_headers(method.headers) }}
99
};
1010

1111
{% if 'multipart/form-data' in method.consumes %}

templates/flutter/base/utils.twig

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
1-
{% macro map_parameter(parameters) %}
2-
{% for parameter in parameters %}
3-
'{{ parameter.name }}': {{ parameter.name | caseCamel | overrideIdentifier }},
4-
{% endfor %}
5-
{% endmacro %}
1+
{%- macro map_parameter(parameters) -%}
2+
{%- for parameter in parameters ~%}
3+
'{{ parameter.name }}': {{ parameter.name | caseCamel | overrideIdentifier }},
4+
{%- endfor ~%}
5+
{%- endmacro ~%}
66

7-
{% macro map_headers(headers) %}
8-
{% for key, header in headers %}
7+
{% macro map_headers(headers) -%}
8+
{%- for key, header in headers %}
99
'{{ key }}': '{{ header }}',
10-
{% endfor %}
11-
{% endmacro %}
10+
{%- endfor -%}
11+
{%- endmacro -%}

templates/flutter/lib/services/service.dart.twig

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ part of {{ language.params.packageName }};
88
{% endmacro %}
99

1010
{%if service.description %}
11-
{{ service.description|dartComment}}
11+
{{- service.description|dartComment}}
1212
{% endif %}
1313
class {{ service.name | caseUcfirst }} extends Service {
1414
{{ service.name | caseUcfirst }}(Client client{{ _self.service_params(service.globalParams) }}): super(client);
@@ -19,13 +19,13 @@ class {{ service.name | caseUcfirst }} extends Service {
1919
{% endif %}
2020
{% for method in service.methods %}
2121

22-
/// {{ method.title }}
23-
{% if method.description %}
24-
///
22+
/// {{ method.title }}
23+
{%~ if method.description %}
24+
///
2525
{{ method.description|dartComment }}
26-
///
26+
///
2727
{% endif %}
28-
{% if method.type == 'webAuth' %}Future{% elseif method.type == 'location' %} Future<Uint8List> {% else %} {% if method.responseModel and method.responseModel != 'any' %}Future<models.{{method.responseModel | caseUcfirst | overrideIdentifier}}>{% else %}Future{% endif %}{% endif %} {{ method.name | caseCamel }}({{ _self.method_parameters(method.parameters.all | filter((param) => not param.isGlobal), method.consumes) }}) async {
28+
{% if method.type == 'webAuth' %}Future{% elseif method.type == 'location' %}Future<Uint8List>{% else %}{% if method.responseModel and method.responseModel != 'any' %}Future<models.{{method.responseModel | caseUcfirst | overrideIdentifier}}>{% else %}Future{% endif %}{% endif %} {{ method.name | caseCamel }}({{ _self.method_parameters(method.parameters.all | filter((param) => not param.isGlobal), method.consumes) }}) async {
2929
{% if method.parameters.path | length > 0 %}final{% else %}const{% endif %} String path = '{{ method.path }}'{% for parameter in method.parameters.path %}.replaceAll('{{ '{' }}{{ parameter.name | caseCamel }}{{ '}' }}', {{ parameter.name | caseCamel | overrideIdentifier }}){% endfor %};
3030

3131
{% if 'multipart/form-data' in method.consumes %}
@@ -37,7 +37,6 @@ class {{ service.name | caseUcfirst }} extends Service {
3737
{% else %}
3838
{{ include('flutter/base/requests/api.twig') }}
3939
{% endif %}
40-
4140
}
4241
{% endfor %}
4342
}

0 commit comments

Comments
 (0)