Finalize Note service

Author: Chamal-Peiris

src/main/java/lk/ijse/dep/note/advice/GlobalExceptionHandler.java

@@ -2,6 +2,7 @@
 
 import lk.ijse.dep.note.service.exception.DuplicateEmailException;
 import lk.ijse.dep.note.service.exception.NotFoundException;
+import lk.ijse.dep.note.service.exception.UnauthorizedAccessException;
 import org.springframework.http.HttpStatus;
 import org.springframework.web.bind.annotation.ExceptionHandler;
 import org.springframework.web.bind.annotation.ResponseStatus;
@@ -23,4 +24,10 @@ public String handleNotfoundExcpetion(NotFoundException e){
         e.initCause(new ResponseStatusException(HttpStatus.NOT_FOUND,e.getMessage()));
         return e.getMessage();
     }
+    @ResponseStatus(HttpStatus.FORBIDDEN)
+    @ExceptionHandler(UnauthorizedAccessException.class)
+    public String handleUnAuthorizedAccessException(NotFoundException e){
+        e.initCause(new ResponseStatusException(HttpStatus.NOT_FOUND,e.getMessage()));
+        return e.getMessage();
+    }
 }

src/main/java/lk/ijse/dep/note/api/NoteController.java

@@ -10,7 +10,7 @@
 import org.springframework.web.server.ResponseStatusException;
 import java.util.List;
 @RestController
-@RequestMapping("api/v1/users/{userId:[A-F0-9\\-]{36}/notes")
+@RequestMapping("/api/v1/users/{userId:[A-Fa-f0-9\\-]{36}}/notes")
 public class NoteController {
 
     @Autowired

src/main/java/lk/ijse/dep/note/dto/NoteDTO.java

@@ -12,9 +12,9 @@
 @NoArgsConstructor @AllArgsConstructor @Data
 public class NoteDTO implements Serializable {
     @Null(message = "Id cannot be set")
-    private int id;
+    private Integer id;
     @NotBlank(message = "Text cannot be empty")
     private String text;
-    @Pattern(regexp = "[A-Fa-F0-9\\-]{36}",message = "Invalid user id")
+    @Pattern(regexp = "[A-Fa-f0-9\\-]{36}",message = "Invalid user id")
     private String userId;
 }

src/main/java/lk/ijse/dep/note/repository/custom/NoteRepository.java

@@ -1,7 +1,13 @@
 package lk.ijse.dep.note.repository.custom;
 
 import lk.ijse.dep.note.entity.Note;
+import lk.ijse.dep.note.entity.User;
 import lk.ijse.dep.note.repository.CrudRepository;
 
+import java.util.List;
+
 public interface NoteRepository extends CrudRepository<Note,Integer> {
+
+
+    List<Note> findAllNotesByUser(User user);
 }

src/main/java/lk/ijse/dep/note/repository/custom/impl/NoteRepositoryImpl.java

@@ -1,10 +1,17 @@
 package lk.ijse.dep.note.repository.custom.impl;
 
 import lk.ijse.dep.note.entity.Note;
+import lk.ijse.dep.note.entity.User;
 import lk.ijse.dep.note.repository.CrudRepositoryImpl;
 import lk.ijse.dep.note.repository.custom.NoteRepository;
 import org.springframework.stereotype.Repository;
 
+import java.util.List;
+
 @Repository
 public class NoteRepositoryImpl extends CrudRepositoryImpl<Note,Integer> implements NoteRepository {
+    @Override
+    public List<Note> findAllNotesByUser(User user) {
+      return  entityManager.createQuery("SELECT n FROM Note n WHERE n.user=:user",Note.class).setParameter("user",user).getResultList();
+    }
 }

src/main/java/lk/ijse/dep/note/service/exception/UnauthorizedAccessException.java

@@ -0,0 +1,15 @@
+package lk.ijse.dep.note.service.exception;
+
+public class UnauthorizedAccessException extends RuntimeException{
+    public UnauthorizedAccessException(String message) {
+        super(message);
+    }
+
+    public UnauthorizedAccessException(String message, Throwable cause) {
+        super(message, cause);
+    }
+
+    public UnauthorizedAccessException(Throwable cause) {
+        super(cause);
+    }
+}

src/main/java/lk/ijse/dep/note/service/impl/NoteServiceImpl.java

@@ -1,33 +1,62 @@
 package lk.ijse.dep.note.service.impl;
 
 import lk.ijse.dep.note.dto.NoteDTO;
+import lk.ijse.dep.note.entity.Note;
+import lk.ijse.dep.note.entity.User;
 import lk.ijse.dep.note.repository.custom.NoteRepository;
+import lk.ijse.dep.note.repository.custom.UserRepository;
 import lk.ijse.dep.note.service.Noteservice;
 import lk.ijse.dep.note.service.exception.NotFoundException;
+import lk.ijse.dep.note.service.exception.UnauthorizedAccessException;
+import lk.ijse.dep.note.service.util.EntityDTOConverter;
+import org.modelmapper.ModelMapper;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
+import java.util.Optional;
+import java.util.stream.Collectors;
 
 @Service
 @Transactional
 public class NoteServiceImpl implements Noteservice {
 
     @Autowired
     private NoteRepository noteRepository;
+    @Autowired
+    private EntityDTOConverter converter;
+    @Autowired
+    private UserRepository userRepository;
     @Override
     public NoteDTO saveNote(NoteDTO note) throws NotFoundException {
-        return null;
+
+        Note noteEntity = converter.getNoteEntity(note);
+        noteEntity.setUser(getUser(note.getUserId()));
+        return converter.getNoteDTO(noteRepository.save(noteEntity));
+
     }
 
     @Override
     public void deleteNote(String userId, int noteId) throws NotFoundException {
 
+        Optional<Note> opNote = noteRepository.findById(noteId);
+        if(!opNote.isPresent()) throw  new NotFoundException("Invalid Note id");
+        if(getUser(userId)!=opNote.get().getUser()) throw  new UnauthorizedAccessException("User is not authorized to delete this note");
+        noteRepository.deleteById(noteId);
+
+
     }
 
     @Override
     public List<NoteDTO> getAllNotes(String userId) {
-        return null;
+
+       return noteRepository.findAllNotesByUser(getUser(userId)).stream().map(converter::getNoteDTO).collect(Collectors.toList());
+    }
+
+    private User getUser(String userId){
+        Optional<User> optUser = userRepository.findById(userId);
+        if(!optUser.isPresent()) throw new NotFoundException("User Doesnt exist");
+        return optUser.get();
     }
 }

src/main/java/lk/ijse/dep/note/service/util/EntityDTOConverter.java

@@ -1,6 +1,8 @@
 package lk.ijse.dep.note.service.util;
 
+import lk.ijse.dep.note.dto.NoteDTO;
 import lk.ijse.dep.note.dto.UserDTO;
+import lk.ijse.dep.note.entity.Note;
 import lk.ijse.dep.note.entity.User;
 import org.modelmapper.ModelMapper;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -21,4 +23,12 @@ public User getUserEntity(UserDTO user){
     public UserDTO getUserDTO(User user){
         return mapper.map(user,UserDTO.class);
     }
+
+    public Note getNoteEntity(NoteDTO noteDTO){
+        return mapper.map(noteDTO,Note.class);
+    }
+    public NoteDTO getNoteDTO(Note noteDTO){
+       return mapper.typeMap(Note.class,NoteDTO.class).addMapping(note -> note.getUser().getId(),NoteDTO::setUserId).map(noteDTO);
+
+    }
 }