Skip to content

Commit 6789a7a

Browse files
Refactor: Rename orders entity to order
1 parent 9723858 commit 6789a7a

File tree

8 files changed

+39
-37
lines changed

8 files changed

+39
-37
lines changed

src/main/java/com/sample/ecommerce/order/controller/OrderController.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
import org.springframework.web.bind.annotation.RestController;
1414

1515
import com.sample.ecommerce.order.dto.OrderDto;
16-
import com.sample.ecommerce.order.model.Orders;
16+
import com.sample.ecommerce.order.model.Order;
1717
import com.sample.ecommerce.order.service.OrderService;
1818

1919
import jakarta.validation.Valid;
@@ -30,21 +30,21 @@ public class OrderController {
3030
*/
3131

3232
@PostMapping()
33-
public Orders create(@Valid @RequestBody OrderDto createOrder){
33+
public Order create(@Valid @RequestBody OrderDto createOrder){
3434
return service.create(createOrder);
3535
}
3636

3737
@GetMapping("/{id}")
38-
public Orders findOne(@PathVariable int id){
38+
public Order findOne(@PathVariable int id){
3939
return service.findOne(id);
4040
}
4141
@GetMapping()
42-
public List<Orders> findAll(){
42+
public List<Order> findAll(){
4343
return service.findAll();
4444
}
4545

4646
@PutMapping("/{id}")
47-
public Orders update(@PathVariable int id, @Valid @RequestBody OrderDto updateOrder){
47+
public Order update(@PathVariable int id, @Valid @RequestBody OrderDto updateOrder){
4848
return service.update(id,updateOrder);
4949
}
5050

src/main/java/com/sample/ecommerce/order/model/Customer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,6 @@ public class Customer {
5454

5555
@OneToMany(mappedBy = "customer")
5656
@JsonBackReference
57-
private List<Orders> orders;
57+
private List<Order> orders;
5858

5959
}

src/main/java/com/sample/ecommerce/order/model/Orders.java renamed to src/main/java/com/sample/ecommerce/order/model/Order.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import jakarta.persistence.JoinColumn;
66
import jakarta.persistence.ManyToOne;
77
import jakarta.persistence.OneToMany;
8+
import jakarta.persistence.Table;
89
import jakarta.persistence.GenerationType;
910

1011
import java.time.LocalDateTime;
@@ -33,7 +34,8 @@
3334
@NoArgsConstructor
3435
@Builder(toBuilder=true)
3536
@Entity
36-
public class Orders {
37+
@Table(name="\"order\"")
38+
public class Order {
3739

3840
@Id
3941
@GeneratedValue(strategy = GenerationType.IDENTITY)

src/main/java/com/sample/ecommerce/order/model/OrderItem.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public class OrderItem {
3535
@ManyToOne
3636
@JoinColumn(name = "order_id")
3737
@JsonBackReference
38-
private Orders order;
38+
private Order order;
3939

4040
@ManyToOne
4141
@JoinColumn(name = "product_id")
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package com.sample.ecommerce.order.repository;
22

33
import org.springframework.data.jpa.repository.JpaRepository;
4-
import com.sample.ecommerce.order.model.Orders;
4+
import com.sample.ecommerce.order.model.Order;
55

6-
public interface OrderRepository extends JpaRepository<Orders, Integer> {
6+
public interface OrderRepository extends JpaRepository<Order, Integer> {
77

88
}

src/main/java/com/sample/ecommerce/order/service/OrderService.java

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import com.sample.ecommerce.order.dto.OrderDto;
1111
import com.sample.ecommerce.order.dto.OrderItemDto;
1212
import com.sample.ecommerce.order.exception.ObjectNotFoundException;
13-
import com.sample.ecommerce.order.model.Orders;
13+
import com.sample.ecommerce.order.model.Order;
1414
import com.sample.ecommerce.order.model.Customer;
1515
import com.sample.ecommerce.order.model.OrderItem;
1616
import com.sample.ecommerce.order.model.Product;
@@ -34,20 +34,20 @@ public class OrderService {
3434
ProductRepository productRepository;
3535

3636

37-
public Orders create(OrderDto orderDto) {
37+
public Order create(OrderDto orderDto) {
3838

3939
if(customerRepository.findById(orderDto.getCustomerId()).isEmpty())
4040
throw new ObjectNotFoundException(Customer.class);
4141

4242
log.info("Creating order for customer - {}" , orderDto.getCustomerId());
43-
Orders order= createOrderfromDto(new Orders(),orderDto);
44-
Orders createdOrder = repository.save(order);
43+
Order order= createOrderfromDto(new Order(),orderDto);
44+
Order createdOrder = repository.save(order);
4545
log.info("Order details : order_id - {}, total_price - {}, total_quantity - {}, customer_id - {}, order_status - {}" ,order.getId(),order.getTotalPrice(),order.getTotalQuantity(),orderDto.getCustomerId(),order.getStatus());
4646
return createdOrder;
4747
}
4848

4949

50-
public Orders createOrderfromDto(Orders order,OrderDto orderDto) {
50+
public Order createOrderfromDto(Order order,OrderDto orderDto) {
5151
int totalQuantity = 0;
5252
int totalPrice = 0;
5353
List<OrderItem> orderItems = new ArrayList<OrderItem>();
@@ -76,30 +76,30 @@ public Orders createOrderfromDto(Orders order,OrderDto orderDto) {
7676

7777
}
7878

79-
public Orders findOne(int id) {
79+
public Order findOne(int id) {
8080
try{
8181
return repository.findById(id).get();
8282
}
8383
catch(Exception e)
8484
{
85-
throw new ObjectNotFoundException(Orders.class,"order");
85+
throw new ObjectNotFoundException(Order.class,"order");
8686
}
8787

8888
}
89-
public List<Orders> findAll() {
89+
public List<Order> findAll() {
9090
return repository.findAll();
9191
}
9292

9393
public void delete(int id){
94-
Orders order = this.findOne(id);
94+
Order order = this.findOne(id);
9595
repository.delete(order);
9696
log.info("Deleted order with order_id - {}",id);
9797
}
9898

99-
public Orders update(Integer orderId, OrderDto updateOrderDto) {
100-
Orders order = this.findOne(orderId);
99+
public Order update(Integer orderId, OrderDto updateOrderDto) {
100+
Order order = this.findOne(orderId);
101101
log.info("Updating order with order_id - {} ",orderId);
102-
Orders updatedOrder= createOrderfromDto(order, updateOrderDto);
102+
Order updatedOrder= createOrderfromDto(order, updateOrderDto);
103103
log.info("Order details : order_id - {}, total_price - {}, total_quantity - {}, customer_id - {}, order_status - {}" ,order.getId(),order.getTotalPrice(),order.getTotalQuantity(),order.getCustomer().getId(),order.getStatus());
104104
return repository.save(updatedOrder);
105105
}
Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
create table orders.category (id serial not null, name varchar(255), parent_category integer, primary key (id));
2-
create table orders.customer (id serial not null, city varchar(255), country_code varchar(255), created_at timestamp(6) default(current_timestamp), email varchar(255), name varchar(255), phone varchar(255), profile_pic varchar(255), state varchar(255), street varchar(255), primary key (id));
3-
create table orders.orders (id serial not null, created_at timestamp(6) default(current_timestamp), status varchar(255), updated_at timestamp(6) default(current_timestamp), customer_id integer, total_quantity integer, total_price integer, primary key (id));
4-
create table orders.order_item (id serial not null, quantity integer, order_id integer, product_id integer, primary key (id));
5-
create table orders.product (id serial not null, description varchar(255), image varchar(255), name varchar(255), unit_price integer, category_id integer, primary key (id));
6-
alter table if exists orders.orders add constraint FKb8tboo4d95mh8gavvovwbb7vg foreign key (customer_id) references orders.customer;
7-
alter table if exists order_item add constraint FKt6wv8m7eshksp5kp8w4b2d1dm foreign key (order_id) references orders.orders;
8-
alter table if exists order_item add constraint FK551losx9j75ss5d6bfsqvijna foreign key (product_id) references orders.product;
9-
alter table if exists product add constraint FK1mtsbur82frn64de7balymq9s foreign key (category_id) references orders.category;
1+
create table category (id serial not null, name varchar(255), parent_category integer, primary key (id));
2+
create table customer (id serial not null, city varchar(255), country_code varchar(255), created_at timestamp(6), email varchar(255), name varchar(255), phone varchar(255), profile_pic varchar(255), state varchar(255), street varchar(255), primary key (id));
3+
create table "order" (id serial not null, created_at timestamp(6), status varchar(255), total_price integer not null, total_quantity integer not null, updated_at timestamp(6), customer_id integer, primary key (id));
4+
create table order_item (id serial not null, quantity integer, order_id integer, product_id integer, primary key (id));
5+
create table product (id serial not null, description varchar(255), image varchar(255), name varchar(255), unit_price integer, category_id integer, primary key (id));
6+
alter table if exists "order" add constraint FK1oduxyuuo3n2g98l3j7754vym foreign key (customer_id) references customer;
7+
alter table if exists order_item add constraint FKs234mi6jususbx4b37k44cipy foreign key (order_id) references "order";
8+
alter table if exists order_item add constraint FK551losx9j75ss5d6bfsqvijna foreign key (product_id) references product;
9+
alter table if exists product add constraint FK1mtsbur82frn64de7balymq9s foreign key (category_id) references category;

src/test/java/com/sample/ecommerce/order/OrderServiceTests.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import com.sample.ecommerce.order.exception.ObjectNotFoundException;
2424
import com.sample.ecommerce.order.model.Category;
2525
import com.sample.ecommerce.order.model.Customer;
26-
import com.sample.ecommerce.order.model.Orders;
26+
import com.sample.ecommerce.order.model.Order;
2727
import com.sample.ecommerce.order.model.OrderItem;
2828
import com.sample.ecommerce.order.model.Product;
2929
import com.sample.ecommerce.order.repository.CustomerRepository;
@@ -53,14 +53,14 @@ void getOrderByIdTest() {
5353
int randomId = 2;
5454
final int totalPrice = 20;
5555
final int quantity = 4;
56-
Orders order = Orders.builder()
56+
Order order = Order.builder()
5757
.id(randomId)
5858
.totalQuantity(quantity)
5959
.totalPrice(totalPrice)
6060
.build();
6161

6262
Mockito.when(orderRepository.findById(anyInt())).thenReturn(Optional.of(order));
63-
Orders obtainedOrder = orderService.findOne(1);
63+
Order obtainedOrder = orderService.findOne(1);
6464
assertEquals(obtainedOrder, order);
6565
}
6666

@@ -78,7 +78,7 @@ void createdOrderTest(){
7878
Mockito.when(productRepository.findById(productId))
7979
.thenReturn(Optional.of(getSampleProduct(productId)));
8080

81-
Orders order = Orders.builder()
81+
Order order = Order.builder()
8282
.id(randomId)
8383
.customer(getSampleCustomer(customerId))
8484
.status(OrderStatus.INITIATED)
@@ -96,7 +96,7 @@ void createdOrderTest(){
9696
.build();
9797

9898
Mockito.when(orderRepository.save(any())).thenReturn(order);
99-
Orders obtainedOrder = orderService.create(orderDto);
99+
Order obtainedOrder = orderService.create(orderDto);
100100
assertEquals(obtainedOrder, order);
101101
}
102102

@@ -157,7 +157,7 @@ private Category getSampleCategory(){
157157
.build();
158158
}
159159

160-
private List<OrderItem> getSampleOrderItems(Orders order,int productId, int quantity){
160+
private List<OrderItem> getSampleOrderItems(Order order,int productId, int quantity){
161161
List<OrderItem> orderItems = new ArrayList<OrderItem>();
162162
orderItems.add(OrderItem.builder().order(order).quantity(quantity).product(getSampleProduct(productId)).build());
163163
return orderItems;

0 commit comments

Comments
 (0)