[app-builder] 多版本和结构优化#210
Merged
CodeCasterX merged 1 commit intoModelEngine-Group:developfrom Jun 5, 2025
Merged
Conversation
Contributor
wuayee
commented
May 29, 2025
wuayee
commented
- 修改应用多版本内部逻辑
- 重构部分代码，采用领域开发模式
wuayee
requested review from
a team and
XinLi-cyber
and removed request for
a team
CodeCasterX
added
in: builder
wuayee
force-pushed
the
appbuilder-enhancement-multi-version-wzc
branch
5 times, most recently
from
0606e24 to
e832097
Compare
XinLi-cyber
requested changes
May 30, 2025
| /** | ||
| * 集合是否为空. | ||
| * | ||
| * @return true/false. |
| private List<String> excludeNames; | ||
|
|
||
| private String appSuiteId; | ||
|
|
| private final AppVersionService appVersionService; | ||
|
|
||
| /** | ||
| * AippFlowController |
| List<AppBuilderConfigProperty> toAddConfigProperties = properties.stream() | ||
| .filter(pd -> StringUtils.isBlank(pd.getId()) || !formPropertyIds.contains(pd.getId())) | ||
| .map(propertyDto -> { | ||
| AppBuilderFormProperty formProperty = AppBuilderFormProperty.builder() |
| .map(AppBuilderFormProperty::getId) | ||
| .collect(Collectors.toSet()); | ||
| List<AppBuilderConfigProperty> toAddConfigProperties = properties.stream() | ||
| .filter(pd -> StringUtils.isBlank(pd.getId()) || !formPropertyIds.contains(pd.getId())) |
| */ | ||
| public class AppVersion { | ||
| private static final Logger LOGGER = Logger.get(AppVersion.class); | ||
|
|
| .putQueryAttribute(ATTR_AIPP_TYPE_KEY, AippTypeEnum.NORMAL.type()) | ||
| .putQueryAttribute(ATTR_META_STATUS_KEY, AippMetaStatusEnum.ACTIVE.getCode()) | ||
| .build(), context.getOperationContext()); | ||
| if (!resultSet.isEmpty()) { |
Contributor
There was a problem hiding this comment.
建议用collectionUtils，保持风格一致
Contributor
Author
There was a problem hiding this comment.
resultSet 是对象，不是集合类型，无法使用collectionUtils
| * @param context 操作上下文 | ||
| */ | ||
| public void updateFlows(OperationContext context) { | ||
| if (this.isUpdated()) { |
| this.data.setAppType(createDto.getAppType()); | ||
| this.setDescription(createDto.getDescription()); | ||
| String icon = this.getIcon(); | ||
| if (StringUtils.isNotBlank(icon) && StringUtils.equals(icon, createDto.getIcon())) { |
Contributor
There was a problem hiding this comment.
1、改为卫语句
2、判断条件抽取为变量或方法
| } | ||
| } | ||
|
|
||
| public void cloneVersion(AppVersion cloneApp) { |
loveTsong
reviewed
Jun 3, 2025
| * 查询meta实例。 | ||
| * | ||
| * @param versionId 表示实例所属meta唯一标识的 {@link String}。 | ||
| * @param filter 表示meta实例过滤器的 {@link MetaInstanceFilter}。 |
loveTsong
reviewed
Jun 3, 2025
| import modelengine.fit.jober.aipp.entity.ChatSession; | ||
|
|
||
| /** | ||
| * 应用执行接口. |
wuayee
force-pushed
the
appbuilder-enhancement-multi-version-wzc
branch
2 times, most recently
from
883f725 to
c6fe69e
Compare
XinLi-cyber
requested changes
Jun 4, 2025
| if (context.isApp()) { | ||
| return this.appService.publishApp(appData); | ||
| } | ||
| if (context.isWaterFlow()) { |
| private final AppBuilderConfigPropertyRepository configPropertyRepository; | ||
| private final AppTaskService appTaskService; | ||
| private final AppVersionFactory appVersionFactory; | ||
|
|
| private Object value; | ||
|
|
||
| public MemoryConfig(List<Map<String, Object>> memoryConfigs) { | ||
| if (!CollectionUtils.isEmpty(memoryConfigs)) { |
| private Boolean enableMemory; | ||
| private Object value; | ||
|
|
||
| public MemoryConfig(List<Map<String, Object>> memoryConfigs) { |
| * @return {@link AppLog} 对象. | ||
| */ | ||
| public AppLog create(AippInstLog logData) { | ||
| if (FORM.name().equals(logData.getLogType())) { |
| private List<AppLog> logs; | ||
| private List<AppLog> allLogs; | ||
|
|
||
| private final TaskInstanceEntity<?> entity; |
XinLi-cyber
force-pushed
the
appbuilder-enhancement-multi-version-wzc
branch
from
12edda9 to
96dca45
Compare
wuayee
force-pushed
the
appbuilder-enhancement-multi-version-wzc
branch
2 times, most recently
from
5c2626f to
6b8c129
Compare
wuayee
force-pushed
the
appbuilder-enhancement-multi-version-wzc
branch
from
6b8c129 to
8ddbefa
Compare
This was
linked to
issues
Jun 6, 2025
Closed
Closed
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.