(improvement)(Headless) fix JSON error (#841)

This commit is contained in:
lexluo09
2024-03-20 21:41:02 +08:00
committed by GitHub
parent bb6ee3ae83
commit 416542e6dc
5 changed files with 8 additions and 13 deletions

View File

@@ -14,7 +14,7 @@ public class DataSetModelConfig {
private Long id; private Long id;
private boolean includesAll; private Boolean includesAll = false;
private List<Long> metrics = Lists.newArrayList(); private List<Long> metrics = Lists.newArrayList();

View File

@@ -1,20 +1,16 @@
package com.tencent.supersonic.headless.api.pojo.request; package com.tencent.supersonic.headless.api.pojo.request;
import com.tencent.supersonic.common.pojo.DateConf; import com.tencent.supersonic.common.pojo.DateConf;
import com.tencent.supersonic.headless.api.pojo.SchemaElementType; import javax.validation.constraints.NotNull;
import lombok.Data; import lombok.Data;
import lombok.ToString; import lombok.ToString;
import javax.validation.constraints.NotNull;
@Data @Data
@ToString @ToString
public class ItemValueReq { public class ItemValueReq {
private SchemaElementType type;
@NotNull @NotNull
private Long itemId; private Long id;
private DateConf dateConf; private DateConf dateConf;

View File

@@ -49,7 +49,7 @@ public class DataSetResp extends SchemaItem {
} }
public List<Long> getAllIncludeAllModels() { public List<Long> getAllIncludeAllModels() {
return getDataSetModelConfigs().stream().filter(DataSetModelConfig::isIncludesAll) return getDataSetModelConfigs().stream().filter(DataSetModelConfig::getIncludesAll)
.map(DataSetModelConfig::getId) .map(DataSetModelConfig::getId)
.collect(Collectors.toList()); .collect(Collectors.toList());
} }

View File

@@ -56,9 +56,9 @@ public class TagQueryServiceImpl implements TagQueryService {
@Override @Override
public ItemValueResp queryTagValue(ItemValueReq itemValueReq, User user) throws Exception { public ItemValueResp queryTagValue(ItemValueReq itemValueReq, User user) throws Exception {
ItemValueResp itemValueResp = new ItemValueResp(); ItemValueResp itemValueResp = new ItemValueResp();
itemValueResp.setItemId(itemValueReq.getItemId()); itemValueResp.setItemId(itemValueReq.getId());
itemValueResp.setType(SchemaElementType.TAG); itemValueResp.setType(SchemaElementType.TAG);
TagResp tag = tagMetaService.getTag(itemValueReq.getItemId(), user); TagResp tag = tagMetaService.getTag(itemValueReq.getId(), user);
if (Objects.isNull(tag)) { if (Objects.isNull(tag)) {
return null; return null;
} }
@@ -88,6 +88,7 @@ public class TagQueryServiceImpl implements TagQueryService {
ModelResp model = modelService.getModel(tag.getModelId()); ModelResp model = modelService.getModel(tag.getModelId());
List<Dim> timeDimension = model.getTimeDimension(); List<Dim> timeDimension = model.getTimeDimension();
if (CollectionUtils.isEmpty(timeDimension)) { if (CollectionUtils.isEmpty(timeDimension)) {
itemValueReq.setDateConf(null);
return; return;
} }

View File

@@ -3,7 +3,6 @@ package com.tencent.supersonic.headless;
import static org.junit.Assert.assertTrue; import static org.junit.Assert.assertTrue;
import com.tencent.supersonic.auth.api.authentication.pojo.User; import com.tencent.supersonic.auth.api.authentication.pojo.User;
import com.tencent.supersonic.headless.api.pojo.SchemaElementType;
import com.tencent.supersonic.headless.api.pojo.request.ItemValueReq; import com.tencent.supersonic.headless.api.pojo.request.ItemValueReq;
import com.tencent.supersonic.headless.api.pojo.response.ItemValueResp; import com.tencent.supersonic.headless.api.pojo.response.ItemValueResp;
import com.tencent.supersonic.headless.server.service.TagQueryService; import com.tencent.supersonic.headless.server.service.TagQueryService;
@@ -18,8 +17,7 @@ public class QueryTagValueTest extends BaseTest {
@Test @Test
public void testQueryTagValue() throws Exception { public void testQueryTagValue() throws Exception {
ItemValueReq itemValueReq = new ItemValueReq(); ItemValueReq itemValueReq = new ItemValueReq();
itemValueReq.setItemId(1L); itemValueReq.setId(1L);
itemValueReq.setType(SchemaElementType.DIMENSION);
ItemValueResp itemValueResp = tagQueryService.queryTagValue(itemValueReq, User.getFakeUser()); ItemValueResp itemValueResp = tagQueryService.queryTagValue(itemValueReq, User.getFakeUser());
assertTrue(itemValueResp != null); assertTrue(itemValueResp != null);
} }