Commit Graph

29217 Commits

Author SHA1 Message Date
Claudio Sanches a43ae35a6e
Merge pull request #22273 from woocommerce/fix/22271
Fix orders list in the admin
2018-12-20 11:08:01 -02:00
Rodrigo Primo ff74ecec86 Fix orders list in the admin
WP 5.0.2, which was released yesterday, sets `$query_vars['post_status']` to an empty string instead of leaving it unset (https://core.trac.wordpress.org/ticket/45719 and https://core.trac.wordpress.org/changeset/44185). This change broke the WC orders list in the admin when listing orders of all post statuses as we were checking if `$query_vars['post_status']` was not set to add all order statuses to this variable (125eaaf241/includes/admin/list-tables/class-wc-admin-list-table-orders.php (L799)).

To workaround this problem, this commit changes the check to verify if `$query_vars['post_status']` is empty instead of verifying if it is not set.
2018-12-20 09:56:02 -02:00
Rodrigo Primo 948a1dc300 Fix PHPCS violations automatically using PHPCBF 2018-12-20 09:48:14 -02:00
Rodrigo Primo 125eaaf241 Merge branch 'master' of github.com:woocommerce/woocommerce 2018-12-19 13:31:34 -02:00
Rodrigo Primo 941217719e phpcs fixes using phpcbf 2018-12-19 13:31:11 -02:00
Rodrigo Primo cb0c23d6ab
Merge pull request #22244 from faisal-alvi/fix-22234
Fix-22234
2018-12-19 12:42:48 -02:00
Rodrigo Primo dd816cf8d1 Merge branch 'master' of github.com:woocommerce/woocommerce 2018-12-19 12:40:33 -02:00
Rodrigo Primo 0cb992925d phpcs fix using phpcbf 2018-12-19 12:40:02 -02:00
Rodrigo Primo eccaa3d286
Merge pull request #22156 from rodrigoprimo/split-travis-build-job
Use separate Travis build jobs to run e2e tests and PHPCS
2018-12-19 11:30:49 -02:00
Rodrigo Primo c6a0c32427 Merge branch 'master' into split-travis-build-job 2018-12-19 11:09:46 -02:00
Rodrigo Primo 87e47cd1a8 Use the "name" config property to better identify build jobs 2018-12-19 11:08:32 -02:00
Rodrigo Primo 753ced917f
Merge pull request #22252 from Prospress/fix/22251
Update end to end tests
2018-12-19 10:37:26 -02:00
Ron Rennick e04b0fff31 use a more appropriate description for new installation tests 2018-12-18 16:28:49 -04:00
Ron Rennick 6079e1f541 ensure WP database is upgraded for current WP version 2018-12-18 16:02:57 -04:00
Ron Rennick e2f34928d3 update checkout flow for 3.5.0 2018-12-18 14:23:11 -04:00
Ron Rennick 603dd8cfcd remove redundant form update in cart page tests 2018-12-17 16:05:53 -04:00
Ron Rennick 3ab427f8bf add config-theme.js to test that a functional supported theme is active 2018-12-17 15:42:20 -04:00
Ron Rennick 729e7083ae move front end tests to front-end folder 2018-12-17 15:19:34 -04:00
Faisal Alvi cbafb8e006 Fix-22234 2018-12-17 15:01:05 +05:30
Rodrigo Primo f13a0bc9eb
Merge pull request #22216 from woocommerce/fix/21894
Introduce `woocommerce_reviews_title` filter
2018-12-13 17:15:14 -02:00
Rodrigo Primo 1a8dcad2e8 Bump template version 2018-12-13 16:28:15 -02:00
Rodrigo Primo 8c9bd03692 Fix phpcs violations and improve indentation 2018-12-13 16:14:14 -02:00
Ron Rennick 73de71d8d4 fix merge conflict 2018-12-12 09:23:38 -06:00
Claudiu Lodromanean c4ae48597c
Merge pull request #21904 from Prospress/fix/21196
allow limited html in `woocommerce_rating_filter_count` filter
2018-12-11 11:18:19 -08:00
Claudiu Lodromanean c3334302bd
Merge pull request #22108 from woocommerce/fix/meta-queries-REST-API
Fixed support for multiple query parameters translated to meta queries via REST API requests
2018-12-11 11:12:56 -08:00
Claudiu Lodromanean 6e457799b3
Merge pull request #21943 from woocommerce/fix/21516
Match array keys on lowercase keys before calling set_props
2018-12-11 11:12:22 -08:00
Claudiu Lodromanean b70fed6868
Merge pull request #21669 from woocommerce/fix/helper-not-able-to-detect-local-woo-plugin
Add File Headers filter at 'plugins_loaded' hook so that WooCommerce Helper can detect all locally installed Woo plugins
2018-12-11 11:08:49 -08:00
Claudiu Lodromanean f6c3766cf0
Merge pull request #22139 from aslamshekh/fixed-wordpress-standard-1
Fixed Wordpress standard as per given suggestion on #22122
2018-12-10 14:12:07 -08:00
Claudiu Lodromanean d1e71994dc
Merge pull request #22170 from faisal-alvi/master
`abstract-wc-widget.php` - coding standard fixes
2018-12-10 14:06:38 -08:00
Ron Rennick 34bdc3744a phpcs sniff fixes for templates/single-product-reviews.php 2018-12-07 09:59:01 -06:00
Ron Rennick 52f13f8708 fix phpcs sniffs in class-wc-widget-rating-filter.php 2018-12-07 09:27:56 -06:00
Rodrigo Primo 0b413351a2
Merge pull request #22031 from Prospress/fix/22027
fix PHP 5.2 sniffs in class-wc-install.php
2018-12-07 11:43:46 -02:00
Faisal Alvi 8dbb795610 abs-wc-widget-coding-stds
Few coding standard fixes in the file abstract-wc-widget.php
2018-12-07 18:57:03 +05:30
Rodrigo Primo 5da1b72464
Merge pull request #21913 from Prospress/fix/21909
Capitalize dashboard widget titles
2018-12-07 11:24:25 -02:00
Faisal Alvi 6fd8971b8d
Merge pull request #2 from woocommerce/master
Updating to head
2018-12-07 18:47:32 +05:30
Claudio Sanches 7cbcaf04e8
Merge pull request #22121 from woocommerce/fix/update-settings-controller
Fix updating settings via the API
2018-12-06 16:49:44 -02:00
Rodrigo Primo 99b633d1e5
Merge pull request #22028 from woocommerce/phpcs-wp-db
Update WPCS sniff names
2018-12-06 15:56:12 -02:00
Peter Fabian dc39d265fd Merge branch 'master' into fix/update-settings-controller 2018-12-06 16:28:46 +01:00
Rodrigo Primo 164758a5f0
Merge pull request #22110 from stromhalm/patch-1
Properly close option tag in customer search
2018-12-06 12:18:54 -02:00
Bradley Davis e97204a483 Removed @author tag from docbloc in the template files 2018-12-06 12:13:39 -02:00
Rodrigo Primo e7296aec03
Merge pull request #22081 from rvdsteege/patch-1
Fix incorrect return type documentation for product factory `get_product()`.
2018-12-06 12:09:52 -02:00
Rodrigo Primo a52e72f87f
Merge pull request #22155 from grigorijurasov/patch-1
Line 27 - replacing a typo in the comment
2018-12-06 11:07:37 -02:00
Rodrigo Primo 644f6b1cba Use separate Travis build jobs to run e2e tests and PHPCS
This commit creates two new Travis build jobs, one to run the e2e tests and another to run PHPCS. Doing this, instead of running those two checks in the same build job as the PHP 7.2 unit tests, should make the total build time shorter and it should make it easier to see why the build failed.
2018-12-06 09:14:13 -02:00
grigorijurasov adbce426c5
Line 27 - replacing a typo in the comment
Line 27 - there's currently a typo, `numer`, changing that to `number`
2018-12-06 12:38:42 +02:00
Rodrigo Primo 62faa2dceb
Merge pull request #22140 from rodrigoprimo/fix/travis-php7
Make sure PHPUnit tests runs when using PHP 7.2+ in the Travis builds
2018-12-06 08:24:37 -02:00
Rodrigo Primo 522df96217 Make sure PHPUnit tests runs when using PHP 7.2+ in the Travis builds
Due to a recent change in the Travis environment, WooCommerce unit tests stopped working with the following error (see for example https://travis-ci.org/woocommerce/woocommerce/jobs/463470674#L876) in the PHP 7.2 and 7.3 build jobs:

Fatal error: Class PHPUnit_Util_Test may not inherit from final class (PHPUnit\Util\Test) in /tmp/wordpress-tests-lib/includes/phpunit6-compat.php on line 18

This error is happening because Travis started ignoring the PHPUnit version that we install manually via Composer (f7bc3fb851/tests/bin/travis.sh (L6)) and started using the PHPUnit version that is shipped with each of its PHP docker images. This means that for the docker images running PHP 7.2 and 7.3, PHPUnit 7 is used but the WordPress unit test framework is not compatible with PHPUnit 7 (see WordPress core ticket https://core.trac.wordpress.org/ticket/43218) and produces the error above. I believe that this is happening because Travis changed the directory where it installs composer global packages from `$HOME/.composer/` to `$HOME/.config/composer/` (https://github.com/travis-ci/travis-ci/issues/7289#issuecomment-427333966) and we add `$HOME/.composer/vendor/bin:$PATH` to the `$PATH`. So this commit simply updates the path in the line where we add it to the `$PATH`.

I tried to use `composer exec` instead of updating `$PATH` but that didn't work for PHP 5.2.
2018-12-05 15:15:49 -02: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 f7bc3fb851
Merge pull request #21366 from birgire/unit-tests-count-asserting-21365
Use assertCount instead of assertEquals
2018-12-05 11:15:52 -02:00
Rodrigo Primo a91bc5b1ec phpcs fixes using phpcbf 2018-12-04 16:04:24 -02:00
Rodrigo Primo 24fd6f89c6 Merge branch 'master' into phpcs-wp-db 2018-12-04 15:39:19 -02:00