strapi-plugin-payone-provider 1.6.7 → 5.6.9

package/admin/src/pages/utils/transactionTableUtils.js

@@ -0,0 +1,60 @@
+
+export const getStatusColor = (status) => {
+  switch (status) {
+    case "APPROVED":
+      return "success200";
+    case "ERROR":
+      return "danger200";
+    case "PENDING":
+      return "warning200";
+    case "REDIRECT":
+      return "success100";
+    default:
+      return "success100";
+  }
+};
+
+export const formatAmount = (amount, currency) => {
+  if (!amount) return "N/A";
+  return `${(amount / 100).toFixed(2)} ${currency || "EUR"}`;
+};
+
+export const formatDate = (dateString) => {
+  if (!dateString) return "N/A";
+  return new Date(dateString).toLocaleString("en-US", {
+    year: "numeric",
+    month: "2-digit",
+    day: "2-digit",
+    hour: "2-digit",
+    minute: "2-digit",
+  });
+};
+
+export const getPaymentMethodName = (clearingtype, wallettype) => {
+  switch (clearingtype) {
+    case "cc":
+      return "Credit Card";
+    case "sb":
+      return "Online Banking";
+    case "wlt":
+      return wallettype === "PPE" ? "PayPal" : "Wallet";
+    case "elv":
+      return "Direct Debit (SEPA)";
+    default:
+      return clearingtype || "Unknown";
+  }
+};
+
+export const getCardTypeName = (cardtype) => {
+  switch (cardtype) {
+    case "V":
+      return "Visa";
+    case "M":
+      return "Mastercard";
+    case "A":
+      return "American Express";
+    default:
+      return cardtype || "Unknown";
+  }
+};
+

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "strapi-plugin-payone-provider",
-  "version": "1.6.7",
+  "version": "5.6.9",
   "description": "Strapi plugin for Payone payment gateway integration",
   "license": "MIT",
   "maintainers": [
@@ -10,22 +10,17 @@
     }
   ],
   "dependencies": {
+    "@uiw/react-json-view": "^2.0.0-alpha.40",
     "apple-pay-button": "^1.2.1",
     "axios": "^1.6.3",
     "prop-types": "^15.7.2"
   },
-  "devDependencies": {
-    "react": "^18.2.0",
-    "react-dom": "^18.2.0",
-    "react-router-dom": "^5.3.4",
-    "styled-components": "^5.3.6"
-  },
   "peerDependencies": {
-    "@strapi/strapi": "^4.0",
-    "react": "^17.0.0 || ^18.0.0",
-    "react-dom": "^17.0.0 || ^18.0.0",
-    "react-router-dom": "^5.2.0",
-    "styled-components": "^5.2.1"
+    "@strapi/strapi": "^5.0.0",
+    "react": "^18.0.0",
+    "react-dom": "^18.0.0",
+    "react-router-dom": "^6.0.0",
+    "styled-components": "^6.0.0"
   },
   "engines": {
     "node": ">=18.0.0",

package/server/config/index.js

@@ -14,7 +14,13 @@ module.exports = {
       merchantName: "",
       displayName: "",
       domainName: "",
-      merchantIdentifier: ""
+      merchantIdentifier: "",
+      enableCreditCard: false,
+      enablePayPal: false,
+      enableGooglePay: false,
+      enableApplePay: false,
+      enableSofort: false,
+      enableSepaDirectDebit: false
     }
   },
   validator(config) {
@@ -33,7 +39,17 @@ module.exports = {
           api_version: yup
             .string()
             .matches(/^\d+\.\d+$/)
-            .defined()
+            .defined(),
+          merchantName: yup.string().optional(),
+          displayName: yup.string().optional(),
+          domainName: yup.string().optional(),
+          merchantIdentifier: yup.string().optional(),
+          enableCreditCard: yup.boolean().optional(),
+          enablePayPal: yup.boolean().optional(),
+          enableGooglePay: yup.boolean().optional(),
+          enableApplePay: yup.boolean().optional(),
+          enableSofort: yup.boolean().optional(),
+          enableSepaDirectDebit: yup.boolean().optional()
         })
         .defined()
     });

package/server/controllers/payone.js

@@ -7,13 +7,25 @@ const getPayoneService = (strapi) => {
 };
 
 const handleError = (ctx, error) => {
-  if (error.response || error.status >= 400) {
-    ctx.strapi.log.error("Payone controller error:", {
-      status: error.status || error.response?.status,
-      message: error.message
+  const status = error.status || error.response?.status || 500;
+  const message = error.message || "Internal server error";
+
+  if (status >= 400) {
+    console.log("[Payone] Controller error:", {
+      status,
+      message,
+      error: error.stack || error
     });
   }
-  ctx.throw(500, error);
+
+  ctx.status = status;
+  ctx.body = {
+    error: {
+      status,
+      message,
+      name: error.name || "Error"
+    }
+  };
 };
 
 const hideKey = (settings) => {
@@ -27,7 +39,9 @@ module.exports = ({ strapi }) => ({
   async getSettings(ctx) {
     try {
       const settings = await getPayoneService(strapi).getSettings();
-      ctx.body = { data: hideKey(settings) };
+      ctx.body = {
+        ...hideKey(settings || {})
+      };
     } catch (error) {
       handleError(ctx, error);
     }
@@ -62,15 +76,20 @@ module.exports = ({ strapi }) => ({
 
   async updateSettings(ctx) {
     try {
-      const { body } = ctx.request;
+      const bodyData = ctx.request.body?.data || ctx.request.body;
+
+      if (!bodyData || typeof bodyData !== 'object') {
+        ctx.throw(400, "Invalid request body");
+      }
+
       const currentSettings = await getPayoneService(strapi).getSettings();
 
-      if (body.key === "***HIDDEN***" || !body.key) {
-        body.key = currentSettings?.key;
+      if (bodyData.key === "***HIDDEN***" || !bodyData.key) {
+        bodyData.key = currentSettings?.key;
       }
 
-      const settings = await getPayoneService(strapi).updateSettings(body);
-      ctx.body = { data: hideKey(settings) };
+      const settings = await getPayoneService(strapi).updateSettings(bodyData);
+      ctx.body = { ...hideKey(settings) };
     } catch (error) {
       handleError(ctx, error);
     }
@@ -78,9 +97,13 @@ module.exports = ({ strapi }) => ({
 
   async preauthorization(ctx) {
     try {
-      const params = ctx.request.body;
+      const params = ctx.request.body?.data || ctx.request.body;
+      if (!params || typeof params !== 'object') {
+        ctx.throw(400, "Invalid request body");
+      }
+
       const result = await getPayoneService(strapi).preauthorization(params);
-      ctx.body = { data: result };
+      ctx.body = result;
     } catch (error) {
       handleError(ctx, error);
     }
@@ -88,9 +111,14 @@ module.exports = ({ strapi }) => ({
 
   async authorization(ctx) {
     try {
-      const params = ctx.request.body;
+      const params = ctx.request.body?.data || ctx.request.body;
+
+      if (!params || typeof params !== 'object') {
+        ctx.throw(400, "Invalid request body");
+      }
+
       const result = await getPayoneService(strapi).authorization(params);
-      ctx.body = { data: result };
+      ctx.body = result;
     } catch (error) {
       handleError(ctx, error);
     }
@@ -98,9 +126,14 @@ module.exports = ({ strapi }) => ({
 
   async capture(ctx) {
     try {
-      const params = ctx.request.body;
+      const params = ctx.request.body?.data || ctx.request.body;
+
+      if (!params || typeof params !== 'object') {
+        ctx.throw(400, "Invalid request body");
+      }
+
       const result = await getPayoneService(strapi).capture(params);
-      ctx.body = { data: result };
+      ctx.body = result;
     } catch (error) {
       handleError(ctx, error);
     }
@@ -108,9 +141,14 @@ module.exports = ({ strapi }) => ({
 
   async refund(ctx) {
     try {
-      const params = ctx.request.body;
+      const params = ctx.request.body?.data || ctx.request.body;
+
+      if (!params || typeof params !== 'object') {
+        ctx.throw(400, "Invalid request body");
+      }
+
       const result = await getPayoneService(strapi).refund(params);
-      ctx.body = { data: result };
+      ctx.body = result;
     } catch (error) {
       handleError(ctx, error);
     }
@@ -118,9 +156,21 @@ module.exports = ({ strapi }) => ({
 
   async getTransactionHistory(ctx) {
     try {
-      const filters = ctx.query || {};
-      const history = await getPayoneService(strapi).getTransactionHistory(filters);
-      ctx.body = { data: history };
+      const { filters = {}, pagination = {} } = ctx.query || {};
+      const page = parseInt(pagination.page || "1", 10);
+      const pageSize = parseInt(pagination.pageSize || "10", 10);
+
+      const result = await getPayoneService(strapi).getTransactionHistory({
+        filters: filters || {},
+        pagination: { page, pageSize }
+      });
+
+      ctx.body = {
+        data: result.data || [],
+        meta: {
+          pagination: result.pagination
+        },
+      };
     } catch (error) {
       handleError(ctx, error);
     }
@@ -129,7 +179,7 @@ module.exports = ({ strapi }) => ({
   async testConnection(ctx) {
     try {
       const result = await getPayoneService(strapi).testConnection();
-      ctx.body = { data: result };
+      ctx.body = result || {};
     } catch (error) {
       handleError(ctx, error);
     }
@@ -149,7 +199,10 @@ module.exports = ({ strapi }) => ({
         resultType = "cancelled";
       }
 
-      const callbackData = isGetRequest ? ctx.query : ctx.request.body;
+      const callbackData = isGetRequest
+        ? ctx.query
+        : (ctx.request.body || ctx.request.body?.data || ctx.request?.data);
+
       const result = await getPayoneService(strapi).handle3DSCallback(callbackData, resultType);
 
       if (isGetRequest) {
@@ -166,7 +219,7 @@ module.exports = ({ strapi }) => ({
         return ctx.redirect(redirectUrl);
       }
 
-      ctx.body = { data: result };
+      ctx.body = result;
     } catch (error) {
       handleError(ctx, error);
     }
@@ -177,13 +230,12 @@ module.exports = ({ strapi }) => ({
       const settings = await getPayoneService(strapi).getSettings();
       const applePayConfig = settings?.applePayConfig || {};
 
-      const params = ctx.request.body;
+      const params = ctx.request.body || ctx.request.body?.data || ctx.request?.data;
 
       if (!params) {
         throw new Error("Request body is missing");
       }
 
-      // Ensure domain is set
       if (!params.domain && !params.domainName) {
         params.domain = ctx.request.hostname || ctx.request.host || 'localhost';
         params.domainName = params.domain;
@@ -210,11 +262,10 @@ module.exports = ({ strapi }) => ({
         throw new Error("Merchant validation returned null. Please check your Payone Apple Pay configuration.");
       }
 
-      ctx.body = { data: result };
+      ctx.body = result;
     } catch (error) {
       const errorStatus = error.status || (error.message?.includes('403') ? 403 : 500);
 
-      // Only log if it's a response error
       if (error.response || errorStatus === 403 || errorStatus === 401 || errorStatus >= 500) {
         strapi.log.error("[Apple Pay] Controller error:", {
           status: errorStatus,
@@ -222,11 +273,9 @@ module.exports = ({ strapi }) => ({
         });
       }
 
-      // Extract detailed error message if available
       let errorMessage = error.message || "Apple Pay merchant validation failed";
       let errorDetails = "Please check your Payone Apple Pay configuration in PMI (CONFIGURATION → PAYMENT PORTALS → [Your Portal] → Apple Pay). Ensure that Merchant ID (mid) is correctly configured and Apple Pay is enabled for your portal.";
 
-      // If it's a 403 error, provide more specific guidance
       if (errorStatus === 403 || error.message?.includes('403')) {
         errorDetails = "403 Forbidden: Authentication failed with Payone. " +
           "Please check: 1) Your Payone credentials (aid, portalid, mid, key) in plugin settings, " +

package/server/policies/is-auth.js

@@ -1,13 +1,19 @@
 "use strict";
 
-module.exports = async (ctx, config, { strapi }) => {
-  const { authorization } = ctx.request.header || {};
+module.exports = async (policyContext, config, { strapi }) => {
+  const { authorization } = policyContext.request.header || {};
 
   if (authorization && authorization.startsWith("Bearer ")) {
     const token = authorization.split(" ")[1];
 
     try {
-      const apiTokenService = strapi.services["admin::api-token"];
+      const apiTokenService = strapi.service("admin::api-token");
+
+      if (!apiTokenService) {
+        strapi.log.warn("strapi-plugin-payone-provider: api-token service not found");
+        return false;
+      }
+
       const accessKey = await apiTokenService.hash(token);
       const storedToken = await apiTokenService.getBy({ accessKey });
 

package/server/policies/isSuperAdmin.js

@@ -1,18 +1,20 @@
 "use strict";
 
-module.exports = async (ctx, next) => {
-  const adminUser = ctx.state && ctx.state.user;
+module.exports = async (policyContext, config, { strapi }) => {
+  const adminUser = policyContext.state && policyContext.state.user;
 
   if (!adminUser) {
-    return ctx.unauthorized("Admin authentication required");
+    policyContext.unauthorized("Admin authentication required");
+    return false;
   }
 
   const roles = Array.isArray(adminUser.roles) ? adminUser.roles : [];
   const isSuperAdmin = roles.some((role) => role.code === "strapi-super-admin");
 
   if (!isSuperAdmin) {
-    return ctx.forbidden("Only super admins can access this resource");
+    policyContext.forbidden("Only super admins can access this resource");
+    return false;
   }
 
-  return next();
+  return true;
 };

package/server/services/paymentService.js

@@ -51,13 +51,7 @@ const sendRequest = async (strapi, params) => {
     });
 
     const responseData = parseResponse(response.data, strapi.log);
-    const errorCode =
-      responseData.errorcode ||
-      responseData.ErrorCode ||
-      responseData.Error?.ErrorCode ||
-      responseData.error_code ||
-      null;
-
+    const errorCode = responseData?.Error?.ErrorCode;
     const requires3DSErrorCodes = ["4219", 4219];
     const is3DSRequiredError = requires3DSErrorCodes.includes(errorCode);
 
@@ -69,21 +63,11 @@ const sendRequest = async (strapi, params) => {
 
     }
 
-    const errorMessage =
-      responseData.errormessage ||
-      responseData.ErrorMessage ||
-      responseData.Error?.ErrorMessage ||
-      responseData.error_message ||
-      null;
-
-    const customerMessage =
-      responseData.customermessage ||
-      responseData.CustomerMessage ||
-      responseData.Error?.CustomerMessage ||
-      responseData.customer_message ||
-      null;
-
-    const status = (responseData.status || responseData.Status || "unknown").toUpperCase();
+    const errorMessage = responseData?.Error?.ErrorMessage || null;
+
+    const customerMessage = responseData?.Error?.CustomerMessage || null;
+
+    const status = (responseData?.status || responseData?.Status || "unknown").toUpperCase() || null;
 
     await logTransaction(strapi, {
       txid: extractTxId(responseData) || params.txid || null,

package/server/services/payone.js

@@ -16,7 +16,6 @@ module.exports = ({ strapi }) => ({
     return await settingsService.updateSettings(strapi, settings);
   },
 
-  // Payment operations
   async preauthorization(params) {
     return await paymentService.preauthorization(strapi, params);
   },
@@ -38,8 +37,9 @@ module.exports = ({ strapi }) => ({
     return await transactionService.logTransaction(strapi, transactionData);
   },
 
-  async getTransactionHistory(filters = {}) {
-    return await transactionService.getTransactionHistory(strapi, filters);
+
+  async getTransactionHistory({ filters = {}, pagination = {} }) {
+    return await transactionService.getTransactionHistory(strapi, { filters, pagination });
   },
 
   // Test connection

package/server/services/settingsService.js

@@ -12,16 +12,26 @@ const getPluginStore = (strapi) => {
 
 const getSettings = async (strapi) => {
   const pluginStore = getPluginStore(strapi);
-  return await pluginStore.get({ key: "settings" });
+  const settings = await pluginStore.get({ key: "settings" }) || {};
+
+  return settings;
 };
 
 const updateSettings = async (strapi, settings) => {
   const pluginStore = getPluginStore(strapi);
+  const currentSettings = await getSettings(strapi) || {};
+  const mergedSettings = {
+    ...currentSettings,
+    ...settings
+  };
+
+
   await pluginStore.set({
     key: "settings",
-    value: settings
+    value: mergedSettings
   });
-  return settings;
+
+  return mergedSettings;
 };
 
 const validateSettings = (settings) => {

package/server/services/testConnectionService.js

@@ -54,82 +54,19 @@ const testConnection = async (strapi) => {
     });
 
     const result = parseResponse(response.data, strapi.log);
-    const status = result.status || result.Status || result.STATUS;
-    const errorMessage =
-      result.errormessage ||
-      result.Errormessage ||
-      result.ERRORMESSAGE ||
-      result.error ||
-      result.Error?.ErrorMessage ||
-      "";
-    const errorCode =
-      result.errorcode ||
-      result.Errorcode ||
-      result.ERRORCODE ||
-      result.Error?.ErrorCode ||
-      "";
-    const customErrorMessage =
-      result.customerrormessage ||
-      result.Customerrormessage ||
-      result.CUSTOMERRORMESSAGE ||
-      result.Error?.CustomerMessage ||
-      "";
+    const status = result?.status || result?.Status || result?.STATUS;
+    const errorMessage = result?.Error?.ErrorMessage;
+    const errorCode = result?.Error?.ErrorCode;
+    const customErrorMessage = result?.Error?.CustomerMessage;
 
     if (status === "ERROR" || status === "error") {
-      if (["2006", "920", "921", "922", "401", "403"].includes(errorCode)) {
-        return {
-          success: false,
-          message: `Authentication failed: ${customErrorMessage || errorMessage || "Invalid credentials"}`,
-          errorcode: errorCode
-        };
-      }
-
-      const errorMessageStr = typeof errorMessage === "string" ? errorMessage : JSON.stringify(errorMessage);
-      const errorMessageLower = (errorMessageStr || "").toLowerCase();
-      const authErrorKeywords = [
-        "key incorrect",
-        "invalid key",
-        "portal key",
-        "unauthorized",
-        "not authorized",
-        "unknown aid",
-        "unknown account",
-        "unknown portal",
-        "unknown merchant",
-        "invalid aid",
-        "invalid mid",
-        "invalid portalid"
-      ];
-
-      if (authErrorKeywords.some((keyword) => errorMessageLower.includes(keyword))) {
-        return {
-          success: false,
-          message: `Authentication failed: ${errorMessageStr}`,
-          errorcode: errorCode || "AUTH"
-        };
-      }
-
-      if (errorCode === "911") {
-        return {
-          success: true,
-          message: "Connection successful! Your Payone credentials are valid.",
-          details: {
-            mode: settings.mode,
-            aid: settings.aid,
-            portalid: settings.portalid,
-            mid: settings.mid
-          }
-        };
-      }
-
       return {
         success: false,
-        message: `Connection failed: ${customErrorMessage || errorMessageStr || "Unknown error"}`,
-        errorcode: errorCode,
-        details: {
-          status,
-          errorCode,
-          rawResponse: JSON.stringify(result).substring(0, 200)
+        message: `Test connection failed: ${errorCode}`,
+        error: {
+          ErrorCode: errorCode,
+          ErrorMessage: errorMessage,
+          CustomerMessage: customErrorMessage
         }
       };
     }
@@ -157,11 +94,12 @@ const testConnection = async (strapi) => {
         rawResponse: JSON.stringify(result).substring(0, 200)
       }
     };
+
   } catch (error) {
     strapi.log.error("Payone test connection error:", error);
     return {
       success: false,
-      message: `Connection error: ${error.message || "Unknown error"}`,
+      message: `Test connection error: ${error.message || "Unknown error"}`,
       error: error.toString(),
       details: {
         errorType: error.constructor.name,