diff --git a/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileItemServiceImpl.java b/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileItemServiceImpl.java index 6537d7e6..a68f4b3c 100644 --- a/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileItemServiceImpl.java +++ b/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileItemServiceImpl.java @@ -14,6 +14,7 @@ import org.springframework.transaction.annotation.Transactional; import java.util.ArrayList; import java.util.List; +import java.util.Objects; import java.util.stream.Collectors; /** @@ -72,10 +73,12 @@ public class ProductModelFileItemServiceImpl extends ServiceImpl { - lambdaUpdate() - .set(ProductModelFileItem::getShowInList, true) - .eq(ProductModelFileItem::getId, item.getFileItemId()) - .update(); + if (Objects.nonNull(item.getFileItemId())) { + lambdaUpdate() + .set(ProductModelFileItem::getShowInList, true) + .eq(ProductModelFileItem::getId, item.getFileItemId()) + .update(); + } }); } diff --git a/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileServiceImpl.java b/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileServiceImpl.java index 2b93511f..d9e342c8 100644 --- a/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileServiceImpl.java +++ b/nflg-mobilebroken-repository/src/main/java/com/nflg/mobilebroken/repository/service/impl/ProductModelFileServiceImpl.java @@ -27,7 +27,6 @@ import javax.annotation.Resource; import java.time.LocalDateTime; import java.util.ArrayList; import java.util.List; -import java.util.Map; import java.util.Objects; import java.util.stream.Collectors; @@ -229,25 +228,20 @@ public class ProductModelFileServiceImpl extends ServiceImpl getDefaultConfig(Integer modelId) { - List languages = languageService.getAllLanguages(); -// List list = productModelFileItemService.lambdaQuery() -// .eq(ProductModelFileItem::getModelId, modelId) -// .orderByAsc(ProductModelFileItem::getLanguageCode) -// .orderByDesc(ProductModelFileItem::getShowInList) -// .list(); - List list =productModelFileItemService.getPublishedByLanguage(modelId); List vos=new ArrayList<>(); - Map> maps=list.stream().collect(Collectors.groupingBy(ProductModelFileItem::getLanguageCode)); - maps.forEach((languageCode,data)->{ - FileDefaultConfigVO vo=new FileDefaultConfigVO() - .setLanguageCode(languageCode) - .setLanguageName(languages.stream().filter(lang->lang.getCode().equals(languageCode)).findFirst().get().getName()) - .setItems(data.stream().map(item->new FileDefaultConfigItemVO() - .setId(item.getId()) - .setName(item.getName()) - .setSelected(item.getShowInList())) - .collect(Collectors.toList()) - ); + List languages = languageService.getLanguages(); + List list = productModelFileItemService.getPublishedByLanguage(modelId); + languages.forEach(language -> { + FileDefaultConfigVO vo = new FileDefaultConfigVO() + .setLanguageCode(language.getCode()) + .setLanguageName(language.getName()) + .setItems(list.stream().filter(item -> StrUtil.equals(language.getCode(), item.getLanguageCode())) + .map(item -> { + return new FileDefaultConfigItemVO() + .setId(item.getId()) + .setName(item.getName()) + .setSelected(item.getShowInList()); + }).collect(Collectors.toList())); vos.add(vo); }); return vos;