From af415faa84ba48fe3f3490e219c92da499915f69 Mon Sep 17 00:00:00 2001 From: Lari Taskula Date: Tue, 14 Jun 2016 12:53:58 +0300 Subject: [PATCH] [SIGNED-OFF] Bug 16699: Support multiple types in primitive definitions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently it is not possible to define multiple types for primitive definitions in /definitions/*. If you try to use the following "firstname": { "type": ["string", "null"], "description": "patron's first name" } in definitions.json, online.swagger.io validator will not validate it: {"messages":["attribute definitions.firstname.type is not of type `string`"]} One way to get around this issue is to extend definitions with custom "x-primitives" object, where we will define all reusable primitive definitions. To test: 1. Add the "firstname" example above to definitions.json 2. Run minifySwagger.pl 3. Validate your specification 4. Observe that error with description mentioned above is given 5. Apply patch 6. Repeat step 2 and 3 7. Observe that validation passes Signed-off-by: Olli-Antti Kivilahti My name is Olli-Antti Kivilahti and I approve this commit. We have been using the Swagger2.0-driven REST API on Mojolicious for 1 year now in production and I am certain we have a pretty good idea on how to work with the limitations of Swagger2.0 We participated in the development of the Mojolicious::Plugin::Swagger and know it well. We have made an extension to the plugin to provide full CORS support and have been building all our in-house features on the new REST API. Signed-off-by: Johanna Raisa My name is Johanna Räisä and I approve this commit. We have been using Swagger2.0-driven REST API in production successfully. Signed-off-by: Benjamin Rokseth --- api/v1/swagger.json | 3 +++ api/v1/x-primitives.json | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 44 insertions(+) create mode 100644 api/v1/x-primitives.json diff --git a/api/v1/swagger.json b/api/v1/swagger.json index 362780a..249e71b 100644 --- a/api/v1/swagger.json +++ b/api/v1/swagger.json @@ -21,5 +21,8 @@ }, "parameters": { "$ref": "parameters.json" + }, + "x-primitives": { + "$ref": "x-primitives.json" } } diff --git a/api/v1/x-primitives.json b/api/v1/x-primitives.json new file mode 100644 index 0000000..ae8750e --- /dev/null +++ b/api/v1/x-primitives.json @@ -0,0 +1,41 @@ +{ + "biblionumber": { + "type": "string", + "description": "internally assigned biblio identifier" + }, + "borrowernumber": { + "type": "string", + "description": "internally assigned user identifier" + }, + "branchcode": { + "type": ["string", "null"], + "description": "code of patron's home branch" + }, + "cardnumber": { + "type": ["string", "null"], + "description": "library assigned user identifier" + }, + "email": { + "type": ["string", "null"], + "description": "primary email address for patron's primary address" + }, + "firstname": { + "type": ["string", "null"], + "description": "patron's first name" + }, + "itemnumber": { + "type": ["string", "null"], + "description": "internally assigned item identifier" + }, + "phone": { + "type": ["string", "null"], + "description": "primary phone number for patron's primary address" + }, + "reserve_id": { + "description": "Internal hold identifier" + }, + "surname": { + "type": "string", + "description": "patron's last name" + } +} -- 2.1.4