Use urlencode

[slash1andy]

Use urlencode in place of get_site_url

All Submissions:

• [x ] Does your code follow the WooCommerce Sniffs variant of WordPress coding standards?
• [x ] Have you written new tests for your changes, as applicable?
• [ x] Have you successfully run tests with your changes locally?
• [ ] Will this change require new documentation or changes to existing documentation?

---

Changes proposed in this Pull Request:

Use urlencode instead of simply returning the site url

Steps to test the changes in this Pull Request:

Activate plugin and process a transaction

Changelog entry

Fix - Use rawurlencode around the call to get_site_url to ensure things are encoded properly.

[ankitguptaindia]

@slash1andy Could you please clarify which area of this plugin might be affected by this improvement? This will enable me to thoroughly test and verify the change manually. Thank you!

[ankitguptaindia]

QA/Test Report- ✅

Testing Environment -

* WordPress: 6.5.3 * Theme: Twenty Twenty-Four Version: 1.1 * WooCommerce - Version 8.8.3 * PHP: 8.3 * Web Server: Nginx * Browser: Chrome - Version 124 * OS: macOS Sonoma 14.4.1 * Git Branch: patch-05-13-24

Test Results - The order was placed successfully after initiating an order while using Payfast as the payment method. All actions related to the order and Payfast payment were found as working.

• Customer details in order ✅
• Order placed successfully ✅
• ITNs updated on new order ✅
• Transactions updated on a new order in the Payfast dashboard ✅
• Order status becomes 'Processing' after a successful transaction ✅
• Order notes when ITNs updated ✅

Next Step- Ready for UAT.