fixed report issues

atms-api/src/main/java/pwc/taxtech/atms/controller/CellTemplateController.java

@@ -19,7 +19,7 @@ public class CellTemplateController extends BaseController {
     @Autowired
     CellTemplateService cellTemplateService;
 
-    @RequestMapping(value = "configList/{templateID}", method = RequestMethod.GET, produces = MediaType.APPLICATION_JSON_VALUE)
+    @RequestMapping(value = "configList/{templateID}", method = RequestMethod.GET, produces = MediaType.APPLICATION_JSON_UTF8_VALUE)
     public @ResponseBody
     OperationResultDto<List<CellTemplateConfigDto>> getConfigList(@PathVariable Long templateID) {
         OperationResultDto<List<CellTemplateConfigDto>> result = new OperationResultDto<>();

atms-api/src/main/java/pwc/taxtech/atms/dto/CellTemplateConfigDto.java

@@ -4,8 +4,8 @@ import java.util.List;
 
 public class CellTemplateConfigDto {
 
-    private Long cellTemplateID;
-    private Long templateID;
+    private String cellTemplateID;
+    private String templateID;
     private int rowIndex;
     private String rowName;
     private int columnIndex;
@@ -36,19 +36,19 @@ public class CellTemplateConfigDto {
     private String voucherKeyword;
 
 
-    public Long getCellTemplateID() {
+    public String getCellTemplateID() {
         return cellTemplateID;
     }
 
-    public void setCellTemplateID(Long cellTemplateID) {
+    public void setCellTemplateID(String cellTemplateID) {
         this.cellTemplateID = cellTemplateID;
     }
 
-    public Long getTemplateID() {
+    public String getTemplateID() {
         return templateID;
     }
 
-    public void setTemplateID(Long templateID) {
+    public void setTemplateID(String templateID) {
         this.templateID = templateID;
     }
 

atms-api/src/main/java/pwc/taxtech/atms/service/impl/CellConfigTranslater.java

@@ -40,8 +40,8 @@ public final class CellConfigTranslater {
         }
 
         CellTemplateConfigDto cellTemplateConfigDto = new CellTemplateConfigDto();
-        cellTemplateConfigDto.setCellTemplateID(configID);
-        cellTemplateConfigDto.setTemplateID(templateID);
+        cellTemplateConfigDto.setCellTemplateID(configID.toString());
+        cellTemplateConfigDto.setTemplateID(templateID.toString());
         cellTemplateConfigDto.setRowIndex(rowIndex);
         cellTemplateConfigDto.setRowName(rowName);
         cellTemplateConfigDto.setColumnIndex(columnIndex);
@@ -59,7 +59,7 @@ public final class CellConfigTranslater {
                 cellTemplateConfigDto.setFormulaDescription(formulaItem.get().getFormulaDescription());
             }
             cellTemplateConfigDto.setFormulaDataSource(formulaItem.get().getFormulaDataSource());
-            cellTemplateConfigDto.setCellTemplateID(formulaItem.get().getCellTemplateId());
+            cellTemplateConfigDto.setCellTemplateID(formulaItem.get().getCellTemplateId().toString());
         }
 
         Optional<CellTemplateConfig> voucherItem = configList.stream().filter(x -> x.getDataSourceType().equals(CellDataSourceType.Voucher.getCode())).findFirst();
@@ -110,7 +110,7 @@ public final class CellConfigTranslater {
             cellTemplateConfigDto.setHasValidation(true);
             cellTemplateConfigDto.setValidation(validationItem.get().getValidation());
             cellTemplateConfigDto.setValidationDescription(validationItem.get().getValidationDescription());
-            cellTemplateConfigDto.setCellTemplateID(validationItem.get().getCellTemplateId());
+            cellTemplateConfigDto.setCellTemplateID(validationItem.get().getCellTemplateId().toString());
         }
         return cellTemplateConfigDto;
     }
@@ -125,8 +125,8 @@ public final class CellConfigTranslater {
         }
 
         CellTemplateConfigDto cellTemplateConfigDto = new CellTemplateConfigDto();
-        cellTemplateConfigDto.setCellTemplateID(configID);
-        cellTemplateConfigDto.setTemplateID(templateID);
+        cellTemplateConfigDto.setCellTemplateID(configID.toString());
+        cellTemplateConfigDto.setTemplateID(templateID.toString());
         cellTemplateConfigDto.setRowIndex(rowIndex);
         cellTemplateConfigDto.setRowName(rowName);
         cellTemplateConfigDto.setColumnIndex(columnIndex);
@@ -148,7 +148,7 @@ public final class CellConfigTranslater {
                 cellTemplateConfigDto.setFormulaDescription(formulaItem.get().getFormulaDescription());
             }
             cellTemplateConfigDto.setFormulaDataSource(formulaItem.get().getFormulaDataSource());
-            cellTemplateConfigDto.setCellTemplateID(formulaItem.get().getCellTemplateId());
+            cellTemplateConfigDto.setCellTemplateID(formulaItem.get().getCellTemplateId().toString());
         }
 
         Optional<PeriodCellTemplateConfig> voucherItem = configList.stream()
@@ -216,7 +216,7 @@ public final class CellConfigTranslater {
             cellTemplateConfigDto.setValidation(validationItem.get().getValidation());
             cellTemplateConfigDto.setParsedFormula(validationItem.get().getParsedFormula());
             cellTemplateConfigDto.setValidationDescription(validationItem.get().getValidationDescription());
-            cellTemplateConfigDto.setCellTemplateID(validationItem.get().getCellTemplateId());
+            cellTemplateConfigDto.setCellTemplateID(validationItem.get().getCellTemplateId().toString());
         }
 
         return cellTemplateConfigDto;

atms-api/src/main/java/pwc/taxtech/atms/service/impl/CellTemplateServiceImpl.java

@@ -80,7 +80,7 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
     @Transactional(propagation = Propagation.REQUIRED, rollbackFor = Exception.class)
     public OperationResultDto saveOrEdit(CellTemplateConfigDto cellTemplateConfig) {
         OperationResultDto result = new OperationResultDto();
-        CellTemplate cellTemplate = cellTemplateMapper.selectByPrimaryKey(cellTemplateConfig.getCellTemplateID());
+        CellTemplate cellTemplate = cellTemplateMapper.selectByPrimaryKey(Long.parseLong(cellTemplateConfig.getCellTemplateID()));
         if (cellTemplate == null) {
             result.setResultMsg("Nodata");
             return result;
@@ -89,7 +89,7 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
         cellTemplate.setRowName(cellTemplateConfig.getRowName());
         cellTemplate.setColumnName(cellTemplateConfig.getColumnName());
         CellTemplateConfigExample example = new CellTemplateConfigExample();
-        example.createCriteria().andCellTemplateIdEqualTo(cellTemplateConfig.getCellTemplateID());
+        example.createCriteria().andCellTemplateIdEqualTo(Long.parseLong(cellTemplateConfig.getCellTemplateID()));
         List<CellTemplateConfig> cellTemplateConfigs = cellTemplateConfigMapper.selectByExample(example);
 
         if (!cellTemplateConfigs.isEmpty()) {
@@ -127,8 +127,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
             operationResultDto.setResultMsg(getFormulaDataSource(cellTemplateConfigDto.getFormula(), keyValueIds));
             CellTemplateConfig cellTemplateConfig = new CellTemplateConfig();
             cellTemplateConfig.setId(distributedIDService.nextId());
-            cellTemplateConfig.setCellTemplateId(cellTemplateConfigDto.getCellTemplateID());
-            cellTemplateConfig.setReportTemplateId(cellTemplateConfigDto.getTemplateID());
+            cellTemplateConfig.setCellTemplateId(Long.parseLong(cellTemplateConfigDto.getCellTemplateID()));
+            cellTemplateConfig.setReportTemplateId(Long.parseLong(cellTemplateConfigDto.getTemplateID()));
             cellTemplateConfig.setCreateBy(cellTemplateConfigDto.getCreator());
             cellTemplateConfig.setUpdateBy(cellTemplateConfigDto.getUpdater());
             cellTemplateConfig.setUpdateTime(new Date());
@@ -143,8 +143,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
         if (cellTemplateConfigDto.getHasVoucher() != null && cellTemplateConfigDto.getHasVoucher()) {
             CellTemplateConfig cellTemplateConfig = new CellTemplateConfig();
             cellTemplateConfig.setId(distributedIDService.nextId());
-            cellTemplateConfig.setCellTemplateId(cellTemplateConfigDto.getCellTemplateID());
-            cellTemplateConfig.setReportTemplateId(cellTemplateConfigDto.getTemplateID());
+            cellTemplateConfig.setCellTemplateId(Long.parseLong(cellTemplateConfigDto.getCellTemplateID()));
+            cellTemplateConfig.setReportTemplateId(Long.parseLong(cellTemplateConfigDto.getTemplateID()));
             cellTemplateConfig.setCreateBy(cellTemplateConfigDto.getCreator());
             cellTemplateConfig.setUpdateBy(cellTemplateConfigDto.getUpdater());
             cellTemplateConfig.setUpdateTime(new Date());
@@ -158,8 +158,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
         if (cellTemplateConfigDto.getHasInvoice() != null && cellTemplateConfigDto.getHasInvoice()) {
             CellTemplateConfig cellTemplateConfig = new CellTemplateConfig();
             cellTemplateConfig.setId(distributedIDService.nextId());
-            cellTemplateConfig.setCellTemplateId(cellTemplateConfigDto.getCellTemplateID());
-            cellTemplateConfig.setReportTemplateId(cellTemplateConfigDto.getTemplateID());
+            cellTemplateConfig.setCellTemplateId(Long.parseLong(cellTemplateConfigDto.getCellTemplateID()));
+            cellTemplateConfig.setReportTemplateId(Long.parseLong(cellTemplateConfigDto.getTemplateID()));
             cellTemplateConfig.setCreateBy(cellTemplateConfigDto.getCreator());
             cellTemplateConfig.setUpdateBy(cellTemplateConfigDto.getUpdater());
             cellTemplateConfig.setUpdateTime(new Date());
@@ -175,8 +175,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
         if (cellTemplateConfigDto.getHasKeyIn() != null && cellTemplateConfigDto.getHasKeyIn()) {
             CellTemplateConfig cellTemplateConfig = new CellTemplateConfig();
             cellTemplateConfig.setId(distributedIDService.nextId());
-            cellTemplateConfig.setCellTemplateId(cellTemplateConfigDto.getCellTemplateID());
-            cellTemplateConfig.setReportTemplateId(cellTemplateConfigDto.getTemplateID());
+            cellTemplateConfig.setCellTemplateId(Long.parseLong(cellTemplateConfigDto.getCellTemplateID()));
+            cellTemplateConfig.setReportTemplateId(Long.parseLong(cellTemplateConfigDto.getTemplateID()));
             cellTemplateConfig.setCreateBy(cellTemplateConfigDto.getCreator());
             cellTemplateConfig.setUpdateBy(cellTemplateConfigDto.getUpdater());
             cellTemplateConfig.setUpdateTime(new Date());
@@ -188,8 +188,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
         if (cellTemplateConfigDto.getHasModel() != null && cellTemplateConfigDto.getHasModel()) {
             CellTemplateConfig cellTemplateConfig = new CellTemplateConfig();
             cellTemplateConfig.setId(distributedIDService.nextId());
-            cellTemplateConfig.setCellTemplateId(cellTemplateConfigDto.getCellTemplateID());
-            cellTemplateConfig.setReportTemplateId(cellTemplateConfigDto.getTemplateID());
+            cellTemplateConfig.setCellTemplateId(Long.parseLong(cellTemplateConfigDto.getCellTemplateID()));
+            cellTemplateConfig.setReportTemplateId(Long.parseLong(cellTemplateConfigDto.getTemplateID()));
             cellTemplateConfig.setCreateBy(cellTemplateConfigDto.getCreator());
             cellTemplateConfig.setUpdateBy(cellTemplateConfigDto.getUpdater());
             cellTemplateConfig.setUpdateTime(new Date());
@@ -202,8 +202,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
         if (cellTemplateConfigDto.getHasValidation() != null && cellTemplateConfigDto.getHasValidation()) {
             CellTemplateConfig cellTemplateConfig = new CellTemplateConfig();
             cellTemplateConfig.setId(distributedIDService.nextId());
-            cellTemplateConfig.setCellTemplateId(cellTemplateConfigDto.getCellTemplateID());
-            cellTemplateConfig.setReportTemplateId(cellTemplateConfigDto.getTemplateID());
+            cellTemplateConfig.setCellTemplateId(Long.parseLong(cellTemplateConfigDto.getCellTemplateID()));
+            cellTemplateConfig.setReportTemplateId(Long.parseLong(cellTemplateConfigDto.getTemplateID()));
             cellTemplateConfig.setCreateBy(cellTemplateConfigDto.getCreator());
             cellTemplateConfig.setUpdateBy(cellTemplateConfigDto.getUpdater());
             cellTemplateConfig.setUpdateTime(new Date());
@@ -293,8 +293,8 @@ public class CellTemplateServiceImpl extends AbstractService implements CellTemp
     private CellTemplateConfigDto getConfigDto(CellTemplate cellTemplate, List<CellTemplateConfig> configList) {
         CellTemplateConfigDto cellTemplateConfigDto = CellConfigTranslater.getConfigDto(cellTemplate, configList);
         if (cellTemplateConfigDto != null) {
-            cellTemplateConfigDto.setCellTemplateID(cellTemplate.getId());
-            cellTemplateConfigDto.setTemplateID(cellTemplate.getReportTemplateId());
+            cellTemplateConfigDto.setCellTemplateID(cellTemplate.getId().toString());
+            cellTemplateConfigDto.setTemplateID(cellTemplate.getReportTemplateId().toString());
             cellTemplateConfigDto.setRowIndex(cellTemplate.getRowIndex());
             cellTemplateConfigDto.setRowName(cellTemplate.getRowName());
             cellTemplateConfigDto.setColumnIndex(cellTemplate.getColumnIndex());

atms-api/src/main/java/pwc/taxtech/atms/vat/service/impl/ReportGeneratorImpl.java

@@ -240,10 +240,10 @@ public class ReportGeneratorImpl extends VatAbstractService implements ReportGen
                         }
 
                         //如果有正则匹配就进行更新公式解析
-                        if (isFind) {
-                            periodCellTemplateConfig.setParsedFormula(resultFormula);
-                            periodCellTemplateConfigMapper.updateByPrimaryKey(periodCellTemplateConfig);
-                        }
+//                        if (isFind) {
+                        periodCellTemplateConfig.setParsedFormula(resultFormula);
+                        periodCellTemplateConfigMapper.updateByPrimaryKey(periodCellTemplateConfig);
+//                        }
 
                         String regexNormalCell = "[A-Z]{1,2}((?!0)[0-9]{1,3})";
                         p = Pattern.compile(regexNormalCell);
@@ -259,9 +259,9 @@ public class ReportGeneratorImpl extends VatAbstractService implements ReportGen
                                 String colStr = mm.group();
                                 int colNum = FormulaHelper.excelColStrToNum(colStr, colStr.length());
                                 int rowNum = Integer.parseInt(StringUtils.removeStart(findStr, colStr));
-                                Row row = sheet.getRow(rowNum-1);
+                                Row row = sheet.getRow(rowNum - 1);
                                 if (row != null) {
-                                    Cell cell = row.getCell(colNum-1);
+                                    Cell cell = row.getCell(colNum - 1);
                                     if (cell != null) {
                                         //开始取值然后存放到DataSource
                                         DataSource dataSource = new DataSource();
@@ -312,11 +312,11 @@ public class ReportGeneratorImpl extends VatAbstractService implements ReportGen
                             periodFormulaBlockExample.createCriteria().andPeriodEqualTo(period)
                                     .andCellTemplateIdEqualTo(tempPeriodCellTemplate.get().getCellTemplateId());
 
-                            if (isFind) {
-                                cellData.setFormulaExp(resultFormula);
-                            } else {
-                                cellData.setFormulaExp(EMPTY);
-                            }
+//                            if (isFind) {
+                            cellData.setFormulaExp(resultFormula);
+//                            } else {
+//                                cellData.setFormulaExp(EMPTY);
+//                            }
                             cellData.setCreateBy("Admin");
                             cellData.setCreateTime(createTime);
                             cellData.setUpdateBy("Admin");
@@ -439,28 +439,28 @@ public class ReportGeneratorImpl extends VatAbstractService implements ReportGen
                 new CellTemplateConfigGroupDto(a.getColumnIndex(), a.getRowIndex()
                         , a.getColumnName(), a.getRowName(), a.getDataType(), a.getIsReadOnly()
                         , a.getCellTemplateID(), a.getReportTemplateID(), a.getComment()).toStr())).forEach((x, y) -> {
-                            Optional<Map<String,Object>> result =  x.values().stream().findFirst();
-                            if (result.isPresent()) {
-                                CellCalcInfoDto cellCalcInfoDto = new CellCalcInfoDto();
-                                cellCalcInfoDto.setColumnIndex(Integer.parseInt(result.get().get("columnIndex").toString()));
-                                cellCalcInfoDto.setColumnName(result.get().get("columnName").toString());
-                                cellCalcInfoDto.setRowIndex(Integer.parseInt(result.get().get("rowIndex").toString()));
-                                cellCalcInfoDto.setRowName(result.get().get("rowName").toString());
-                                cellCalcInfoDto.setFormula(convertListToString(y.stream()
-                                        .map(PeriodCellTemplateConfig::getFormula).collect(Collectors.toList())));
-                                cellCalcInfoDto.setParsedFormula(convertListToString(y.stream()
-                                        .map(PeriodCellTemplateConfig::getParsedFormula).collect(Collectors.toList())));
-                                //todo: add validation at here later
-                                cellCalcInfoDto.setCellTemplateID(Long.parseLong(result.get().get("cellTemplateID").toString()));
-                                cellCalcInfoDto.setReportTemplateID(Long.parseLong(result.get().get("reportTemplateID").toString()));
-                                cellCalcInfoDto.setDataType(Integer.parseInt(result.get().get("dataType").toString()));
-                                cellCalcInfoDto.setIsReadOnly(Boolean.valueOf(result.get().get("isReadOnly").toString()));
-                                cellCalcInfoDto.setComment(result.get().get("comment").toString());
-                                cellCalcInfoDto.setConfigList(y.stream()
-                                        .map(PeriodCellTemplateConfigExtendDto::getPeriodCellTemplateConfig)
-                                        .collect(Collectors.toList()));
-                                cellCalcInfoDtos.add(cellCalcInfoDto);
-                            }
+            Optional<Map<String, Object>> result = x.values().stream().findFirst();
+            if (result.isPresent()) {
+                CellCalcInfoDto cellCalcInfoDto = new CellCalcInfoDto();
+                cellCalcInfoDto.setColumnIndex(Integer.parseInt(result.get().get("columnIndex").toString()));
+                cellCalcInfoDto.setColumnName(result.get().get("columnName").toString());
+                cellCalcInfoDto.setRowIndex(Integer.parseInt(result.get().get("rowIndex").toString()));
+                cellCalcInfoDto.setRowName(result.get().get("rowName").toString());
+                cellCalcInfoDto.setFormula(convertListToString(y.stream()
+                        .map(PeriodCellTemplateConfig::getFormula).collect(Collectors.toList())));
+                cellCalcInfoDto.setParsedFormula(convertListToString(y.stream()
+                        .map(PeriodCellTemplateConfig::getParsedFormula).collect(Collectors.toList())));
+                //todo: add validation at here later
+                cellCalcInfoDto.setCellTemplateID(Long.parseLong(result.get().get("cellTemplateID").toString()));
+                cellCalcInfoDto.setReportTemplateID(Long.parseLong(result.get().get("reportTemplateID").toString()));
+                cellCalcInfoDto.setDataType(Integer.parseInt(result.get().get("dataType").toString()));
+                cellCalcInfoDto.setIsReadOnly(Boolean.valueOf(result.get().get("isReadOnly").toString()));
+                cellCalcInfoDto.setComment(result.get().get("comment").toString());
+                cellCalcInfoDto.setConfigList(y.stream()
+                        .map(PeriodCellTemplateConfigExtendDto::getPeriodCellTemplateConfig)
+                        .collect(Collectors.toList()));
+                cellCalcInfoDtos.add(cellCalcInfoDto);
+            }
         });
         return cellCalcInfoDtos;
     }

atms-api/src/main/java/pwc/taxtech/atms/vat/service/impl/ReportServiceImpl.java

@@ -620,8 +620,8 @@ public class ReportServiceImpl extends VatAbstractService implements ReportServi
         CellData cellData = cellDataMapper.selectByPrimaryKey(cellDataID);
         BigDecimal cellVal = new BigDecimal("0");
         try {
-            cellVal = new BigDecimal(cellData.getData());
-            if (cellData != null && cellData.getData() != null && (StringUtils.isBlank(cellData.getData()))) {
+            if (cellData != null){
+                cellVal = new BigDecimal(cellData.getData());
                 Integer cellDataType = periodCellTemplateMapper
                         .getDataType(cellData.getCellTemplateId()
                                 , cellData.getReportId());

atms-web/src/main/webapp/app/admin/systemConfiguration/declarationFormConfiguration/declaration-form-configuration.ctrl.js

@@ -783,7 +783,7 @@
                 if ($scope.templateFormulaList && $scope.templateFormulaList.length > 0) {
 
                     var formulaModel = _.find($scope.templateFormulaList, function (num) {
-                        return num.templateID === template.id && num.rowIndex === row && num.columnIndex === col
+                        return num.templateID == template.id && num.rowIndex === row && num.columnIndex === col
                     });
                     if (formulaModel) {
                         $scope.editModel = angular.copy(formulaModel);

atms-web/src/main/webapp/bundles/systemConfiguration.js

@@ -783,7 +783,7 @@ systemConfigurationModule
                 if ($scope.templateFormulaList && $scope.templateFormulaList.length > 0) {
 
                     var formulaModel = _.find($scope.templateFormulaList, function (num) {
-                        return num.templateID === template.id && num.rowIndex === row && num.columnIndex === col
+                        return num.templateID == template.id && num.rowIndex === row && num.columnIndex === col
                     });
                     if (formulaModel) {
                         $scope.editModel = angular.copy(formulaModel);