Commit Graph

545 Commits

Author SHA1 Message Date
Rodrigo Primo f06ad5d78a Replace removed WordPress.Security.NonceVerification.NoNonceVerification
WordPress Coding Standard 2.0 removed the sniff
WordPress.Security.NonceVerification.NoNonceVerification:

```
The WordPress.Security.NonceVerification sniff used the same error code for both an error as well as a warning.
The old error code NoNonceVerification is no longer used.
The error now uses the Missing error code, while the warning now uses the Recommended error code.
```

(from
d45f5e5cf3/CHANGELOG.md (200-rc1---2018-12-31))

This commit updates WooCommerce code and replaces all instances where WordPress.Security.NonceVerification.NoNonceVerification verification was used with either WordPress.Security.NonceVerification.Missing or
WordPress.Security.NonceVerification.Recommended. In a few cases WordPress.Security.NonceVerification.NoNonceVerification was used but was not needed, so instead of replacing the sniff, the line was removed. In two other cases, I removed other unrelated sniffs that were not needed.
2019-12-13 17:45:32 -03:00
Claudio Sanches 2453dc9fe1 Fixed check for the method 2019-10-02 14:36:05 -03:00
Claudio Sanches 4c023c5e8a Changed thank you page text to follow PayPal guidelines
Also fixed coding standards
2019-10-02 14:21:12 -03:00
Claudio Sanches 3c9b9ac6a0 Revert changes from PayPal 2019-10-02 14:20:13 -03:00
Claudio Sanches 20ee6c7aed Fixed coding standards 2019-10-02 14:18:03 -03:00
Claudio Sanches b655af2e3b Changed thank you page text to follow PayPal guidelines 2019-10-02 14:17:25 -03:00
Rodrigo Primo b63ecca450
Merge pull request #24249 from miguelfspinto/patch-2
Avoid duplicated process for IPN and PDT payment
2019-09-24 17:59:56 -03:00
Rodrigo Primo e7b8e3ed7b Fix typo in order note
PR #24501 fixed a typo in a order note string. This commit fix the same
problem in another place where the same string is used.
2019-08-30 10:01:55 -03:00
Miguel Pinto 60ac27ba3e
Update class-wc-gateway-paypal-response.php 2019-08-27 22:30:33 +01:00
Mithu A Quayium fa62be3fa1
Typo fix in line: 428
The string "Payment could not captured" in line:428 should be "Payment could not be captured" instead, since it is appropriate grammatically.
2019-08-26 11:32:40 +06:00
Miguel Pinto ae4eba45ec
Update class-wc-gateway-paypal-response.php 2019-08-01 16:19:17 +01:00
Miguel Pinto 77f4a3e25b
Update class-wc-gateway-paypal-response.php 2019-07-31 09:48:41 +01:00
Miguel Pinto de978dad28
Update class-wc-gateway-paypal-response.php 2019-07-30 21:31:55 +01:00
Miguel Pinto d7b54ccb35
Update class-wc-gateway-paypal-response.php 2019-07-30 20:56:43 +01:00
Miguel Pinto ce4f0edc3a
Avoid duplicated process for IPN and PDT payment
Currently, Woocommerce is processing twice the payment when IPN and PDT notifications are enable. This mean that stock is reduced twice.
2019-07-29 09:25:30 +01:00
Gerhard 70231461b2 Refactor logic for passing no_shipping, it should only be passed when no shipping items are available. 2019-05-21 08:11:36 +02:00
Gerhard 8332bed7bc Do not pass no_shipping when not using send shipping address options in paypal, it still needs to use the billing details. 2019-05-20 14:59:48 +02:00
Gerhard af678dfc84 Switch paypal capture_payment method to use woocommerce_order_status_processing and woocommerce_order_status_completed instead to avoid issues with custom from status transitions.
Update get_post_meta usage with $order->get_meta instead.
2019-05-07 12:57:59 +02:00
David Jensen 9b58e68e85 Remove html tags from product name in request to PayPal 2019-04-12 12:36:30 -07:00
Claudio Sanches e5e64d1618 Apply wc_sanitize_phone_number in WC_Gateway_Paypal_Request 2019-03-07 14:08:40 -03:00
Mike Jolley 55b27a2afc Adjust calling code prefix logic in paypal 2019-03-06 12:33:46 +00:00
Mike Jolley 65c0169726 Merge branch 'master' into fix/21110 2019-03-06 12:25:12 +00:00
Mike Jolley 4330b2eb38 Update calling code handling 2019-03-06 12:25:06 +00:00
Mike Jolley 49065c0d1a Add meta data, before changing order status 2019-02-27 14:50:37 +00:00
Mike Jolley 7306bf6382
Merge pull request #22763 from woocommerce/fix/paypal-bn-code
Move PayPal BN partner ID
2019-02-19 09:42:05 +00:00
Mike Jolley 8abe75f36d Merge branch 'fix/21463' 2019-02-18 17:20:48 +00:00
Mike Jolley 3ededc2a07 Move BN partner ID 2019-02-18 16:29:29 +00:00
Mike Jolley f0675781c8 Include calling code in pp request 2019-02-08 11:10:26 +00:00
Mike Jolley dca47d5d14 Use mb functions for limiting string length 2019-02-07 19:09:41 +00:00
Mike Jolley dbd324abc3 phpcs 2019-02-07 14:55:56 +00:00
Mike Jolley ebdeab4c22 Added woocommerce_paypal_force_one_line_item filter 2019-02-04 16:31:17 +00:00
Mike Jolley 34dbf96847
Merge pull request #22410 from woocommerce/remove/simplify-commerce
Remove simplify commerce gateway from core
2019-02-04 11:23:23 +00:00
Mike Jolley 6367d19615 phpcs 2019-01-21 08:11:40 -08:00
Mike Jolley 9d381d456b Remove simplify commerce gateway 2019-01-10 12:01:27 +00:00
Mike Jolley 4d0306b977
Merge pull request #21402 from xlplugins/master
Added new filter to modify order status during orders using cheque as a gateway.
2019-01-07 12:50:35 +00:00
JeroenSormani f4775ccd90 Use shipping method instead of property through magic + doc return statements 2019-01-04 12:41:38 +00:00
Aslam Shekh 0611204238 Fixed Wordpress standard as per given suggestion on here:- https://github.com/woocommerce/woocommerce/pull/22122#issuecomment-444207375 2018-12-05 08:12:57 -08:00
Rodrigo Primo bfa9424d17 Update WPCS sniff name
The name of one of the WPCS sniffs changed from WordPress.CSRF.NonceVerification.NoNonceVerification to WordPress.Security.NonceVerification.NoNonceVerification. This commit updates all instances where this sniff is used in WooCommerce codebase.
2018-11-21 11:07:05 -02:00
Rodrigo Primo e467737653 phpcs fixes 2018-11-21 11:06:42 -02:00
Claudiu Lodromanean 00d254ac18
Merge pull request #21715 from angelleye/21714
Sets signature field type to password for security, resolves #21714
2018-11-02 08:09:56 -07:00
angelleye f995c2e070 Changes field type to password for live Signature field, resolves #21714 2018-10-31 12:32:40 -05:00
Gerhard aba734f033 Make PDT validation use the same rounding as the IPN validation. 2018-10-29 11:50:47 +02:00
angelleye 7ec6f3f67b Sets signature field type to password for security, resolves #21714 2018-10-27 06:21:56 -05:00
XLPlugins e283eff149 Added new filter to modify order status during orders using cheque as a gateway. 2018-09-20 19:07:23 +05:30
Maria Górska 0369e3d622 Enable PDT handling for custom pending statuses 2018-08-27 12:10:36 +02:00
Gerhard Potgieter c7884003dc Do not build paypal pay link when no base country is set. 2018-07-16 11:44:58 +02:00
Mike Jolley a17945d9a8
Merge pull request #20574 from woocommerce/fix/20551
Fix: set order status to refunded when a PayPal transaction is refunded
2018-06-20 14:56:25 +01:00
Rodrigo Primo 5b5256370a Ignore a single PHPCS rule instead of ignoring all rules
Replace the usage of `// @codingStandardsIgnoreStart` and `// @codingStandardsIgnoreEnd` with `// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores` on WC_Gateway_Paypal_IPN_Handler::check_response(). Ignoring a single PHPCS rule is better than ignoring everything to protect the line against potential violations of the other rules.
2018-06-19 10:44:45 -03:00
Rodrigo Primo 9263a0b169 Fix: set order status to refunded when a PayPal transaction is refunded
Commit c7a3fd2 changed the logic to check if the refund is a full refund to use the strict equal operator (`===`) in the following line:

25be9fc13c/includes/gateways/paypal/includes/class-wc-gateway-paypal-ipn-handler.php (L295)

This change broke the comparison as `$order->get_total()` will return the value respecting the number of decimals after the decimal point set in the option `woocommerce_price_num_decimals` and `wc_format_decimal()`, when called without a second parameter, will not format the received value to use the same number of decimals set in `woocommerce_price_num_decimals`. To fix this issue, this commit passes `wc_get_price_decimals()` as the second parameter to `wc_format_decimal()`. This way both values will always have the same number of decimals and the comparison should work when handling a full refund.

Fixes #20551
2018-06-19 10:42:27 -03:00
Mike Jolley 5e6405b650 Merge branch 'master' into update/13359 2018-06-15 15:12:12 +01:00