Skip to content

Commit

Permalink
Merge pull request #424 from Adyen/feature/AD-276
Browse files Browse the repository at this point in the history
Feature/ad 276
  • Loading branch information
pjaneta authored Jul 18, 2024
2 parents e86902f + d1bcd67 commit 9a06882
Show file tree
Hide file tree
Showing 7 changed files with 111 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.ResponseEntity;
import org.springframework.security.access.annotation.Secured;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RestController;

@Controller
@RestController
@RequestMapping(value = AdyenoccConstants.ADYEN_USER_CART_PREFIX)
@ApiVersion("v2")
@Tag(name = "Adyen")
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
package com.adyen.commerce.controllers;

import com.adyen.commerce.constants.AdyenoccConstants;
import com.adyen.v6.facades.AdyenOrderFacade;
import de.hybris.platform.commerceservices.request.mapping.annotation.ApiVersion;
import de.hybris.platform.commercewebservices.core.strategies.OrderCodeIdentificationStrategy;
import de.hybris.platform.webservicescommons.swagger.ApiBaseSiteIdAndUserIdParam;
import io.swagger.v3.oas.annotations.Operation;
import io.swagger.v3.oas.annotations.Parameter;
import io.swagger.v3.oas.annotations.tags.Tag;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.ResponseEntity;
import org.springframework.security.access.annotation.Secured;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RestController;

@RestController
@RequestMapping(value = AdyenoccConstants.ADYEN_USER_PREFIX)
@ApiVersion("v2")
@Tag(name = "Adyen")
public class PaymentStatusController {

@Autowired
private OrderCodeIdentificationStrategy orderCodeIdentificationStrategy;

@Autowired
private AdyenOrderFacade adyenOrderFacade;


@Secured({"ROLE_CUSTOMERGROUP", "ROLE_CLIENT", "ROLE_TRUSTED_CLIENT", "ROLE_CUSTOMERMANAGERGROUP"})
@GetMapping(value = "/payment-status/{orderCode}")
@Operation(operationId = "getPaymentStatus", summary = "Get order payment status.", description = "Returns payment status of order with given code.")
@ApiBaseSiteIdAndUserIdParam
public ResponseEntity<String> getPaymentStatus(
@Parameter(description = "Order GUID (Globally Unique Identifier) or order CODE", required = true) @PathVariable final String orderCode) {
try {
String paymentStatus = adyenOrderFacade.getPaymentStatusOCC(orderCode);
return ResponseEntity.ok(paymentStatus);
} catch (Exception e) {
return ResponseEntity.badRequest().build();
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import com.adyen.commerce.controllerbase.PlaceOrderControllerBase;
import com.adyen.commerce.facades.AdyenCheckoutApiFacade;
import com.adyen.commerce.request.PlaceOrderRequest;
import com.adyen.commerce.response.OCCPlaceOrderResponse;
import com.adyen.commerce.response.PlaceOrderResponse;
import com.adyen.model.checkout.PaymentDetailsRequest;
import com.adyen.v6.facades.AdyenCheckoutFacade;
Expand All @@ -22,15 +23,15 @@
import org.springframework.http.MediaType;
import org.springframework.http.ResponseEntity;
import org.springframework.security.access.annotation.Secured;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RestController;

import javax.annotation.Resource;
import javax.servlet.http.HttpServletRequest;

@Controller
@RestController
@RequestMapping(value = AdyenoccConstants.ADYEN_USER_CART_PREFIX)
@ApiVersion("v2")
@Tag(name = "Adyen")
Expand Down Expand Up @@ -61,7 +62,7 @@ public class PlaceOrderController extends PlaceOrderControllerBase {
@ApiBaseSiteIdUserIdAndCartIdParam
public ResponseEntity<String> onPlaceOrder(@RequestBody String placeOrderStringRequest, HttpServletRequest request) throws Exception {
PlaceOrderRequest placeOrderRequest = objectMapper.readValue(placeOrderStringRequest, PlaceOrderRequest.class);
PlaceOrderResponse placeOrderResponse = super.placeOrder(placeOrderRequest, request);
OCCPlaceOrderResponse placeOrderResponse = super.placeOrderOCC(placeOrderRequest, request);
String response = objectMapper.writeValueAsString(placeOrderResponse);
return ResponseEntity.ok(response);
}
Expand Down
1 change: 1 addition & 0 deletions adyenv6core/src/com/adyen/v6/facades/AdyenOrderFacade.java
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,5 @@

public interface AdyenOrderFacade {
String getPaymentStatus(final String orderCode, final String sessionGuid);
String getPaymentStatusOCC(final String code);
}
Original file line number Diff line number Diff line change
Expand Up @@ -32,15 +32,44 @@ public class DefaultAdyenOrderFacade implements AdyenOrderFacade {

@Override
public String getPaymentStatus(final String orderCode, final String sessionGuid) {
OrderModel orderModel = getOrderDetailsForCodeInternal(orderCode, sessionGuid);
OrderModel orderModel = getOrderModelForCode(orderCode, sessionGuid);

return getPaymentStatusForOrder(orderModel);
}

@Override
public String getPaymentStatusOCC(final String code) {
final OrderModel orderModel = getOrderModelForCodeOCC(code);

return getPaymentStatusForOrder(orderModel);
}

private String getPaymentStatusForOrder(final OrderModel orderModel) {
List<PaymentTransactionModel> paymentTransactions = orderModel.getPaymentTransactions();
if (paymentTransactions.isEmpty()) {
return getMessageFromStatus(TransactionStatus.REVIEW.name());
}
return getStatus(paymentTransactions);
}

private OrderModel getOrderDetailsForCodeInternal(final String code, final String sessionGuid) {
private OrderModel getOrderModelForCodeOCC(String code) {
BaseStoreModel currentBaseStore = baseStoreService.getCurrentBaseStore();
final OrderModel orderModel;

if (checkoutCustomerStrategy.isAnonymousCheckout()) {
orderModel = customerAccountService.getGuestOrderForGUID(code,
currentBaseStore);
} else {
orderModel = customerAccountService.getOrderForCode((CustomerModel) userService.getCurrentUser(), code, currentBaseStore);
}

if (orderModel == null) {
throw new UnknownIdentifierException(String.format(ORDER_NOT_FOUND_FOR_USER_AND_BASE_STORE, code));
}
return orderModel;
}

private OrderModel getOrderModelForCode(final String code, final String sessionGuid) {
final BaseStoreModel baseStoreModel = baseStoreService.getCurrentBaseStore();

OrderModel orderModel = null;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import com.adyen.commerce.exception.AdyenControllerException;
import com.adyen.commerce.facades.AdyenCheckoutApiFacade;
import com.adyen.commerce.request.PlaceOrderRequest;
import com.adyen.commerce.response.OCCPlaceOrderResponse;
import com.adyen.commerce.response.PlaceOrderResponse;
import com.adyen.commerce.validators.PaymentRequestValidator;
import com.adyen.model.checkout.PaymentDetailsRequest;
Expand Down Expand Up @@ -48,6 +49,13 @@ public PlaceOrderControllerBase() {

public PlaceOrderResponse placeOrder(PlaceOrderRequest placeOrderRequest, HttpServletRequest request) {

OCCPlaceOrderResponse occPlaceOrderResponse = placeOrderOCC(placeOrderRequest, request);
occPlaceOrderResponse.setOrderData(null);
return occPlaceOrderResponse;
}

public OCCPlaceOrderResponse placeOrderOCC(PlaceOrderRequest placeOrderRequest, HttpServletRequest request) {

String adyenPaymentMethodType = extractPaymentMethodType(placeOrderRequest);

preHandleAndValidateRequest(placeOrderRequest, adyenPaymentMethodType);
Expand Down Expand Up @@ -151,7 +159,7 @@ private boolean isCartValid() {
return true;
}

private PlaceOrderResponse handlePayment(HttpServletRequest request, PlaceOrderRequest placeOrderRequest) {
private OCCPlaceOrderResponse handlePayment(HttpServletRequest request, PlaceOrderRequest placeOrderRequest) {
final CartData cartData = getCartFacade().getSessionCart();

String errorMessage = CHECKOUT_ERROR_AUTHORIZATION_FAILED;
Expand All @@ -160,8 +168,9 @@ private PlaceOrderResponse handlePayment(HttpServletRequest request, PlaceOrder
cartData.setAdyenReturnUrl(getPaymentRedirectReturnUrl());
OrderData orderData = getAdyenCheckoutApiFacade().placeOrderWithPayment(request, cartData, placeOrderRequest.getPaymentRequest());

PlaceOrderResponse placeOrderResponse = new PlaceOrderResponse();
OCCPlaceOrderResponse placeOrderResponse = new OCCPlaceOrderResponse();
placeOrderResponse.setOrderNumber(orderData.getCode());
placeOrderResponse.setOrderData(orderData);
return placeOrderResponse;

} catch (ApiException e) {
Expand All @@ -187,8 +196,8 @@ private PlaceOrderResponse handlePayment(HttpServletRequest request, PlaceOrder
throw new AdyenControllerException(errorMessage);
}

private PlaceOrderResponse executeAction(PaymentResponse paymentsResponse) {
PlaceOrderResponse placeOrderResponse = new PlaceOrderResponse();
private OCCPlaceOrderResponse executeAction(PaymentResponse paymentsResponse) {
OCCPlaceOrderResponse placeOrderResponse = new OCCPlaceOrderResponse();
placeOrderResponse.setPaymentsResponse(paymentsResponse);
placeOrderResponse.setExecuteAction(true);
placeOrderResponse.setPaymentsAction(paymentsResponse.getAction());
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
package com.adyen.commerce.response;

import de.hybris.platform.commercefacades.order.data.OrderData;

public class OCCPlaceOrderResponse extends PlaceOrderResponse {
private OrderData orderData;

public OrderData getOrderData() {
return orderData;
}

public void setOrderData(OrderData orderData) {
this.orderData = orderData;
}
}

0 comments on commit 9a06882

Please sign in to comment.