Change category field of publication entity to its id.

codiki-application/src/main/java/org/codiki/application/category/CategoryUseCases.java

@@ -27,6 +27,10 @@ public class CategoryUseCases {
         return categoryPort.findById(categoryId);
     }
 
+    public boolean existsById(UUID categoryId) {
+        return categoryPort.existsById(categoryId);
+    }
+
     public Category createCategory(String name, List<UUID> subCategoryIds) {
         if (isNull(name)) {
             throw new CategoryEditionException("name can not be empty");

codiki-application/src/main/java/org/codiki/application/picture/PictureUseCases.java

@@ -5,21 +5,30 @@ import java.util.Optional;
 import java.util.UUID;
 
 import static org.codiki.domain.picture.model.builder.PictureBuilder.aPicture;
+import org.codiki.application.user.UserUseCases;
+import org.codiki.domain.exception.AuthenticationRequiredException;
 import org.codiki.domain.picture.model.Picture;
 import org.codiki.domain.picture.port.PicturePort;
+import org.codiki.domain.user.model.User;
 import org.springframework.stereotype.Service;
 
 @Service
 public class PictureUseCases {
     private final PicturePort picturePort;
+    private final UserUseCases userUseCases;
 
-    public PictureUseCases(PicturePort picturePort) {
+    public PictureUseCases(PicturePort picturePort, UserUseCases userUseCases) {
         this.picturePort = picturePort;
+        this.userUseCases = userUseCases;
     }
 
     public Picture createPicture(File pictureFile) {
+        User authenticatedUser = userUseCases.getAuthenticatedUser()
+            .orElseThrow(AuthenticationRequiredException::new);
+
         Picture newPicture = aPicture()
             .withId(UUID.randomUUID())
+            .withPublisher(authenticatedUser)
             .withContentFile(pictureFile)
             .build();
 
@@ -35,4 +44,8 @@ public class PictureUseCases {
     public Optional<Picture> findById(UUID pictureId) {
         return picturePort.findById(pictureId);
     }
+
+    public boolean existsById(UUID pictureId) {
+        return picturePort.existsById(pictureId);
+    }
 }

codiki-application/src/main/java/org/codiki/application/publication/PublicationCreationRequestValidator.java

@@ -19,8 +19,8 @@ public class PublicationCreationRequestValidator {
             throw new PublicationEditionException("description cannot be null");
         }
 
-        if (request.pictureId() == null) {
-            throw new PublicationEditionException("pictureId cannot be null");
+        if (request.illustrationId() == null) {
+            throw new PublicationEditionException("illustrationId cannot be null");
         }
     }
 }

codiki-application/src/main/java/org/codiki/application/publication/PublicationUpdateRequestValidator.java

@@ -13,7 +13,7 @@ public class PublicationUpdateRequestValidator {
             isNull(request.title()) &&
             isNull(request.text()) &&
             isNull(request.description()) &&
-            isNull(request.pictureId()) &&
+            isNull(request.illustrationId()) &&
             isNull(request.categoryId())
         ) {
             throw new PublicationEditionException("no any field is filled");

codiki-application/src/main/java/org/codiki/application/publication/PublicationUseCases.java

@@ -15,7 +15,6 @@ import org.codiki.domain.category.exception.CategoryNotFoundException;
 import org.codiki.domain.category.model.Category;
 import org.codiki.domain.exception.AuthenticationRequiredException;
 import org.codiki.domain.picture.exception.PictureNotFoundException;
-import org.codiki.domain.picture.model.Picture;
 import org.codiki.domain.publication.exception.PublicationEditionException;
 import org.codiki.domain.publication.exception.PublicationNotFoundException;
 import org.codiki.domain.publication.exception.PublicationUpdateForbiddenException;
@@ -63,15 +62,17 @@ public class PublicationUseCases {
         User authenticatedUser = userUseCases.getAuthenticatedUser()
             .orElseThrow(AuthenticationRequiredException::new);
 
-        Category category = categoryUseCases.findById(request.categoryId())
-            .orElseThrow(() -> new PublicationEditionException(
+        if (!categoryUseCases.existsById(request.categoryId())) {
+            throw new PublicationEditionException(
                 new CategoryNotFoundException(request.categoryId())
-            ));
+            );
+        }
 
-        Picture picture = pictureUseCases.findById(request.pictureId())
-            .orElseThrow(() -> new PublicationEditionException(
-                new PictureNotFoundException(request.pictureId())
-            ));
+        if (!pictureUseCases.existsById(request.illustrationId())) {
+            throw new PublicationEditionException(
+                new PictureNotFoundException(request.illustrationId())
+            );
+        }
 
         Publication newPublication = aPublication()
             .withId(UUID.randomUUID())
@@ -79,10 +80,10 @@ public class PublicationUseCases {
             .withTitle(request.title())
             .withText(request.text())
             .withDescription(request.description())
-            .withPicture(picture)
             .withCreationDate(ZonedDateTime.now(clock))
+            .withIllustrationId(request.illustrationId())
+            .withCategoryId(request.categoryId())
             .withAuthor(anAuthor().basedOn(authenticatedUser).build())
-            .withCategory(category)
             .build();
 
         publicationPort.save(newPublication);
@@ -117,21 +118,23 @@ public class PublicationUseCases {
             publicationBuilder.withDescription(request.description());
         }
 
-        if (!isNull(request.pictureId())) {
-            Picture picture = pictureUseCases.findById(request.pictureId())
-                .orElseThrow(() -> new PublicationEditionException(
-                    new PictureNotFoundException(request.pictureId())
-                ));
-            publicationBuilder.withPicture(picture);
+        if (!isNull(request.illustrationId())) {
+            if (!pictureUseCases.existsById(request.illustrationId())) {
+                throw new PublicationEditionException(
+                    new PictureNotFoundException(request.illustrationId())
+                );
+            }
+            publicationBuilder.withIllustrationId(request.illustrationId());
         }
 
         if (!isNull(request.categoryId())) {
-            Category newCategory = categoryUseCases.findById(request.categoryId())
-                .orElseThrow(() -> new PublicationEditionException(
+            if (!categoryUseCases.existsById(request.categoryId())) {
+                throw new PublicationEditionException(
                     new CategoryNotFoundException(request.categoryId())
-                ));
+                );
+            }
 
-            publicationBuilder.withCategory(newCategory);
+            publicationBuilder.withCategoryId(request.categoryId());
         }
 
         Publication updatedPublication = publicationBuilder.build();

codiki-domain/src/main/java/org/codiki/domain/picture/model/Picture.java

@@ -5,5 +5,6 @@ import java.util.UUID;
 
 public record Picture(
     UUID id,
+    UUID publisherId,
     File contentFile
 ) {}

codiki-domain/src/main/java/org/codiki/domain/picture/model/builder/PictureBuilder.java

@@ -4,9 +4,11 @@ import java.io.File;
 import java.util.UUID;
 
 import org.codiki.domain.picture.model.Picture;
+import org.codiki.domain.user.model.User;
 
 public class PictureBuilder {
     private UUID id;
+    private UUID publisherId;
     private File contentFile;
 
     private PictureBuilder() {}
@@ -26,6 +28,14 @@ public class PictureBuilder {
         return this;
     }
 
+    public PictureBuilder withPublisherId(UUID publisherId) {
+        this.publisherId = publisherId;
+        return this;
+    }
+
+    public PictureBuilder withPublisher(User publisher) {
+        return withPublisherId(publisher.id());
+    }
 
     public PictureBuilder withContentFile(File contentFile) {
         this.contentFile = contentFile;
@@ -33,6 +43,6 @@ public class PictureBuilder {
     }
 
     public Picture build() {
-        return new Picture(id, contentFile);
+        return new Picture(id, publisherId, contentFile);
     }
 }

codiki-domain/src/main/java/org/codiki/domain/publication/model/Publication.java

@@ -3,9 +3,6 @@ package org.codiki.domain.publication.model;
 import java.time.ZonedDateTime;
 import java.util.UUID;
 
-import org.codiki.domain.category.model.Category;
-import org.codiki.domain.picture.model.Picture;
-
 public record Publication(
     UUID id,
     String key,
@@ -13,8 +10,8 @@ public record Publication(
     String text,
     String description,
     ZonedDateTime creationDate,
-    Picture picture,
-    Author author,
-    Category category
+    UUID illustrationId,
+    UUID categoryId,
+    Author author
 ) {
 }

codiki-domain/src/main/java/org/codiki/domain/publication/model/PublicationEditionRequest.java

@@ -6,6 +6,6 @@ public record PublicationEditionRequest(
     String title,
     String text,
     String description,
-    UUID pictureId,
+    UUID illustrationId,
     UUID categoryId
 ) {}

codiki-domain/src/main/java/org/codiki/domain/publication/model/builder/AuthorBuilder.java

@@ -20,7 +20,7 @@ public class AuthorBuilder {
         return new AuthorBuilder()
             .withId(user.id())
 //            .withName(user.name())
-//            .withImage(user.pictureId())
+//            .withImage(user.illustrationId())
         ;
     }
 

codiki-domain/src/main/java/org/codiki/domain/publication/model/builder/PublicationBuilder.java

@@ -3,9 +3,8 @@ package org.codiki.domain.publication.model.builder;
 import java.time.ZonedDateTime;
 import java.util.UUID;
 
-import org.codiki.domain.picture.model.Picture;
-import org.codiki.domain.publication.model.Author;
 import org.codiki.domain.category.model.Category;
+import org.codiki.domain.publication.model.Author;
 import org.codiki.domain.publication.model.Publication;
 
 public class PublicationBuilder {
@@ -14,10 +13,10 @@ public class PublicationBuilder {
     private String title;
     private String text;
     private String description;
-    private Picture picture;
     private ZonedDateTime creationDate;
+    private UUID illustrationId;
+    private UUID categoryId;
     private Author author;
-    private Category category;
 
     private PublicationBuilder() {}
 
@@ -25,6 +24,19 @@ public class PublicationBuilder {
         return new PublicationBuilder();
     }
 
+    public PublicationBuilder basedOn(Publication publication) {
+        return new PublicationBuilder()
+            .withId(publication.id())
+            .withKey(publication.key())
+            .withTitle(publication.title())
+            .withText(publication.text())
+            .withDescription(publication.description())
+            .withCreationDate(publication.creationDate())
+            .withIllustrationId(publication.illustrationId())
+            .withCategoryId(publication.categoryId())
+            .withAuthor(publication.author());
+    }
+
     public PublicationBuilder withId(UUID id) {
         this.id = id;
         return this;
@@ -50,23 +62,23 @@ public class PublicationBuilder {
         return this;
     }
 
-    public PublicationBuilder withPicture(Picture picture) {
-        this.picture = picture;
-        return this;
-    }
-
     public PublicationBuilder withCreationDate(ZonedDateTime creationDate) {
         this.creationDate = creationDate;
         return this;
     }
 
-    public PublicationBuilder withAuthor(Author author) {
-        this.author = author;
+    public PublicationBuilder withIllustrationId(UUID illustrationId) {
+        this.illustrationId = illustrationId;
         return this;
     }
 
-    public PublicationBuilder withCategory(Category category) {
-        this.category = category;
+    public PublicationBuilder withCategoryId(UUID categoryId) {
+        this.categoryId = categoryId;
+        return this;
+    }
+
+    public PublicationBuilder withAuthor(Author author) {
+        this.author = author;
         return this;
     }
 
@@ -77,22 +89,10 @@ public class PublicationBuilder {
             title,
             text,
             description,
-            creationDate, picture,
-            author,
-            category
+            creationDate,
+            illustrationId,
+            categoryId,
+            author
         );
     }
-
-    public PublicationBuilder basedOn(Publication publication) {
-        return new PublicationBuilder()
-            .withId(publication.id())
-            .withKey(publication.key())
-            .withTitle(publication.title())
-            .withText(publication.text())
-            .withDescription(publication.description())
-            .withPicture(publication.picture())
-            .withCreationDate(publication.creationDate())
-            .withAuthor(publication.author())
-            .withCategory(publication.category());
-    }
 }

codiki-exposition/src/main/java/org/codiki/exposition/publication/model/PublicationDto.java

@@ -13,9 +13,9 @@ public record PublicationDto(
     String text,
     String description,
     ZonedDateTime creationDate,
-    UUID picture,
-    AuthorDto author,
-    CategoryDto category
+    UUID illustrationId,
+    UUID categoryId,
+    AuthorDto author
 ) {
     public PublicationDto(Publication publication) {
         this(
@@ -25,9 +25,9 @@ public record PublicationDto(
             publication.text(),
             publication.description(),
             publication.creationDate(),
-            publication.picture().id(),
-            new AuthorDto(publication.author()),
-            new CategoryDto(publication.category())
+            publication.illustrationId(),
+            publication.categoryId(),
+            new AuthorDto(publication.author())
         );
     }
 }

codiki-exposition/src/main/java/org/codiki/exposition/publication/model/PublicationEditionRequestDto.java

@@ -8,10 +8,10 @@ public record PublicationEditionRequestDto(
     String title,
     String text,
     String description,
-    UUID pictureId,
+    UUID illustrationId,
     UUID categoryId
 ) {
     public PublicationEditionRequest toDomain() {
-        return new PublicationEditionRequest(title, text, description, pictureId, categoryId);
+        return new PublicationEditionRequest(title, text, description, illustrationId, categoryId);
     }
 }

codiki-infrastructure/src/main/java/org/codiki/infrastructure/picture/model/PictureEntity.java

@@ -4,6 +4,7 @@ import java.util.UUID;
 
 import org.codiki.domain.picture.model.Picture;
 
+import jakarta.persistence.Column;
 import jakarta.persistence.Entity;
 import jakarta.persistence.Id;
 import jakarta.persistence.Table;
@@ -21,12 +22,15 @@ import lombok.Setter;
 public class PictureEntity {
     @Id
     private UUID id;
+    @Column(nullable = false)
+    private UUID publisherId;
 
     public PictureEntity(Picture picture) {
         id = picture.id();
+        publisherId = picture.publisherId();
     }
 
     public Picture toDomain() {
-        return new Picture(id, null);
+        return new Picture(id, publisherId, null);
     }
 }

codiki-infrastructure/src/main/java/org/codiki/infrastructure/publication/model/AuthorEntity.java

@@ -24,13 +24,13 @@ public class AuthorEntity {
     private UUID id;
     @Column(nullable = false)
     private String name;
-//    private String pictureId;
+//    private String illustrationId;
 
     public AuthorEntity(Author author) {
         this(
             author.id(),
             author.name()
-//            author.pictureId()
+//            author.illustrationId()
         );
     }
 

codiki-infrastructure/src/main/java/org/codiki/infrastructure/publication/model/PublicationEntity.java

@@ -5,7 +5,6 @@ import java.util.UUID;
 
 import org.codiki.domain.publication.model.Publication;
 import org.codiki.infrastructure.category.model.CategoryEntity;
-import org.codiki.infrastructure.picture.model.PictureEntity;
 
 import static jakarta.persistence.FetchType.LAZY;
 import jakarta.persistence.Column;
@@ -38,15 +37,13 @@ public class PublicationEntity {
     private String description;
     @Column(nullable = false)
     private ZonedDateTime creationDate;
-    @ManyToOne(fetch = LAZY)
-    @JoinColumn(name = "picture_id")
-    private PictureEntity picture;
+    @Column(nullable = false)
+    private UUID illustrationId;
+    @Column(nullable = false)
+    private UUID categoryId;
     @ManyToOne(fetch = LAZY)
     @JoinColumn(name = "author_id")
     private AuthorEntity author;
-    @ManyToOne(fetch = LAZY)
-    @JoinColumn(name = "category_id")
-    private CategoryEntity category;
 
     public PublicationEntity(Publication publication) {
         this(
@@ -56,9 +53,9 @@ public class PublicationEntity {
             publication.text(),
             publication.description(),
             publication.creationDate(),
-            new PictureEntity(publication.picture()),
-            new AuthorEntity(publication.author()),
-            new CategoryEntity(publication.category())
+            publication.illustrationId(),
+            publication.categoryId(),
+            new AuthorEntity(publication.author())
         );
     }
 
@@ -69,9 +66,10 @@ public class PublicationEntity {
             title,
             text,
             description,
-            creationDate, picture.toDomain(),
-            author.toDomain(),
-            category.toDomain()
+            creationDate,
+            illustrationId,
+            categoryId,
+            author.toDomain()
         );
     }
 }

codiki-infrastructure/src/main/java/org/codiki/infrastructure/publication/repository/PublicationRepository.java

@@ -12,9 +12,7 @@ public interface PublicationRepository extends JpaRepository<PublicationEntity,
     @Query("""
         SELECT p
         FROM PublicationEntity p
-        JOIN FETCH p.picture pi
         JOIN FETCH p.author a
-        JOIN FETCH p.category c
         WHERE p.id = :publicationId
         """)
     Optional<PublicationEntity> findById(@Param("publicationId") UUID publicationId);

codiki-infrastructure/src/main/resources/sql/001-initial-script-tables-creation.sql

@@ -33,8 +33,11 @@ CREATE INDEX category_parent_category_id_idx ON category (parent_category_id);
 
 CREATE TABLE IF NOT EXISTS picture (
     id UUID NOT NULL,
-   CONSTRAINT picture_pk PRIMARY KEY (id)
+    publisher_id UUID NOT NULL,
+    CONSTRAINT picture_pk PRIMARY KEY (id),
+    CONSTRAINT picture_publisher_id_fk FOREIGN KEY (publisher_id) REFERENCES "user" (id)
 );
+CREATE INDEX picture_publisher_id_idx ON picture (publisher_id);
 
 CREATE TABLE IF NOT EXISTS publication (
     id UUID NOT NULL,
@@ -43,15 +46,15 @@ CREATE TABLE IF NOT EXISTS publication (
     text VARCHAR NOT NULL,
     description VARCHAR NOT NULL,
     creation_date TIMESTAMP NOT NULL,
-    picture_id UUID NOT NULL,
+    illustration_id UUID NOT NULL,
     author_id UUID NOT NULL,
     category_id UUID NOT NULL,
     CONSTRAINT publication_pk PRIMARY KEY (id),
-    CONSTRAINT publication_picture_id_fk FOREIGN KEY (picture_id) REFERENCES picture (id),
+    CONSTRAINT publication_picture_id_fk FOREIGN KEY (illustration_id) REFERENCES picture (id),
     CONSTRAINT publication_author_id_fk FOREIGN KEY (author_id) REFERENCES "user" (id),
     CONSTRAINT publication_category_id_fk FOREIGN KEY (category_id) REFERENCES category (id)
 );
-CREATE INDEX publication_picture_id_idx ON publication (picture_id);
+CREATE INDEX publication_picture_id_idx ON publication (illustration_id);
 CREATE INDEX publication_author_id_idx ON publication (author_id);
 CREATE INDEX publication_category_id_idx ON publication (category_id);
 

rest-client-collection/Codiki/environments/localhost.bru

@@ -1,7 +1,7 @@
 vars {
   url: http://localhost:8080
   publicationId: e23831a6-9cc0-4f3d-9efa-7a1cae191cb1
-  bearerToken: eyJhbGciOiJIUzUxMiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiI1YWQ0NjJiOC04ZjllLTRhMjYtYmI4Ni1jNzRmZWY1ZDExYjYiLCJleHAiOjE3MTA0MDc5Mjd9.QdanZXjfLztIVJU-pRS3gZxnC4GIycyhmlCkhYSPchiXpeNzruw_GY3fH1_qshC_AjOCSqGmCq3X1S_zVFdodQ
+  bearerToken: eyJhbGciOiJIUzUxMiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiI1YWQ0NjJiOC04ZjllLTRhMjYtYmI4Ni1jNzRmZWY1ZDExYjYiLCJleHAiOjE3MTA0MjI2NTl9.yHT26uwON5Kk5CWgNvzq2a9OrdJACG4Rk034GPKoZlaxXwK0k8meSHVlrX4ZqTyR3zoL3fm_ujootZRISqOPZw
   categoryId: 172fa901-3f4b-4540-92f3-1c15820e8ec9
   pictureId: 65b660b7-66bb-4e4a-a62c-fd0ca101f972
 }