This commit removes the following PHPCS VIP rules:
- WordPress.VIP.DirectDatabaseQuery.NoCaching
- WordPress.VIP.DirectDatabaseQuery.SchemaChange
- WordPress.VIP.FileSystemWritesDisallow.file_ops_fwrite
* 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.
PR #17680 added a new PHP 7.1 Travis build job to generate code coverage report. PHPCS was configured to run on all PHP 7.1 build jobs. So this means that after #17680 was merged, Travis started running PHPCS twice.
This commit fixes this issue by setting a new environment variable called `$RUN_PHPCS` and using this variable, instead of the PHP version, to decide when to run PHPCS.
* Data escaped and validated. Plus translator comment documentation added.
* If it should be int then we should change the empty string default parameter to 0 as in 'if' condition checking 0 and empty string are same.
* Extra bracket removed.
* Indentation is fixed now.
* The taxonomy object should not be assigned to a variable before checking if this is a taxonomy or not.
* echo esc_attr__ replaced by esc_attr_e and visually tested.
* As I use WordPress VIP Coding Standard, they were marking them as warnings. But now it's fixed again.
* Escaping fix
* Extra Bracket Fix Again
This commit changes Travis configuration file to specify a list of branches that it should build. This way Travis will only create a new build when a commit is pushed to the master branch and release branches and when tags are created. The behavior for PRs is not modified and PR creation and updates will still trigger builds.
Doing this to speed up Travis build by preventing it from running two builds for PR that are created from a branch of the same repository. Before this change, Travis would create a build when the branch is pushed to GitHub (continuous-integration/travis-ci/push) and another one when the PR is created (continuous-integration/travis-ci/pr). For an example, see https://github.com/woocommerce/woocommerce/pull/17680.
Claudio Sanches
Rodrigo Primo
Mike Jolley
The Dramatist
urosmil
Claudiu Lodromanean
ragulka
Ahmed Ali
Shiva Poudel
JeroenSormani