Review ntoes

Author: Karolina Karolak

src/main/java/com/capgemini/training/appointmentbooking/dataaccess/entity/AppointmentEntity.java

@@ -1,6 +1,6 @@
 package com.capgemini.training.appointmentbooking.dataaccess.entity;
 
-import java.util.Date;
+import java.time.LocalDateTime;
 
 import com.capgemini.training.appointmentbooking.common.datatype.AppointmentStatus;
 
@@ -36,7 +36,7 @@ public class AppointmentEntity extends BaseEntity {
 	private TreatmentEntity treatment;
 
 	@Column(name="DATE_TIME")
-	private Date dateTime;
+	private LocalDateTime dateTime;
 
 	@Enumerated(EnumType.STRING)
 	private AppointmentStatus status;

src/main/java/com/capgemini/training/appointmentbooking/dataaccess/entity/BaseEntity.java

@@ -1,5 +1,6 @@
 package com.capgemini.training.appointmentbooking.dataaccess.entity;
 
+import java.time.Instant;
 import java.util.Date;
 
 import jakarta.persistence.Column;
@@ -12,26 +13,27 @@
 
 @MappedSuperclass
 @Getter
-@Setter
 public class BaseEntity {
 
 	@Version
+	@Setter
 	private int version;
 
 	@Column(insertable = true, updatable = false)
-	private Date created;
+	private Instant created;
 
 	@Column(name="LAST_UPDATED")
-	private Date lastUpdated;
+	private Instant lastUpdated;
 
 	@PrePersist
 	public void prePersist() {
-		setCreated(new Date());
-		setLastUpdated(new Date());
+		Instant now = Instant.now();
+		this.created = now;
+		this.lastUpdated = now;
 	}
 
 	@PreUpdate
 	public void preUpdate() {
-		setLastUpdated(new Date());
+		this.lastUpdated = Instant.now();
 	}
 }

src/main/java/com/capgemini/training/appointmentbooking/dataaccess/entity/ClientEntity.java

@@ -28,6 +28,7 @@ public class ClientEntity  extends BaseEntity {
 
 	@OneToOne(optional = false, fetch = FetchType.LAZY, cascade = { CascadeType.PERSIST })
 	private UserEntity user;
+	
 	@OneToMany(mappedBy = "client", fetch = FetchType.LAZY, orphanRemoval=true, cascade = { CascadeType.PERSIST })
 	private List<AppointmentEntity> appointments;