Skip to content

Commit 5567f0a

Browse files
authored
Merge pull request #1583 from lowcoder-org/app_create_with_gid
Update app create endpoint to use specific gid
2 parents b4f61eb + 3693442 commit 5567f0a

File tree

5 files changed

+8
-5
lines changed

5 files changed

+8
-5
lines changed

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/ApplicationApiServiceImpl.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import lombok.extern.slf4j.Slf4j;
1010
import org.apache.commons.collections4.CollectionUtils;
1111
import org.apache.commons.collections4.SetUtils;
12+
import org.apache.commons.lang3.ObjectUtils;
1213
import org.apache.commons.lang3.StringUtils;
1314
import org.lowcoder.api.application.ApplicationEndpoints.CreateApplicationRequest;
1415
import org.lowcoder.api.application.view.ApplicationInfoView;
@@ -101,7 +102,7 @@ public class ApplicationApiServiceImpl implements ApplicationApiService {
101102
public Mono<ApplicationView> create(CreateApplicationRequest createApplicationRequest) {
102103

103104
Application application = new Application(createApplicationRequest.organizationId(),
104-
UuidCreator.getTimeOrderedEpoch().toString(),
105+
ObjectUtils.defaultIfNull(createApplicationRequest.gid(), UuidCreator.getTimeOrderedEpoch().toString()),
105106
createApplicationRequest.name(),
106107
createApplicationRequest.applicationType(),
107108
NORMAL,

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/application/ApplicationEndpoints.java

+1
Original file line numberDiff line numberDiff line change
@@ -298,6 +298,7 @@ public record UpdatePermissionRequest(String role) {
298298
}
299299

300300
public record CreateApplicationRequest(@JsonProperty("orgId") String organizationId,
301+
@Nullable String gid,
301302
String name,
302303
Integer applicationType,
303304
Map<String, Object> editingApplicationDSL,

server/api-service/lowcoder-server/src/main/java/org/lowcoder/api/datasource/UpsertDatasourceRequestMapper.java

+1
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ public Datasource resolve(UpsertDatasourceRequest dto) {
3434

3535
Datasource datasource = new Datasource();
3636
datasource.setId(dto.getId());
37+
datasource.setGid(dto.getGid());
3738
datasource.setName(dto.getName());
3839
datasource.setType(dto.getType());
3940
datasource.setOrganizationId(dto.getOrganizationId());

server/api-service/lowcoder-server/src/test/java/org/lowcoder/api/application/ApplicationApiServiceIntegrationTest.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ public void testCreateApplicationSuccess() {
7070
});
7171
//
7272
Mono<ApplicationView> applicationViewMono = datasourceMono.map(datasource -> new CreateApplicationRequest(
73-
"org01",
73+
"org01", null,
7474
"app05",
7575
ApplicationType.APPLICATION.getValue(),
7676
Map.of("comp", "list", "queries", Set.of(Map.of("datasourceId", datasource.getId()))),
@@ -104,7 +104,7 @@ public void testUpdateApplicationFailedDueToLackOfDatasourcePermissions() {
104104
});
105105
//
106106
Mono<ApplicationView> applicationViewMono = datasourceMono.map(datasource -> new CreateApplicationRequest(
107-
"org01",
107+
"org01", null,
108108
"app03",
109109
ApplicationType.APPLICATION.getValue(),
110110
Map.of("comp", "list", "queries", Set.of(Map.of("datasourceId", datasource.getId()))),
@@ -129,7 +129,7 @@ public void testUpdateApplicationFailedDueToLackOfDatasourcePermissions() {
129129
@Test
130130
@WithMockUser
131131
public void testUpdateEditingStateSuccess() {
132-
Mono<ApplicationView> applicationViewMono = applicationApiService.create(new CreateApplicationRequest("org01", "app1", ApplicationType.APPLICATION.getValue(), Map.of("comp", "list"), null));
132+
Mono<ApplicationView> applicationViewMono = applicationApiService.create(new CreateApplicationRequest("org01", null, "app1", ApplicationType.APPLICATION.getValue(), Map.of("comp", "list"), null));
133133
Mono<ApplicationView> updateEditStateMono = applicationViewMono.delayUntil(app -> applicationApiService.updateEditState(app.getApplicationInfoView().getApplicationId(), new ApplicationEndpoints.UpdateEditStateRequest(true)));
134134
Mono<ApplicationView> app = updateEditStateMono.flatMap(applicationView -> applicationApiService.getEditingApplication(applicationView.getApplicationInfoView().getApplicationId()));
135135
StepVerifier.create(app)

server/api-service/lowcoder-server/src/test/java/org/lowcoder/api/application/ApplicationApiServiceTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ public void testDeleteNormalApplicationWithError() {
130130

131131
private Mono<ApplicationView> createApplication(String name, String folderId) {
132132
CreateApplicationRequest createApplicationRequest =
133-
new CreateApplicationRequest("org01", name, ApplicationType.APPLICATION.getValue(),
133+
new CreateApplicationRequest("org01", null, name, ApplicationType.APPLICATION.getValue(),
134134
Map.of("comp", "list"), folderId);
135135
return applicationApiService.create(createApplicationRequest);
136136
}

0 commit comments

Comments
 (0)