Skip to content

Commit

Permalink
server: fix process card createOrUpdate
Browse files Browse the repository at this point in the history
  • Loading branch information
brig committed Nov 12, 2024
1 parent 3cc024b commit e9e80d8
Showing 1 changed file with 9 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
*
* http://www.apache.org/licenses/LICENSE-2.0
*
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
Expand Down Expand Up @@ -95,14 +95,20 @@ public List<ProcessCardEntry> listUserCards(UUID userId) {
}

public ProcessCardOperationResponse createOrUpdate(UUID id, UUID projectId, UUID repoId, String name, Optional<String> entryPoint, String description, InputStream icon, InputStream form, Map<String, Object> data) {
boolean exists;
if (id == null) {
if (projectId == null) {
throw new ConcordApplicationException("projectId or projectName is required");
}
if (name == null) {
throw new ConcordApplicationException("name is required");
}
id = dao.getIdByName(projectId, name).orElse(null);
exists = id != null;
} else {
exists = dao.get(id) != null;
}

boolean exists = id != null;
if (!exists) {
UUID resultId = dao.insert(id, projectId, repoId, name, entryPoint.orElse(Constants.Request.DEFAULT_ENTRY_POINT_NAME), description, icon, form, data);
return new ProcessCardOperationResponse(resultId, OperationResult.CREATED);
Expand Down

0 comments on commit e9e80d8

Please sign in to comment.