From 4f0b20c4680c5197a25fee211a3a09ee71bcddcf Mon Sep 17 00:00:00 2001 From: cptran777 Date: Fri, 31 Aug 2018 21:07:50 -0700 Subject: [PATCH] Refactor to return request body as response if success for export policy --- wherehows-frontend/app/controllers/api/v2/Dataset.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/wherehows-frontend/app/controllers/api/v2/Dataset.java b/wherehows-frontend/app/controllers/api/v2/Dataset.java index e51dc00f79..c571ac2710 100644 --- a/wherehows-frontend/app/controllers/api/v2/Dataset.java +++ b/wherehows-frontend/app/controllers/api/v2/Dataset.java @@ -272,14 +272,14 @@ public class Dataset extends Controller { public static Promise updateExportPolicy(String datasetUrn) { final String username = session("user"); - final DatasetExportPolicy exportPolicy; + JsonNode requestBody = request().body().asJson(); if (StringUtils.isBlank(username)) { return Promise.promise(() -> unauthorized(_EMPTY_RESPONSE)); } try { - exportPolicy = Json.mapper().convertValue(request().body().asJson(), DatasetExportPolicy.class); + final DatasetExportPolicy exportPolicy = Json.mapper().convertValue(requestBody, DatasetExportPolicy.class); EXPORT_POLICY_DAO.updateDatasetExportPolicy(datasetUrn, exportPolicy, username); } catch (Exception e) { @@ -287,7 +287,7 @@ public class Dataset extends Controller { return Promise.promise(() -> internalServerError(errorResponse(e))); } - return Promise.promise(() -> ok(Json.newObject().set("exportPolicy", Json.toJson(exportPolicy)))); + return Promise.promise(() -> ok(Json.newObject().set("exportPolicy", Json.toJson(requestBody)))); } public static Promise getDatasetOwners(String datasetUrn) {