diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepository.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepository.java index 646067fad9..fe329b25f9 100644 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepository.java +++ b/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepository.java @@ -1,6 +1,5 @@ package com.appsmith.server.repositories; -import com.mongodb.client.result.UpdateResult; import org.springframework.data.mongodb.repository.ReactiveMongoRepository; import org.springframework.data.repository.NoRepositoryBean; import reactor.core.publisher.Mono; @@ -8,7 +7,6 @@ import reactor.core.publisher.Mono; import java.io.Serializable; import java.util.Collection; import java.util.List; -import java.util.Map; @NoRepositoryBean public interface BaseRepository extends ReactiveMongoRepository { @@ -53,9 +51,4 @@ public interface BaseRepository extends ReactiveMong * against the `fieldNames` property in the matching object. */ Mono findByIdAndFieldNames(ID id, List fieldNames); - - /** - * This method is supposed to update the given list of fields in an object as opposed to replacing the entire object. - */ - Mono updateByIdAndFieldNames(ID id, Map fieldNameValueMap); } diff --git a/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepositoryImpl.java b/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepositoryImpl.java index 4f4cefc1d6..843cfe75f9 100644 --- a/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepositoryImpl.java +++ b/app/server/appsmith-server/src/main/java/com/appsmith/server/repositories/BaseRepositoryImpl.java @@ -2,8 +2,6 @@ package com.appsmith.server.repositories; import com.appsmith.external.models.BaseDomain; import com.appsmith.server.constants.FieldName; -import com.mongodb.client.result.UpdateResult; -import jakarta.validation.constraints.NotNull; import lombok.NonNull; import lombok.extern.slf4j.Slf4j; import org.springframework.data.domain.Example; @@ -23,7 +21,6 @@ import java.io.Serializable; import java.time.Instant; import java.util.Collection; import java.util.List; -import java.util.Map; import static org.apache.commons.lang3.StringUtils.isBlank; import static org.springframework.data.mongodb.core.query.Criteria.where; @@ -144,27 +141,6 @@ public class BaseRepositoryImpl return this.findById(id); } - /** - * This method is supposed to update the given list of field names with the associated values in an object as opposed to replacing the entire object. - */ - @Override - public Mono updateByIdAndFieldNames(@NotNull ID id, @NotNull Map fieldNameValueMap) { - return ReactiveSecurityContextHolder.getContext() - .map(ctx -> ctx.getAuthentication()) - .map(auth -> auth.getPrincipal()) - .flatMap(principal -> { - Query query = new Query(getIdCriteria(id)); - query.addCriteria(notDeleted()); - - Update update = new Update(); - fieldNameValueMap.forEach((fieldName, fieldValue) -> { - update.set(fieldName, fieldValue); - }); - - return mongoOperations.updateFirst(query, update, entityInformation.getJavaType()); - }); - } - @Override public Flux findAll() { return ReactiveSecurityContextHolder.getContext()