From fff721dd185f5b97f7ae7a6e64189e55887ff590 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Sun, 05 Jul 2026 18:36:57 +0000
Subject: [PATCH] =Cleaning up the Square integration (still a bit more to do yet). Also majorly overhauled /rest/ files to ignore a rest request 'user' paramater, and rely on get_current_user_id() instead.
---
inc/rest/routes/IntegrationsSquareRoutes.php | 65 +++++++++++---------------------
1 files changed, 22 insertions(+), 43 deletions(-)
diff --git a/inc/rest/routes/IntegrationsSquareRoutes.php b/inc/rest/routes/IntegrationsSquareRoutes.php
index 596be1e..f60dc34 100644
--- a/inc/rest/routes/IntegrationsSquareRoutes.php
+++ b/inc/rest/routes/IntegrationsSquareRoutes.php
@@ -2,6 +2,7 @@
namespace JVBase\rest\routes;
+use JVBase\integrations\Square;
use JVBase\meta\Meta;
use JVBase\rest\Rest;
use Exception;
@@ -20,22 +21,26 @@
Route::for('square/process-payment')
->post([$this, 'handlePaymentProcessing'])
->auth('public')
- ->rateLimit(2);
+ ->rateLimit(2)
+ ->register();
Route::for('square/saved-cards')
->post([$this, 'getSavedCards'])
->auth('user')
- ->rateLimit(5);
+ ->rateLimit(5)
+ ->register();
Route::for('square/order-history')
->get([$this, 'getOrderHistory'])
->auth('user')
- ->rateLimit(5);
+ ->rateLimit(5)
+ ->register();
Route::for(Route::pattern('square/order-status/{order_id}'))
->get([$this, 'getOrderStatus'])
->auth('public')
- ->rateLimit(20);
+ ->rateLimit(20)
+ ->register();
}
//TODO: Are we processing this through our server at all? Or is it in the javascript going straight to square?
@@ -43,6 +48,7 @@
{
$data = $request->get_json_params();
+
// Generate idempotency key from cart_id + timestamp
// This ensures retries use SAME key
$cart_id = $data['cart_id'] ?? '';
@@ -73,6 +79,7 @@
}
try {
+ /** @var Square $square */
$square = JVB()->connect('square');
// Step 1: Get or create Square customer
$customer_id = $square->getOrCreateSquareCustomer($data['customer']);
@@ -92,7 +99,7 @@
// Step 3: Create Payment in Square
$payment_response = $square->createSquarePayment(
$data['source_id'],
- $data['idempotency_key'],
+ $idempotency_key,
$data['amount'],
$order_id,
$customer_id
@@ -142,12 +149,12 @@
public function getSavedCards(WP_REST_Request $request):WP_REST_Response
{
$data = $request->get_params();
- $user_id = absint($data['user']??0);
- if ($user_id === 0) {
- return $this->validationError(['message' => 'Not logged in']);
+ error_log('Getting Saved Cards: '.print_r($data, true));
+ $user_id = get_current_user_id();
+ if (!$user_id) {
+ return $this->success(['cards' => []]);
}
- $square = JVB()->connect('square');
// Get Square customer ID for this user
$square_customer_id = get_user_meta($user_id, BASE . '_square_customer_id', true);
@@ -156,58 +163,30 @@
return $this->success(['cards' => []]);
}
- // Fetch cards from Square (2025-compliant - separate endpoint)
- $cards_response = $square->getRequest('cards?customer_id=' . $square_customer_id);
-
- if (is_wp_error($cards_response)) {
- return $this->error('Failed to fetch cards');
- }
-
- return $this->success(['cards' => $cards_response['cards']??[]]);
+ return $this->success(['cards' => Square::getInstance()->getUserCards( $square_customer_id)]);
}
public function getOrderHistory(WP_REST_Request $request):WP_REST_Response
{
- $data = $request->get_params();
- $user_id = absint($data['user']??0);
- if ($user_id === 0) {
+ $user_id = get_current_user_id();
+ if (!$user_id) {
return $this->validationError(['message' => 'Not logged in']);
}
- // Get orders from custom post type
- $orders = get_posts([
- 'post_type' => BASE . '_sq_orders',
- 'author' => $user_id,
- 'posts_per_page' => 50,
- 'orderby' => 'date',
- 'order' => 'DESC'
- ]);
-
- $order_data = [];
- foreach ($orders as $order) {
- $meta = Meta::forPost($order->ID);
- $fields = $meta->getAll(['square_order_id', 'status', 'amount', 'items', 'created_at', 'pickup_time']);
- $order_data[] = array_merge([
- 'wp_order_id' => $order->ID,
- ], $fields);
- }
-
- return $this->success(['orders' => $order_data]);
+ return $this->success(['orders' => Square::getInstance()->getOrderHistory($user_id)]);
}
public function getOrderStatus(WP_REST_Request $request):WP_REST_Response
{
$order_id = $request->get_param('order_id');
-
- // Find WP post by Square order ID
- $wp_order_id = get_option(BASE . 'square_order_map_' . $order_id);
+ $wp_order_id = Square::getInstance()->getOrderPost($order_id);
if (!$wp_order_id) {
return $this->error('Order not found');
}
$meta = Meta::forPost($wp_order_id);
- $fields = $meta->getAll(['status', 'fulfillment_status', 'pickup_time', 'items']);
+ $fields = $meta->getAll(['square_payment_status', 'fulfillment_status', 'pickup_time', 'items']);
return $this->success(['order' => $fields]);
}
--
Gitblit v1.10.0