fix generation of jaguar api with auth (OpenAPITools#1009)

jaumard · Sep 21, 2018 · 5c6dade · 5c6dade
1 parent 4dde500
commit 5c6dade
Show file tree

Hide file tree

Showing 14 changed files with 71 additions and 71 deletions.
diff --git a/modules/openapi-generator/src/main/resources/dart-jaguar/README.mustache b/modules/openapi-generator/src/main/resources/dart-jaguar/README.mustache
@@ -3,7 +3,7 @@
 {{{appDescription}}}
 {{/appDescription}}
 
-This Dart package is automatically generated by the [Swagger Codegen](https://github.com/swagger-api/swagger-codegen) project:
+This Dart package is automatically generated by the [Open API Codegen](https://github.com/OpenAPITools/openapi-generator) project:
 
 - API version: {{appVersion}}
 {{#artifactVersion}}
@@ -82,8 +82,8 @@ import 'package:{{pubName}}/api.dart';
 {{/authMethods}}
 {{/hasAuthMethods}}
 
-final swaggerGen = SwaggerGen();
-var api_instance = swaggerGen.get{{classname}}();
+final jaguarApiGen = JaguarApiGen();
+var api_instance = jaguarApiGen.get{{classname}}();
 {{#allParams}}
 var {{paramName}} = {{#isListContainer}}[{{/isListContainer}}{{#isBodyParam}}new {{dataType}}(){{/isBodyParam}}{{^isBodyParam}}{{{example}}}{{/isBodyParam}}{{#isListContainer}}]{{/isListContainer}}; // {{{dataType}}} | {{{description}}}
 {{/allParams}}

diff --git a/modules/openapi-generator/src/main/resources/dart-jaguar/api.mustache b/modules/openapi-generator/src/main/resources/dart-jaguar/api.mustache
@@ -25,7 +25,7 @@ class {{classname}} extends _${{classname}}Client implements ApiClient {
     @{{httpMethod}}Req(path: "{{path}}"{{#hasAuthMethods}}, metadata: {"auth": [{{#authMethods}} {"type": "{{type}}", "name": "{{name}}"{{#isApiKey}}, "keyName": "{{keyParamName}}", "where": "{{#isKeyInQuery}}query{{/isKeyInQuery}}{{#isKeyInHeader}}header{{/isKeyInHeader}}"{{/isApiKey}} }{{#hasMore}}, {{/hasMore}}{{/authMethods}}]}{{/hasAuthMethods}})
     Future<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}}> {{nickname}}(
     {{#pathParams}}
-        {{dataType}} {{paramName}}{{#hasMore}}, {{/hasMore}}
+            @PathParam("{{baseName}}") {{dataType}} {{paramName}}{{#hasMore}}, {{/hasMore}}
     {{/pathParams}}
     {{#headerParams}}
         {{#-first}}{{#hasPathParams}},{{/hasPathParams}}{{/-first}}

diff --git a/modules/openapi-generator/src/main/resources/dart-jaguar/apilib.mustache b/modules/openapi-generator/src/main/resources/dart-jaguar/apilib.mustache
@@ -18,16 +18,16 @@ final jsonJaguarRepo = JsonRepo()
 
 final _defaultInterceptors = [OAuthInterceptor(), BasicAuthInterceptor(), ApiKeyAuthInterceptor()];
 
-class SwaggerGen {
+class JaguarApiGen {
     List<Interceptor> interceptors;
-    String baseSwaggerPath = "{{basePath}}";
+    String basePath = "{{basePath}}";
     Route _baseRoute;
 
     /**
-     * Add custom global interceptors, put overrideInterceptors to true to set your interceptors only (auth interceptors will not be added)
-     */
-    SwaggerGen({List<Interceptor> interceptors, bool overrideInterceptors = false, String baseUrl}) {
-        _baseRoute = Route(baseUrl ?? baseSwaggerPath).withClient(globalClient ?? IOClient());
+    * Add custom global interceptors, put overrideInterceptors to true to set your interceptors only (auth interceptors will not be added)
+    */
+    JaguarApiGen({List<Interceptor> interceptors, bool overrideInterceptors = false, String baseUrl}) {
+        _baseRoute = Route(baseUrl ?? basePath).withClient(globalClient ?? IOClient());
         if(interceptors == null) {
             this.interceptors = _defaultInterceptors;
         }
@@ -58,9 +58,9 @@ class SwaggerGen {
 
     {{#apiInfo}}{{#apis}}
     /**
-     * Get {{classname}} instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get {{classname}} instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     {{classname}} get{{classname}}({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -72,4 +72,4 @@ class SwaggerGen {
     }
 
     {{/apis}}{{/apiInfo}}
-}
+}
diff --git a/modules/openapi-generator/src/main/resources/dart-jaguar/pubspec.mustache b/modules/openapi-generator/src/main/resources/dart-jaguar/pubspec.mustache
@@ -4,9 +4,9 @@ description: {{pubDescription}}
 environment:
     sdk: ">=2.0.0 <3.0.0"
 dependencies:
-    jaguar_retrofit: '^2.4.1'
-    jaguar_serializer: '^2.2.0'
+    jaguar_retrofit: '^2.5.4'
+    jaguar_serializer: '^2.2.2'
 dev_dependencies:
-    jaguar_retrofit_gen: '^2.4.2'
+    jaguar_retrofit_gen: '^2.5.1'
     jaguar_serializer_cli: '^2.2.1'
     build_runner: '^0.10.0'
diff --git a/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api.dart b/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api.dart
@@ -30,15 +30,15 @@ final jsonJaguarRepo = JsonRepo()
 
 final _defaultInterceptors = [OAuthInterceptor(), BasicAuthInterceptor(), ApiKeyAuthInterceptor()];
 
-class SwaggerGen {
+class JaguarApiGen {
     List<Interceptor> interceptors;
     String baseSwaggerPath = "http://petstore.swagger.io/v2";
     Route _baseRoute;
 
     /**
-     * Add custom global interceptors, put overrideInterceptors to true to set your interceptors only (auth interceptors will not be added)
-     */
-    SwaggerGen({List<Interceptor> interceptors, bool overrideInterceptors = false, String baseUrl}) {
+    * Add custom global interceptors, put overrideInterceptors to true to set your interceptors only (auth interceptors will not be added)
+    */
+    JaguarApiGen({List<Interceptor> interceptors, bool overrideInterceptors = false, String baseUrl}) {
         _baseRoute = Route(baseUrl ?? baseSwaggerPath).withClient(globalClient ?? IOClient());
         if(interceptors == null) {
             this.interceptors = _defaultInterceptors;
@@ -70,9 +70,9 @@ class SwaggerGen {
 
 
     /**
-     * Get PetApi instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get PetApi instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     PetApi getPetApi({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -85,9 +85,9 @@ class SwaggerGen {
 
 
     /**
-     * Get StoreApi instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get StoreApi instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     StoreApi getStoreApi({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -100,9 +100,9 @@ class SwaggerGen {
 
 
     /**
-     * Get UserApi instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get UserApi instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     UserApi getUserApi({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -114,4 +114,4 @@ class SwaggerGen {
     }
 
 
-}
+}
diff --git a/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api/pet_api.dart b/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api/pet_api.dart
@@ -31,7 +31,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// 
     @DeleteReq(path: "/pet/:petId", metadata: {"auth": [ {"type": "oauth2", "name": "petstore_auth" }]})
     Future<void> deletePet(
-        int petId
+            @PathParam("petId") int petId
         ,
         @Header("api_key") String apiKey
     );
@@ -59,7 +59,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// Returns a single pet
     @GetReq(path: "/pet/:petId", metadata: {"auth": [ {"type": "apiKey", "name": "api_key", "keyName": "api_key", "where": "header" }]})
     Future<Pet> getPetById(
-        int petId
+            @PathParam("petId") int petId
     );
 
     /// Update an existing pet
@@ -76,7 +76,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// 
     @PostReq(path: "/pet/:petId", metadata: {"auth": [ {"type": "oauth2", "name": "petstore_auth" }]})
     Future<void> updatePetWithForm(
-        int petId
+            @PathParam("petId") int petId
         ,
         @AsFormField() String name, 
 
@@ -88,7 +88,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// 
     @PostReq(path: "/pet/:petId/uploadImage", metadata: {"auth": [ {"type": "oauth2", "name": "petstore_auth" }]})
     Future<ApiResponse> uploadFile(
-        int petId
+            @PathParam("petId") int petId
         ,
         @AsMultipartField() String additionalMetadata, 
 

diff --git a/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api/store_api.dart b/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api/store_api.dart
@@ -21,7 +21,7 @@ class StoreApi extends _$StoreApiClient implements ApiClient {
     /// For valid response try integer IDs with value &lt; 1000. Anything above 1000 or nonintegers will generate API errors
     @DeleteReq(path: "/store/order/:orderId")
     Future<void> deleteOrder(
-        String orderId
+            @PathParam("orderId") String orderId
     );
 
     /// Returns pet inventories by status
@@ -36,7 +36,7 @@ class StoreApi extends _$StoreApiClient implements ApiClient {
     /// For valid response try integer IDs with value &lt;&#x3D; 5 or &gt; 10. Other values will generated exceptions
     @GetReq(path: "/store/order/:orderId")
     Future<Order> getOrderById(
-        int orderId
+            @PathParam("orderId") int orderId
     );
 
     /// Place an order for a pet

diff --git a/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api/user_api.dart b/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/lib/api/user_api.dart
@@ -48,15 +48,15 @@ class UserApi extends _$UserApiClient implements ApiClient {
     /// This can only be done by the logged in user.
     @DeleteReq(path: "/user/:username")
     Future<void> deleteUser(
-        String username
+            @PathParam("username") String username
     );
 
     /// Get user by user name
     ///
     /// 
     @GetReq(path: "/user/:username")
     Future<User> getUserByName(
-        String username
+            @PathParam("username") String username
     );
 
     /// Logs user into the system
@@ -82,7 +82,7 @@ class UserApi extends _$UserApiClient implements ApiClient {
     /// This can only be done by the logged in user.
     @PutReq(path: "/user/:username")
     Future<void> updateUser(
-        String username
+            @PathParam("username") String username
         ,
         @AsJson() User user
     );

diff --git a/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/pubspec.yaml b/samples/client/petstore/dart-jaguar/flutter_petstore/openapi/pubspec.yaml
@@ -4,9 +4,9 @@ description: OpenAPI API client
 environment:
     sdk: ">=2.0.0 <3.0.0"
 dependencies:
-    jaguar_retrofit: '^2.4.1'
-    jaguar_serializer: '^2.2.0'
+    jaguar_retrofit: '^2.5.4'
+    jaguar_serializer: '^2.2.2'
 dev_dependencies:
-    jaguar_retrofit_gen: '^2.4.2'
+    jaguar_retrofit_gen: '^2.5.1'
     jaguar_serializer_cli: '^2.2.1'
     build_runner: '^0.10.0'
diff --git a/samples/client/petstore/dart-jaguar/openapi/lib/api.dart b/samples/client/petstore/dart-jaguar/openapi/lib/api.dart
@@ -30,16 +30,16 @@ final jsonJaguarRepo = JsonRepo()
 
 final _defaultInterceptors = [OAuthInterceptor(), BasicAuthInterceptor(), ApiKeyAuthInterceptor()];
 
-class SwaggerGen {
+class JaguarApiGen {
     List<Interceptor> interceptors;
-    String baseSwaggerPath = "http://petstore.swagger.io/v2";
+    String basePath = "http://petstore.swagger.io/v2";
     Route _baseRoute;
 
     /**
-     * Add custom global interceptors, put overrideInterceptors to true to set your interceptors only (auth interceptors will not be added)
-     */
-    SwaggerGen({List<Interceptor> interceptors, bool overrideInterceptors = false, String baseUrl}) {
-        _baseRoute = Route(baseUrl ?? baseSwaggerPath).withClient(globalClient ?? IOClient());
+    * Add custom global interceptors, put overrideInterceptors to true to set your interceptors only (auth interceptors will not be added)
+    */
+    JaguarApiGen({List<Interceptor> interceptors, bool overrideInterceptors = false, String baseUrl}) {
+        _baseRoute = Route(baseUrl ?? basePath).withClient(globalClient ?? IOClient());
         if(interceptors == null) {
             this.interceptors = _defaultInterceptors;
         }
@@ -70,9 +70,9 @@ class SwaggerGen {
 
 
     /**
-     * Get PetApi instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get PetApi instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     PetApi getPetApi({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -85,9 +85,9 @@ class SwaggerGen {
 
 
     /**
-     * Get StoreApi instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get StoreApi instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     StoreApi getStoreApi({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -100,9 +100,9 @@ class SwaggerGen {
 
 
     /**
-     * Get UserApi instance, base route and serializer can be overridden by a given but be careful,
-     * by doing that all interceptors will not be executed
-     */
+    * Get UserApi instance, base route and serializer can be overridden by a given but be careful,
+    * by doing that all interceptors will not be executed
+    */
     UserApi getUserApi({Route base, SerializerRepo serializers}) {
         if(base == null) {
             base = _baseRoute;
@@ -114,4 +114,4 @@ class SwaggerGen {
     }
 
 
-}
+}
diff --git a/samples/client/petstore/dart-jaguar/openapi/lib/api/pet_api.dart b/samples/client/petstore/dart-jaguar/openapi/lib/api/pet_api.dart
@@ -31,7 +31,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// 
     @DeleteReq(path: "/pet/:petId", metadata: {"auth": [ {"type": "oauth2", "name": "petstore_auth" }]})
     Future<void> deletePet(
-        int petId
+            @PathParam("petId") int petId
         ,
         @Header("api_key") String apiKey
     );
@@ -59,7 +59,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// Returns a single pet
     @GetReq(path: "/pet/:petId", metadata: {"auth": [ {"type": "apiKey", "name": "api_key", "keyName": "api_key", "where": "header" }]})
     Future<Pet> getPetById(
-        int petId
+            @PathParam("petId") int petId
     );
 
     /// Update an existing pet
@@ -76,7 +76,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// 
     @PostReq(path: "/pet/:petId", metadata: {"auth": [ {"type": "oauth2", "name": "petstore_auth" }]})
     Future<void> updatePetWithForm(
-        int petId
+            @PathParam("petId") int petId
         ,
         @AsFormField() String name, 
 
@@ -88,7 +88,7 @@ class PetApi extends _$PetApiClient implements ApiClient {
     /// 
     @PostReq(path: "/pet/:petId/uploadImage", metadata: {"auth": [ {"type": "oauth2", "name": "petstore_auth" }]})
     Future<ApiResponse> uploadFile(
-        int petId
+            @PathParam("petId") int petId
         ,
         @AsMultipartField() String additionalMetadata, 
 

diff --git a/samples/client/petstore/dart-jaguar/openapi/lib/api/store_api.dart b/samples/client/petstore/dart-jaguar/openapi/lib/api/store_api.dart
@@ -21,7 +21,7 @@ class StoreApi extends _$StoreApiClient implements ApiClient {
     /// For valid response try integer IDs with value &lt; 1000. Anything above 1000 or nonintegers will generate API errors
     @DeleteReq(path: "/store/order/:orderId")
     Future<void> deleteOrder(
-        String orderId
+            @PathParam("orderId") String orderId
     );
 
     /// Returns pet inventories by status
@@ -36,7 +36,7 @@ class StoreApi extends _$StoreApiClient implements ApiClient {
     /// For valid response try integer IDs with value &lt;&#x3D; 5 or &gt; 10. Other values will generated exceptions
     @GetReq(path: "/store/order/:orderId")
     Future<Order> getOrderById(
-        int orderId
+            @PathParam("orderId") int orderId
     );
 
     /// Place an order for a pet

diff --git a/samples/client/petstore/dart-jaguar/openapi/lib/api/user_api.dart b/samples/client/petstore/dart-jaguar/openapi/lib/api/user_api.dart
@@ -48,15 +48,15 @@ class UserApi extends _$UserApiClient implements ApiClient {
     /// This can only be done by the logged in user.
     @DeleteReq(path: "/user/:username")
     Future<void> deleteUser(
-        String username
+            @PathParam("username") String username
     );
 
     /// Get user by user name
     ///
     /// 
     @GetReq(path: "/user/:username")
     Future<User> getUserByName(
-        String username
+            @PathParam("username") String username
     );
 
     /// Logs user into the system
@@ -82,7 +82,7 @@ class UserApi extends _$UserApiClient implements ApiClient {
     /// This can only be done by the logged in user.
     @PutReq(path: "/user/:username")
     Future<void> updateUser(
-        String username
+            @PathParam("username") String username
         ,
         @AsJson() User user
     );