Modify compliance DAO (#688)

This commit is contained in:
Yi (Alan) Wang 2017-08-20 17:32:23 -07:00 committed by GitHub
parent 0ba5315a6c
commit 47b1799a0c
2 changed files with 6 additions and 9 deletions

View File

@ -14,7 +14,6 @@
package wherehows.dao; package wherehows.dao;
import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.core.type.TypeReference;
import com.fasterxml.jackson.databind.JsonNode;
import com.fasterxml.jackson.databind.ObjectMapper; import com.fasterxml.jackson.databind.ObjectMapper;
import java.io.IOException; import java.io.IOException;
import java.sql.PreparedStatement; import java.sql.PreparedStatement;
@ -225,12 +224,11 @@ public class DatasetsDao {
return record; return record;
} }
public DatasetCompliance updateDatasetComplianceInfo(JdbcTemplate jdbcTemplate, public void updateDatasetComplianceInfo(JdbcTemplate jdbcTemplate,
NamedParameterJdbcTemplate namedParameterJdbcTemplate, int datasetId, JsonNode node, String user) NamedParameterJdbcTemplate namedParameterJdbcTemplate, int datasetId, DatasetCompliance record, String user)
throws Exception { throws Exception {
ObjectMapper om = new ObjectMapper(); ObjectMapper om = new ObjectMapper();
DatasetCompliance record = om.convertValue(node, DatasetCompliance.class);
if (record.getDatasetId() != null && datasetId != record.getDatasetId()) { if (record.getDatasetId() != null && datasetId != record.getDatasetId()) {
throw new IllegalArgumentException("Dataset id doesn't match."); throw new IllegalArgumentException("Dataset id doesn't match.");
} }
@ -253,7 +251,5 @@ public class DatasetsDao {
parameters.put("modified_by", user); parameters.put("modified_by", user);
parameters.put("modified_time", System.currentTimeMillis() / 1000); parameters.put("modified_time", System.currentTimeMillis() / 1000);
namedParameterJdbcTemplate.update(INSERT_DATASET_COMPLIANCE, parameters); namedParameterJdbcTemplate.update(INSERT_DATASET_COMPLIANCE, parameters);
return record;
} }
} }

View File

@ -777,8 +777,9 @@ public class Dataset extends Controller {
} }
try { try {
DATASETS_DAO.updateDatasetComplianceInfo(JDBC_TEMPLATE, NAMED_JDBC_TEMPLATE, datasetId, request().body().asJson(), DatasetCompliance record = Json.mapper().convertValue(request().body().asJson(), DatasetCompliance.class);
username);
DATASETS_DAO.updateDatasetComplianceInfo(JDBC_TEMPLATE, NAMED_JDBC_TEMPLATE, datasetId, record, username);
} catch (Exception e) { } catch (Exception e) {
JsonNode result = Json.newObject() JsonNode result = Json.newObject()
.put("status", "failed") .put("status", "failed")