Commit Graph

351 Commits

Author SHA1 Message Date
Rodrigo Primo 86c1b61f03 Fix Generic.WhiteSpace.ScopeIndent violations
This commit fixes all violations of the Generic.WhiteSpace.ScopeIndent sniff automatically using phpcbf.
2019-12-20 14:48:11 -03:00
Rodrigo Primo 62d6b05b58 Fix Generic.Formatting.MultipleStatementAlignment violations
This commit fixes all violations of the Generic.Formatting.MultipleStatementAlignment sniff automatically using phpcbf.
2019-12-20 14:21:08 -03:00
Rodrigo Primo 94373f78ea Fix WordPress.Arrays.MultipleStatementAlignment errors automatically using phpcbf 2019-12-20 13:51:26 -03:00
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
aslamshekh 3df0769366 I have done with few changes which is suggested. 2019-07-10 14:52:59 +05:30
aslamshekh 15eaaebdeb Fix-24115, Downloads report not getting 503 any more, If previously downloaded files are removed from product 2019-07-10 12:58:07 +05:30
Rodrigo Primo f3b0d755fd Correct singular form usage in class-wc-admin-report.php
For more information see https://github.com/woocommerce/woocommerce/issues/24004
2019-06-27 11:03:35 -03:00
Rodrigo Primo 3f2cacb1be Use phpcbf to fix phpcs violations in class-wc-admin-report.php 2019-06-27 10:48:31 -03:00
Mike Jolley 1504f8af3e
Merge pull request #23011 from Tofandel/patch-2
Improve the speed of the admin dashboard by only updating transients once per class
2019-04-15 14:03:31 +01:00
Adrien Foulon 700d89f8ff
Fix missing parenthesis 2019-03-14 23:55:02 +01:00
Adrien Foulon d53d6f6a7d
phpcs 2019-03-14 23:18:47 +01:00
Adrien Foulon 7a470fb117
Separating nocache and cache logic for clarity 2019-03-14 23:16:55 +01:00
Adrien Foulon 0391b11b20
Review 2019-03-14 23:05:54 +01:00
Adrien Foulon 7d77241ed2
Move variable definitions in class 2019-03-14 16:44:26 +01:00
Adrien Foulon 005d7491cf
Cleanup 2019-03-13 22:28:09 +01:00
Adrien Foulon 55339a2870
Cache setter and getter wrapper 2019-03-13 22:21:09 +01:00
Adrien Foulon 8d866d9df7
Fix error wpdb global 2019-03-13 21:19:21 +01:00
Adrien Foulon 322eed0b2c
Fix error 2019-03-13 19:06:47 +01:00
Adrien Foulon 11e8f087e8
Fix get cached query call 2019-03-13 18:18:59 +01:00
Adrien Foulon 99358e7763
Remove whitespaces on empty lines 2019-03-13 18:00:13 +01:00
Adrien Foulon eefbaff3ea
Remove whitespaces on empty lines 2019-03-13 17:59:34 +01:00
Adrien Foulon cc65b2d856
Apply code review 2019-03-13 17:54:13 +01:00
Adrien Foulon 80435d779f
Apply code review 2019-03-13 17:52:28 +01:00
Adrien Foulon fa9b08c9a4
Make the maybe_update_transients hook static so it's not constructor dependant
Because extended classes don't call the parent constructor and that would break BC
2019-03-13 13:47:00 +01:00
Adrien Foulon bc34c6e3eb
Fix php5.2 support 2019-03-13 12:11:49 +01:00
Adrien Foulon a520cb5f44
Rename function 2019-03-13 11:55:18 +01:00
Adrien Foulon b2ad443db5
phpcs 2019-03-13 11:51:48 +01:00
Adrien Foulon 95cbe5c969
phpcs 2019-03-13 11:51:24 +01:00
Adrien Foulon 13b5a07383
Update class-wc-admin-report.php 2019-03-13 11:35:42 +01:00
Adrien Foulon ee403d6000
Improve the speed of the admin dashboard by only updating transients once per class
While debugging the admin dashboard I found out that some transients where updated up to 5 times during the page load, reading is fast but writting is costy, so the query where adding a total of 200ms to the page load (because they are also huge transients)

I've rewritten the transient storage procedure so that per class a transient is only written once at the end of the request

I also found that a continue was changed for a break line 139 for php7.3 and the new warning it incurs, the problem is it was changed the  wrong way, it should have been a "continue 2" to continue the foreach and not a break.

That makes me wonder if some others like this may have slipped in the code.
2019-03-13 11:33:03 +01:00
Mike Jolley ef352c2b48 Outofstock lookup 2019-03-13 10:10:24 +00:00
Mike Jolley b4aa0c3915 use found rows 2019-03-13 10:06:49 +00:00
Mike Jolley f5bd3be2c2 most stocked lookup table 2019-03-13 09:59:14 +00:00
Mike Jolley ac21a4bc33 phpcs 2019-03-12 22:26:53 +00:00
Mike Jolley 3b98d2de8d Switch low stock report to lookup table
Baseline page load time down from 60s to 0.7s
2019-03-12 22:26:46 +00:00
Rodrigo Primo c4d6f3dac3 Replace deprecated WPCS rule
This commit replaces all the instances where the WPCS rule `WordPress.VIP.SlowDBQuery` was used with `WordPress.DB.SlowDBQuery`. `WordPress.VIP.SlowDBQuery` was deprecated in WPCS 2.0 and `WordPress.DB.SlowDBQuery` was added as a replacement (see 5f27d8e3a4/CHANGELOG.md (L130)).
2019-03-12 11:24:09 -03:00
Mike Jolley 7def966e4f Impoved JSON handling 2019-02-20 12:00:47 +00:00
Ron Rennick b92cfc51a5 update product table to correctly calculate net product sales 2019-02-11 16:34:28 -04:00
Mike Jolley 8ed1dacdc9
Merge pull request #22581 from woocommerce/fix/22561
include refunded orders in top sellers, earners in sales by product report
2019-01-31 13:51:51 +00:00
Gerhard a2e4ca70b6 PHPCS fixes 2019-01-30 11:33:51 +02:00
Gerhard 429f515704 Introduce woocommerce_admin_report_customers_user_query_args and woocommerce_admin_report_customer_list_user_query_args filters to altering the queries running when retrieving users for reports. 2019-01-30 11:19:34 +02:00
Ron Rennick a93418b826 phpcs sniff fixes for class-wc-report-sales-by-product.php 2019-01-27 19:27:30 -04:00
Ron Rennick f42c42066b include refunded orders in top sellers, earners in sales by product report 2019-01-27 19:24:08 -04:00
Gerhard Potgieter f77e2d7316 Handle full refunds in the taxes by code report (#21843)
* Add full refund support to the taxes by code report.

* Some PHPCS fixes

* Do not include refund in initial tax lookup for report.

* Correct logic which loops orders

* match date report
2019-01-23 15:12:58 +00:00
Ron Rennick c3879f0d63 only deduct refunded shipping, taxes once from gross to get net sales by date CSV 2019-01-21 15:44:53 -04:00
Ron Rennick aa9961695f phpcs sniff fixes for class-wc-report-orders-by-date.php 2019-01-21 15:36:29 -04:00
Gerhard cf38ed4aba Remove on-hold orders from tax reports as they are not paid yet. Tax reports should only contain paid orders. 2019-01-11 14:44:09 +02:00
Rodrigo Primo a9e532022a Replace `continue` inside `switch` statement with `break`
PHP 7.3 will generate a warning when `continue` is used inside a `switch` statement. For more information see the relevant section in https://make.wordpress.org/core/2018/10/15/wordpress-and-php-7-3/
2018-11-20 16:24:11 -02:00
Rodrigo Primo f109008e0d Revert "Update wc_customer_bought_product() and customers report to use post_author instead of _customer_user"
This reverts commit b97f2749d6.
2018-10-29 11:15:22 -03:00
Claudio Sanches 6d54a533ae Fixed customer list query 2018-09-08 21:21:00 -03:00