Merge pull request #241 from aljachimiak/regression_for_include

fixes missing include query handles in catalog item controller
oddnetworks · Oct 25, 2016 · 38bd871 · 38bd871
2 parents c09cece + 697d005
commit 38bd871
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/lib/middleware/response-json-api.js b/lib/middleware/response-json-api.js
@@ -9,6 +9,7 @@ const JSON_API_KEYS = ['id', 'type', 'relationships', 'meta', 'links'];
 
 // options.bus - Object *required*
 // options.baseUrlPrefix - String ex: "/v2".
+// options.allowPartialIncluded - Boolean (default == false)
 // options.excludePortFromLinks - Boolean (default == false) If true, don't
 //   include the port number in links.
 module.exports = function (options) {

diff --git a/lib/services/catalog/controllers/catalog-item-controller.js b/lib/services/catalog/controllers/catalog-item-controller.js
@@ -29,6 +29,10 @@ class CatalogItemController extends Controller {
 			.then(channel => {
 				const args = {type, id};
 
+				if (req.query.include) {
+					args.include = req.query.include.split(',');
+				}
+
 				// When no platform ID is present in the JWT during an admin role request,
 				// the query for the resource is made directly to the store instead of through
 				// the catalog service.