Testing branch #47

Merged
DmitriiKaban merged 4 commits from testing_branch into master 2023-12-12 07:51:07 +00:00
5 changed files with 70 additions and 39 deletions
Showing only changes of commit 8420f15732 - Show all commits

View File

@@ -12,6 +12,7 @@ import com.faf223.expensetrackerfaf.util.exceptions.UserNotCreatedException;
import com.faf223.expensetrackerfaf.util.exceptions.UserNotFoundException; import com.faf223.expensetrackerfaf.util.exceptions.UserNotFoundException;
import jakarta.validation.Valid; import jakarta.validation.Valid;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity; import org.springframework.http.ResponseEntity;
import org.springframework.security.access.prepost.PreAuthorize; import org.springframework.security.access.prepost.PreAuthorize;
import org.springframework.security.core.Authentication; import org.springframework.security.core.Authentication;
@@ -80,5 +81,14 @@ public class UserController {
return ResponseEntity.ok(userMapper.toDto(users)); return ResponseEntity.ok(userMapper.toDto(users));
} }
@GetMapping("/delete/{username}")
@PreAuthorize("hasRole('ADMIN')")
public ResponseEntity<Void> deleteUserByUsername(@PathVariable String username) {
userService.deleteByUsername(username);
return ResponseEntity.status(HttpStatus.OK).build();
}
} }

View File

@@ -1,48 +1,48 @@
package com.faf223.expensetrackerfaf.model; package com.faf223.expensetrackerfaf.model;
import jakarta.persistence.*; import jakarta.persistence.*;
import jakarta.validation.constraints.NotEmpty; import jakarta.validation.constraints.NotEmpty;
import jakarta.validation.constraints.NotNull; import jakarta.validation.constraints.NotNull;
import lombok.*; import lombok.*;
import java.util.List; import java.util.List;
@Entity(name = "users") @Entity(name = "users")
@Data @Data
@Builder @Builder
@NoArgsConstructor @NoArgsConstructor
@AllArgsConstructor @AllArgsConstructor
public class User { public class User {
@Id @Id
@Column(name = "user_uuid") @Column(name = "user_uuid")
@GeneratedValue(strategy = GenerationType.UUID) @GeneratedValue(strategy = GenerationType.UUID)
private String userUuid; private String userUuid;
@Column(name = "name") @Column(name = "name")
@NotNull(message = "First name must not be null") @NotNull(message = "First name must not be null")
@NotEmpty(message = "First name must not be empty") @NotEmpty(message = "First name must not be empty")
private String firstName; private String firstName;
@Column(name = "surname") @Column(name = "surname")
@NotNull(message = "Last name must not be null") @NotNull(message = "Last name must not be null")
@NotEmpty(message = "Last name must not be empty") @NotEmpty(message = "Last name must not be empty")
private String lastName; private String lastName;
@Column(name = "username") @Column(name = "username")
@NotNull(message = "Username must not be null") @NotNull(message = "Username must not be null")
@NotEmpty(message = "Username must not be empty") @NotEmpty(message = "Username must not be empty")
private String username; private String username;
@Transient @Transient
// @NotNull(message = "Password must not be null") // @NotNull(message = "Password must not be null")
// @NotEmpty(message = "Password must not be empty") // @NotEmpty(message = "Password must not be empty")
private String password; private String password;
@OneToMany(mappedBy = "user", fetch = FetchType.LAZY) @OneToMany(mappedBy = "user", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
@ToString.Exclude @ToString.Exclude
private List<Expense> expenses; private List<Expense> expenses;
@OneToMany(mappedBy = "user", fetch = FetchType.LAZY) @OneToMany(mappedBy = "user", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
@ToString.Exclude @ToString.Exclude
private List<Income> incomes; private List<Income> incomes;
} }

View File

@@ -12,4 +12,6 @@ public interface CredentialRepository extends JpaRepository<Credential, Long> {
Optional<Credential> findByEmail(String email); Optional<Credential> findByEmail(String email);
Optional<Credential> findByUser(User user); Optional<Credential> findByUser(User user);
void deleteByEmail(String email);
} }

View File

@@ -9,4 +9,6 @@ public interface UserRepository extends JpaRepository<User, String> {
Optional<User> getUserByUserUuid(String userUuid); Optional<User> getUserByUserUuid(String userUuid);
Optional<User> findByUsername(String username); Optional<User> findByUsername(String username);
void deleteByUsername(String username);
} }

View File

@@ -4,6 +4,7 @@ import com.faf223.expensetrackerfaf.model.Credential;
import com.faf223.expensetrackerfaf.model.User; import com.faf223.expensetrackerfaf.model.User;
import com.faf223.expensetrackerfaf.repository.CredentialRepository; import com.faf223.expensetrackerfaf.repository.CredentialRepository;
import com.faf223.expensetrackerfaf.repository.UserRepository; import com.faf223.expensetrackerfaf.repository.UserRepository;
import jakarta.transaction.Transactional;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Service; import org.springframework.stereotype.Service;
@@ -36,4 +37,20 @@ public class UserService {
} }
return null; return null;
} }
@Transactional
public void deleteByUsername(String username) {
Optional<User> user = userRepository.findByUsername(username);
if (user.isPresent()) {
Optional<Credential> credential = credentialRepository.findByUser(user.get());
if (credential.isPresent()) {
credentialRepository.deleteByEmail(credential.get().getEmail());
userRepository.deleteByUsername(username);
}
}
}
} }