woocommerce / woocommerce-square

Square POS and Payments Integration
https://woo.com/products/square
Other
11 stars 7 forks source link

Update Product.php #191

Closed sunile2m closed 3 months ago

sunile2m commented 4 months ago

Bug accessing database table with statically added "wp_" db prefix.

All Submissions:


Changes proposed in this Pull Request:

Closes # .

Steps to test the changes in this Pull Request:

1. 1. 1.

Changelog entry

Update - Database table prefix to be variable and not hard-coded to wp_.

qasumitbagthariya commented 3 months ago

QA Update ✅


I have verified this PR in the sunile2m:trunk branch and functioning as intended.

I tested the following on this branch:

Testing Environment

- WordPress: 6.6.1 - Theme: Storefront 4.6.0 - Theme: Twenty Twenty-Four 1.2 - WooCommerce - 9.1.4 - PHP: 8.0.30 - Web Server: Nginx 1.20.2 - Browser: Chrome - OS: macOS Ventura 13.3 - Branch: sunile2m:trunk

Steps to Test- As mentioned in the PR description. Test Results - It is working as expected. Functional Demo / Screencast - Special Notes - Ready for code review (Woo) Testing Document status: Cases related to this Issue/PR are added to the Critical Flow Wiki pages:

qasumitbagthariya commented 3 months ago

Regression / Smoke Test Report ✅

Tested with Archive File created via "php woorelease.phar build repo_URL" (Composer version 2.5.5, npm version 8.19.4, node version 16.20.0)

Status- Working expected with Plugin Archive/Zip file same as fix specific branch.

Testing Environment

- WordPress: 6.6.1 - Theme: Storefront 4.6.0 - Theme: Twenty Twenty-Four 1.2 - WooCommerce - 9.1.4 - PHP: 8.0.30 - Web Server: Nginx 1.20.2 - Browser: Chrome - OS: macOS Ventura 13.3 - Branch: smoke-testing

Next Step- Ready to Merge 🚀