Commit Graph

24486 Commits

Author SHA1 Message Date
Rodrigo Primo e006e14d23 Ignore PHPCS in two instances
Ignoring PHPCS in two lines of the code where apparently it is not possible to fix the violations. I tried using what @claudiosanches suggested in this comment https://github.com/woocommerce/woocommerce/pull/17909#issuecomment-347168029 but both tags didn't work. Lets revisit this when PHPCS 3.2.0 is released and we can selectively disable rules (see https://github.com/squizlabs/PHP_CodeSniffer/issues/604).
2017-11-27 11:00:42 -02:00
Rodrigo Primo 3fe834ba3c Exclude abstract classes from PHPCS file name checks
The plan is to revisit this for WC 4.0 and rename abstract classes files.
2017-11-27 11:00:21 -02:00
Claudio Sanches 9123656155 [REST API] Fixed date format in reports schema 2017-11-27 10:24:43 -02:00
Rodrigo Primo 7b17b79460 Fix PHPCS violations in file tests/unit-tests/order/functions.php 2017-11-24 17:25:31 -02:00
Rodrigo Primo 3d6bd30d8a Fix some PHPCS violations in includes/data-stores/abstract-wc-order-data-store-cpt.php 2017-11-24 17:25:31 -02:00
Rodrigo Primo 5e43d92e07 Fix some PHPCS violations in includes/data-stores/class-wc-order-data-store-cpt.php 2017-11-24 17:25:16 -02:00
Rodrigo Primo acb970dcf7 Add some basic test coverage for wc_order_search() 2017-11-24 17:04:14 -02:00
Claudio Sanches 3cc84a1821
Merge pull request #17904 from rodrigoprimo/update/travis-dont-reinstall-phpunit
Travis: don't reinstall PHPUnit
2017-11-24 10:56:14 -02:00
Rodrigo Primo 6f207d39f3 travis.sh cleanup
We don't call `composer install` for all PHP version anymore in this script, so it is not necessary to exit early if running PHP 5.2.
2017-11-24 10:36:42 -02:00
Rodrigo Primo 183078914b Travis: don't reinstall PHPUnit
This commit removes the command to install PHPUnit on every Travis build job. I'm doing this because I don't see a reason for WooCommerce to re-install PHPUnit since it is already installed by default on every build job. Travis automatically handles installing the right PHPUnit version for each PHP version that we use.

This change should save around 10 seconds from each build job (https://travis-ci.org/woocommerce/woocommerce/jobs/306623161#L515).
2017-11-24 10:23:30 -02:00
Gerhard Potgieter 9f8460c97a
Merge pull request #17887 from woocommerce/fix/17880
Only remove base taxes in cart totals class if item is taxable
2017-11-24 11:21:49 +02:00
Gerhard Potgieter d2e9b36612
Merge pull request #17865 from woocommerce/fix/17852
Exclude spam comments from counts
2017-11-24 11:19:32 +02:00
Gerhard Potgieter 35d33ef9df
Merge pull request #17901 from woocommerce/fix/products-shortcode-post-reference
Updated $GLOBALS['post'] data in products shortcode
2017-11-24 08:50:53 +02:00
David Marín 12bc63d64f
Fix space before parenthesis 2017-11-23 20:01:41 +01:00
David Marín ab6528e844
Update class-wc-admin-list-table-products.php 2017-11-23 19:59:44 +01:00
Claudio Sanches 48132a9dbf Fixed coding standards 2017-11-23 16:43:54 -02:00
Claudio Sanches e40138d188 Updated $GLOBALS['post'] data in products shortcode
No need to pass it as reference.
And passing by reference is breaking Divi's page builder.
2017-11-23 16:40:59 -02:00
Mike Jolley 42acec92b9 Force term setting 2017-11-23 16:00:34 +00:00
Mike Jolley 856ae87d54 Update method 2017-11-23 15:44:17 +00:00
David Marín e21631925a
Swapping filter parameters, to adapt to what @mikejolley proposed 2017-11-23 15:52:02 +01:00
David Marín 4210da0341
Using proposed filter name 2017-11-23 15:49:58 +01:00
Mike Jolley c5a80d080b Rename so delete is hidden 2017-11-23 14:41:17 +00:00
Mike Jolley 3e921e524d Install default cat 2017-11-23 14:38:40 +00:00
Mike Jolley 8dea28e6c2 Save default cat when saving products 2017-11-23 14:26:43 +00:00
Mike Jolley 3dc4277490 Install a default cat 2017-11-23 14:26:34 +00:00
Mike Jolley cd3d61bf98 Add back category filtering which somehow was removed 2017-11-23 14:26:12 +00:00
Rodrigo Primo 5cc84925e3 Typo 2017-11-23 12:17:54 -02:00
Rodrigo Primo 5f0e722ca4 Fix PHPCS violations in includes/wc-update-functions.php 2017-11-23 12:17:21 -02:00
Rodrigo Primo de5a21dcb0 Fix PHPCS violations in includes/class-wc-install.php 2017-11-23 12:16:47 -02:00
Rodrigo Primo e12fa54bc6 phpcbf automatic coding standard fixes 2017-11-23 12:16:47 -02:00
Claudio Sanches 82e803ba76
Merge pull request #17893 from woocommerce/update/phpcs-disable-more-vip-rules
Exclude two more PHPCS VIP rules
2017-11-23 12:08:29 -02:00
Rodrigo Primo 322619cdc5 Exclude two more PHPCS VIP rules 2017-11-23 12:06:00 -02:00
Claudio Sanches bdcc8a28f9
Merge pull request #17892 from woocommerce/update/phpcs-remove-some-vip-rules
Remove three PHPCS VIP rules that don't apply to WooCommerce
2017-11-23 11:45:21 -02:00
Rodrigo Primo f5d2a1d98b Remove three PHPCS VIP rules that don't apply to WooCommerce
This commit removes the following PHPCS VIP rules:

- WordPress.VIP.DirectDatabaseQuery.NoCaching
- WordPress.VIP.DirectDatabaseQuery.SchemaChange
- WordPress.VIP.FileSystemWritesDisallow.file_ops_fwrite
2017-11-23 11:41:31 -02:00
Mike Jolley ee08983e5e
Merge pull request #17890 from rodrigoprimo/update/remove-xdebug-from-php-5-3-build
Further improve the speed of the Travis build
2017-11-23 13:24:51 +00:00
Mike Jolley 85cd9647df
Merge pull request #17871 from ragulka/delete-refund-on-exception
Ensure refund is deleted when exception is thrown during wc_create_refund()
2017-11-23 13:06:40 +00:00
Mike Jolley b7d8d63c3d
Merge pull request #17870 from woocommerce/update/bump-wp-version
Bump "Requires at least" version
2017-11-23 11:52:48 +00:00
The Dramatist 8ac49d448b Unused parameter removed form get_filters method of WC_Helper class. (#17888) 2017-11-23 11:49:12 +00:00
David Marín 654a036f1a
Added third parameter with taxonomy to filter call 2017-11-23 12:43:37 +01:00
David Marín 50f3db7693
Changed filter so it receives the array instead of the final imploded string 2017-11-23 12:40:00 +01:00
Mike Jolley 5a4e85b5f5 Tweak total_comments 2017-11-23 11:17:23 +00:00
Rodrigo Primo 4fea2eb211 Disable Xdebug before calling composer 2017-11-23 09:13:09 -02:00
Mike Jolley 6ef391666f Only remove base taxes in cart totals class if item is taxable
Fixes #17880
2017-11-23 11:08:10 +00:00
Rodrigo Primo 51867ac7c3 Remove Xdebug from PHP 5.3 Travis build job as well 2017-11-23 09:06:08 -02:00
Shiva Poudel a0215d85da Txn clickable (#17878)
* Make transactional link clickable

* Oops, fix in modal tmpl
2017-11-23 10:18:23 +00:00
urosmil 18e7b25eab Filter for line items in abstract-wc-order.php (#17759)
* Filter for line items in abstract-wc-order.php

When creating custom product class there are places where line items is only value that can be handled. This is one of that places causing subtotal values after checkout not to be calculated and displayed.

* Extension to woocommerce_order_get_items filter

This is extension to filter as get_items() is used for different classes and types. This way it would be possible to distinct which type is called for items.

* Removed filtering inside subtotal function

$types parameter is added as parameter for filter inside get_items() function so previous commit is reverted as there is no need for it.
2017-11-23 10:00:03 +00:00
David Marín 55972f4623
Changing the filter name to 'admin_term_links' 2017-11-23 09:33:55 +01:00
Claudiu Lodromanean 9a95441c61
Merge pull request #17872 from woocommerce/fix/17508
Properly handle deprecated hooks
2017-11-22 14:34:15 -08:00
claudiulodro a878b0cde5 Some phpcs 2017-11-22 14:03:26 -08:00
claudiulodro 7e52af8b9a Fix logic 2017-11-22 11:57:59 -08:00