Commit Graph

557 Commits

Author SHA1 Message Date
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
Khan M Rashedun-Naby 8d7ae8b413 Remove unused variable and unnecesary else if 2018-06-09 23:24:08 +06:00
Mike Jolley 2cad2414b3 Remove manual wc_reduce_stock_levels calls 2018-06-07 18:01:38 +01:00
Mike Jolley ad10e846ea get_canonical_package_rate_ids should check it has an array 2018-05-25 12:54:58 +01:00
Mike Jolley cf9fa5b3ff Updates table styling and columns 2018-05-04 18:48:41 +01:00
Mike Jolley 57ae8285b6 Mask logged PayPal data. 2018-04-30 18:17:12 +01:00
Mike Jolley aef46ce8d3 Clear paypal logs when debug mode is turned off. 2018-04-30 18:04:52 +01:00
Mike Jolley 610e9b0989 Add disclaimer to debug logging setting 2018-04-30 17:30:55 +01:00
Mike Jolley d14e59cdb0 Remove some PII from paypal logs and avoid storing custom meta from responses 2018-04-30 17:30:39 +01:00
Claudio Sanches 781d38dc51 Removed undefined wp_json_decode() function from PayPal response
Closes #19807
2018-04-20 13:26:05 -03:00
Mike Jolley 1b34cd4745 Moved settings tabs around to allow for privacy page. 2018-04-12 16:59:42 +01:00
Claudio Sanches 900c8fc33f
Merge pull request #19380 from woocommerce/update/19302
Override supports for PayPal to see if credentials exist
2018-04-06 15:44:42 -03:00
Peter Fabian 80dc183880 Use nbsp to separate * denoting required fields. 2018-04-04 16:45:40 +02:00
Mike Jolley 5ac7aac369 Improve can_refund_order and add unit tests 2018-04-03 17:09:09 +01:00
Mike Jolley d02106f697 Merge branch 'master' into pr/18596 2018-04-03 16:24:19 +01:00
Mike Jolley 7b45e90357 Merge branch 'master' into update/19302 2018-04-03 14:57:24 +01:00
Mike Jolley 51252915e1
Merge pull request #19493 from woocommerce/fix/19404
Removed line item limit and made sure the URL is shorter than 2083 ch…
2018-03-28 13:56:57 +01:00
Peter Fabian 3809ab3f4c Fixed shipping tax logic to behave the same as originally 2018-03-28 13:08:05 +02:00
Peter Fabian 59c47c4407 Refactored line item logic
- prepare_line_items is not used as a test anymore, it only updates line items
- add_line_item is not used as a test anymore, only adds line items
- added method line_items_valid to test for valid line items
- shipping tax inclusion test moved to get_shipping_cost_line_item to encapsulate shipping tax-related logic
2018-03-28 11:04:34 +02:00
Peter Fabian f39e8e3900 Removed incorrect comment 2018-03-27 16:28:25 +02:00
Peter Fabian 5eda49f98a Line item logic refactored
Renamed function get_non_line_item_args
Documentation updates
...all part of code review fixes
2018-03-27 15:55:03 +02:00
Peter Fabian 88588eccbc Code duplication removed
Added test for non line item agrument presence
Fixed incorrect comment
2018-03-26 13:05:55 +02:00
Peter Fabian e19d64e51e Fixed PHPCS violations 2018-03-26 11:00:28 +02:00
Peter Fabian 26144afbc6 Test request length after applying filter
- added default value to get_line_item_args to not break interface
- smaller code review updates
2018-03-23 17:22:09 +01:00
Mike Jolley 0573232c62 Call empty cart when completing payment in PayPal 2018-03-23 12:54:03 +00:00
Peter Fabian 59af63e642 Merge branch 'master' into fix/19404 2018-03-23 09:36:15 +01:00
Peter Fabian 27a309d2df Removed line item limit and make sure the URL is shorter than 2083 characters
- removed limit to 9 line items per Paypal order
- if the request URL with all line items will be longer than 2083 characters, send it as one line item
- fix the character limit function to account for URL encoding happening in http_build_query
2018-03-22 15:39:30 +01:00
Claudio Sanches 0660a50e07 Removed obvious boolean results 2018-03-21 19:57:10 -03:00
Mike Jolley 291de9c1fb Override supports for PayPal to see if credentials exist 2018-03-13 11:35:20 +00:00
Claudio Sanches 54a0e01b8a
Merge branch 'master' into patch-2 2018-03-12 16:28:00 -03:00
Rodrigo Primo 4e3ba1b73b phpcs fixes in includes/gateways/paypal/includes/class-wc-gateway-paypal-response.php 2018-03-09 15:45:47 -03:00
Rodrigo Primo cf5ce33e29 phpcs fixes in includes/gateways/paypal/includes/class-wc-gateway-paypal-request.php 2018-03-09 15:45:35 -03:00
Rodrigo Primo 128c4398a5 phpcs fixes in includes/gateways/paypal/includes/class-wc-gateway-paypal-pdt-handler.php 2018-03-09 15:45:22 -03:00
Rodrigo Primo 06af993595 phpcs fixes in includes/gateways/paypal/includes/class-wc-gateway-paypal-api-handler.php 2018-03-09 15:45:13 -03:00
Rodrigo Primo 93a50caa42 phpcs fixes in includes/gateways/paypal/class-wc-gateway-paypal.php 2018-03-09 15:45:00 -03:00
Rodrigo Primo cdf8b483a1 phpcs fixes in includes/gateways/cod/class-wc-gateway-cod.php 2018-03-09 15:44:48 -03:00
Rodrigo Primo 4989e0ff2b phpcs fixes in includes/gateways/class-wc-payment-gateway-echeck.php 2018-03-09 15:44:37 -03:00
Rodrigo Primo 609bc3bf71 phpcs fixes in includes/gateways/class-wc-payment-gateway-cc.php 2018-03-09 15:44:24 -03:00
Rodrigo Primo 3451dae895 phpcs fixes in includes/gateways/cheque/class-wc-gateway-cheque.php 2018-03-09 15:44:14 -03:00
Rodrigo Primo 701320ad32 phpcs fixes in includes/gateways/bacs/class-wc-gateway-bacs.php 2018-03-09 15:43:56 -03:00
Rodrigo Primo df249d3b76 Rename class WC_Payment_Gateway_eCheck to WC_Payment_Gateway_ECheck
To match WP coding standards.
2018-03-09 09:57:59 -03:00
Rodrigo Primo 8678ed8685 Fix phpcs violations using phpcbf 2018-03-09 09:53:10 -03:00
mweimerskirch 28df2ce3e0
Added a filter allowing to change the order status to the BACS gateway
The structure is the same as for the COD gateway (there, the filter is called "woocommerce_cod_process_payment_order_status").
2018-03-08 16:11:54 +01:00
Mike Jolley 85450f60ec Merge branch 'master' into pr/18596 2018-03-02 19:31:37 +00:00
danielshields e2da7fd277
Fix the PayPal API URL
Assuming they changed the section title, as the permalink is now "create" vs "creating"
2018-03-01 12:30:38 -05:00
Mike Jolley 341dfe1e7b Fix table styling in mobile 2018-01-30 13:24:59 +00:00
Mike Jolley 46f9e00497 Tweak paypal seller protection text
Closes #18632
2018-01-29 14:43:25 +00:00
Manos Psychogyiopoulos 690c7a9753 Check if paying for an existing order before looking at cart contents
Resolves a bug causing this logic to ignore the `order-pay` query var presence if the cart is populated with items.
2018-01-25 16:42:54 +02:00
Manos Psychogyiopoulos d10987d813 Match saved (canonical) rate IDs against chosen rates
Generates canonical rate IDs (`method_id:instance_id`) from package rates and order shipping line items.

Requires https://github.com/woocommerce/woocommerce/pull/18483
2018-01-25 16:40:31 +02:00
Manos Psychogyiopoulos d86a948f04 Add method instances in COD, grouped by method 2018-01-25 16:37:10 +02:00
Paul Dechov 54fa85f388 Disable all payment gateways by default
Switches defaults for Check payments and PayPal Standard to 'no', so that no payment gateways are enabled by default before running through setup wizard.
2018-01-19 02:23:55 -05:00
Mike Jolley 405c2614a8 Use new methods 2018-01-11 12:54:48 +00:00
Mike Jolley c7a3fd266c phpcs 2018-01-11 12:53:47 +00:00
James Allan f6eb02806b Mark orders as payment complete if the order total is 0
Similar to how other payment gateways like Stripe call
payment_complete on orders with $0 totals.
2017-11-30 13:31:10 +10:00
Mike Jolley 17710b864d Tweak IPN setting wording
@claudiulodro
2017-11-15 15:12:52 +00:00
George Lagonikas 6b7cc782d2
Removing obsolete code 2017-11-14 22:30:28 +00:00
George Lagonikas bca6bc7375
Text changes 2017-11-14 20:31:39 +00:00
George Lagonikas 87a75ce96e
Simplifying logic 2017-11-14 20:30:49 +00:00
SGroupGeorge 64332d82c7 Text changes 2017-11-14 17:51:46 +00:00
SGroupGeorge 3b2eec75d3 Adding checkbox to make IPN emails optional 2017-11-14 17:34:48 +00:00
Mike Jolley ced4551413 Merge branch 'pr/16986' 2017-10-27 15:49:00 +01:00
Mike Jolley 7cafc71526 Wording tweaks 2017-10-27 15:48:50 +01:00
Mike Jolley c15491b9ad Improved PDT handling and logs 2017-10-02 14:24:39 +01:00
SGroupGeorge 8a95d1111b Text changes 2017-09-29 10:47:07 +01:00
SGroupGeorge 38e6f07c70 Store both Live and Sandbox details for PayPal 2017-09-29 10:41:09 +01:00
Mike Jolley 9728cb5252 Added PayPal India support 2017-09-27 16:14:26 +01:00
Jaydeep Rami 46d3638089 Fix missing return statement (#16148)
* Fix missing return statement

* Fix missing return statement

* Fix missing return statement

* Fix missing return statement

* Fix missing return statement

* Fix missing return statement
2017-07-20 18:01:14 +01:00
Jaydeep Rami 3522857d23 Fix typo in WooCommerce plugin (#16135)
* Fixed typo

* Fixed more typo

* Fixed more typo

* Fix tyop

* Fix more typo

* Fix more typo

* Fix typo

* Fix typo

* Fix typo

* Fix typo datatime object

* Fix short name of compat

* Fix typo: update short name

* Fix typo "deactive" to "deactivate" and short BW - Compat

* Fix typo "Backwards compat"

* Fix typo 'parameters'

* Fix more typo 'pararmeters'

* Fix typo 'compund'

* FIx typo order

* Fix typo

* Fix typo 'incorrecly'

* Fix typo 'genarate'

* Fix typo 'reletive'

* Fix typo 'Handly'

* Fix typo 'rotatated'

* Fix typo

* Fix typo 'additonal'

* Fix typos
2017-07-17 11:10:52 +01:00
Mike Jolley 982edb2f9f Improves cash on delivery logic when dealing with multiple methods
If an order has multiple shipping methods, check all against the
allowed methods using a simple array_diff.
2017-07-13 16:21:59 +01:00
Beka Rice 1f95709e42 Fix deprecation notice from accessing order props directly 2017-07-13 09:56:46 -04:00
Claudio Sanches fa850ecfb4 Removed PayPal legacy code 2017-07-11 12:09:57 -03:00
Mike Jolley 59a4f3b0d9 Remove sandbox status forcing in PayPal gateway
Closes #15899

@californiakat Should we add
https://stackoverflow.com/questions/4298117/paypal-ipn-always-return-pay
ment-status-pending-on-sandbox to the docs when 3.2 launches?
2017-07-03 10:46:32 +01:00
Mike Jolley bf6c39fbf2 Use wc_float_to_string on item prices sent to paypal
Fixes #15872
2017-06-30 10:28:21 +01:00
Mike Jolley a1940f2da3 Grunt dev 2017-06-19 12:49:34 +01:00
Mike Jolley b8eafe985e Set defaults to avoid error if too few args passed 2017-06-15 15:43:32 +01:00
Mike Jolley 55e9add8e5 More strict COD method matching
Fixes #15606
2017-06-14 12:17:28 +01:00
Mike Jolley ba2bf67bb9 Use hash_equals where needed 2017-06-13 13:54:02 +01:00
Boro Sitnikovski 8a615a6e79 Update conditional for non-auth payments 2017-06-05 14:07:49 +02:00
Boro Sitnikovski 4b82a750fc Fix IPN authorization setting orders to completed automatically 2017-06-05 10:07:22 +02:00
Mike Jolley 20637e53b8 For COD orders, force payment complete status to be completed. 2017-05-24 18:48:34 +01:00
Mike Jolley c16acc6b51 Merge remote-tracking branch 'origin/master'
# Conflicts:
#	assets/css/wc-setup-rtl.css
#	assets/css/wc-setup.css
2017-05-23 19:00:24 +01:00
Mike Jolley d3666738d8 Merge branch 'pr/15069' 2017-05-23 18:56:58 +01:00
Mike Jolley 00e5189a05 Merge pull request #15070 from rasmusbe/phpdoc-missing-return-throws
Phpdoc: Add missing return and throws
2017-05-23 18:50:58 +01:00
Rasmus Bengtsson 185934fad9 Fix phpdoc to include params and correct types 2017-05-15 13:50:52 +02:00
Rasmus Bengtsson 4f00d7e5e3 PHPDoc: Add missing return statement 2017-05-15 13:37:59 +02:00
Mike Jolley bc4d36fd3f Add abspath checks
Closes #14999
2017-05-10 19:03:19 +01:00
Mike Jolley 891e55e1cf Merge pull request #14721 from woocommerce/fix/14670
Avoid escaped quote chars in strings
2017-04-27 15:34:27 +01:00
Mike Jolley aacb01d1e7 Merge branch 'pr/13503'
# Conflicts:
#	assets/css/twenty-seventeen.css
#	assets/css/woocommerce-layout.css
#	assets/css/woocommerce-layout.scss
#	assets/css/woocommerce.css
2017-04-27 12:46:05 +01:00
Mike Jolley 801f5e1eb1 Avoid escaped quote chars in strings
Closes #14670
2017-04-26 13:51:53 +01:00