Commit Graph

23678 Commits

Author SHA1 Message Date
Mike Jolley 4379a47cd5 Fixed discount logic to also look at parent categories when validatiing coupons
Fixes #17150
2017-10-12 15:40:58 +01:00
Mike Jolley 5252d93465 Merge pull request #17154 from woocommerce/add/test-variation-totals
Update totals test to include a variation with parent tax class.
2017-10-12 15:34:59 +01:00
Mike Jolley 88ea1eaf7b Remove rounding in wc_remove_number_precision
Remove the rounding when removing precision so the cart can handle it.

For taxes, this means it can round down.

Fixes #17158

Tests are passing.
2017-10-12 15:16:48 +01:00
Mike Jolley b0631a040c Merge pull request #17157 from woocommerce/add/tests-variations-tax-classes
WC_Product_Variation::get_tax_class unit tests
2017-10-12 14:47:35 +01:00
Mike Jolley 878e4816fb Use city and postcode count to determine sort order 2017-10-12 14:04:10 +01:00
Gerhard Potgieter 8045596ed7 Add get_tax_class tests for WC_Product_Variation. Will ensure #17147 never happens again. 2017-10-12 14:46:28 +02:00
Gerhard Potgieter 9eee5afc2c Be sure to add the new product for cleanup later 2017-10-12 13:33:56 +02:00
Mike Jolley 2db0d1d4d6 Merge pull request #17151 from woocommerce/fix/shipping-rate-save
Fix tax rate save when only changing shipping checkbox
2017-10-12 12:33:46 +01:00
Mike Jolley cffcaaf773 wc-admin-order-meta-boxes as a selectwoo dependency 2017-10-12 12:32:36 +01:00
Gerhard Potgieter 91aa76a51b Update totals test to include a variation with parent tax class. 2017-10-12 13:30:08 +02:00
Mike Jolley 6aca58324a Merge pull request #17152 from woocommerce/fix/lazy-load-embeds
Load videos when the help tab is opened, not before.
2017-10-12 12:28:00 +01:00
Mike Jolley 819951815e Use base text color for links in emails 2017-10-12 12:27:15 +01:00
Mike Jolley 47d69dad2d Load videos when the help tab is opened, not before.
Closes #17145
2017-10-12 12:18:37 +01:00
Mike Jolley 43f1938629 Merge pull request #17149 from woocommerce/fix/17147
Tax calculations should use `unfiltered` context for tax class
2017-10-12 11:38:35 +01:00
Mike Jolley 5a22c7fa97 Fix tax rate save when only changing shipping checkbox 2017-10-12 11:30:41 +01:00
Mike Jolley 0908bc0b13 phpcs 2017-10-12 11:26:55 +01:00
Mike Jolley 4f3393fc2f Tax calculations should use `unfiltered` context for tax class
This fixes variations which have a ‘parent’ tax class. It should
inherit from the parent.

Closes #17147
2017-10-12 10:12:20 +01:00
Claudio Sanches 8ead48fc64 Merge pull request #17144 from woocommerce/fix/grouped-menu-order
Remove menu-ordering sorting
2017-10-11 17:59:26 -03:00
Mike Jolley a6059ea759 Remove menu-ordering sorting 2017-10-11 21:36:13 +01:00
Chris Schrijver dee7a8c8d4 If no tax rate is defined for a specific country it still shows "incl. VAT" at the checkout / cart when tax display is set to "As single total". This will solved that 2017-10-11 17:58:43 +02:00
grosssmutttr 1ccec7d156 class-wc-widget-layered-nav.php - Crawling Budget
Additional optimization to save up crawling budget.
Added $link = str_replace('%2C', ',', $link); on line 513

https://mydomain.com/produkt-kategorie/my-category/?filter_filter-color=orange,black

If you add another filter it encodes the existing commas:
https://mydomain.com/produkt-kategorie/my-category/?filter_filter-color=orange%2Cblack&filter_manufacturer=adidas&filter_cut=men.children

So sometimes you have "," as seperator and sometimes you have encoded "%2C".

After implementing this pull-request the seperator stays ",":
https://mydomain.com/produkt-kategorie/my-category/?filter_filter-farbe=orange,black&filter_manufacturer=adidas&filter_cut=men.children

This pull request belongs to the optimization https://github.com/woocommerce/woocommerce/pull/17134 to save up crawling budget,
2017-10-11 14:47:00 +02:00
Mike Jolley ce47fa65c2 Changelog + grunt 2017-10-11 10:50:39 +01:00
Mike Jolley 12d93803ee Merge pull request #17121 from woocommerce/update/wizard-payments-copy
Update payment step copy for Stripe, and fixes margins/spacing
2017-10-11 10:17:57 +01:00
Mike Jolley ec9385a148 Merge pull request #17130 from woocommerce/rodrigoprimo-patch-1
Quote code in the tests README.md file
2017-10-11 10:04:04 +01:00
grosssmutttr d9f0bd5e92 Keep link structure - prevent duplicate content
Added asort($current_filter) so the urls always look the same and there are not millions of urls possibilities when adding multiple filters.

Current url example:
https://myurl.com/produkt-kategorie/mycategory/?filter_filter-color=yellow,brown,blue,red
https://myurl.com/produkt-kategorie/mycategory/?filter_filter-color=yellow,blue,red,brown
https://myurl.com/produkt-kategorie/mycategory/?filter_filter-color=blue,yellow,red,brown
...

With asort you only have this url:
https://myurl.com/produkt-kategorie/mycategory/?filter_filter-color=blue,brown,red,yellow

So it is independent on the sequence a user adds a filter
2017-10-11 08:33:46 +02:00
Rodrigo Primo d862aaf29d Quote code in the tests README.md file
Use triple backticks to quote code in the tests README.md file. Without this change all the parameters of the script `tests/bin/install.sh` that are surrounded by `<>` are not displayed.
2017-10-10 15:07:24 -03:00
Marcin Bot 17e4f2a609 Merge pull request #17127 from woocommerce/fix/17122-setup-required-shipping-fields
Setup wizard - made the shipping cost fields required
2017-10-10 18:50:20 +02:00
Marcin Bot 01578d55c9 Wrapped the class name in esc_attr 2017-10-10 17:49:04 +01:00
Mike Jolley ae4838c073 Merge pull request #17128 from woocommerce/fix/gruntfile
Fixed extra comma in Gruntfile.js
2017-10-10 17:20:41 +01:00
Claudio Sanches 5244340bae Fixed extra comma in Gruntfile.js 2017-10-10 13:08:23 -03:00
Marcin Bot eaf8a09e82 Setup wizard - made the shipping cost fields required 2017-10-10 16:59:08 +01:00
Mike Jolley 31367a72b8 Merge pull request #17123 from woocommerce/update/wizard-extras-copy
Setup wizard: add “powered by” to automated taxes description.
2017-10-10 16:39:45 +01:00
Mike Jolley c6d0a6e0be Merge pull request #17124 from cliffordp/patch-1
Correct docblock @return
2017-10-10 16:38:26 +01:00
Clifford 1474350f0f Correct docblock @return
From incorrect `string` to correct `array`
2017-10-10 10:31:16 -05:00
Jeff Stieler 24599524ba Setup wizard: add “powered by” to automated taxes description. 2017-10-10 09:24:13 -06:00
Mike Jolley 6b357bcfdc Updates the payment step copy and fixes margins 2017-10-10 15:40:03 +01:00
Mike Jolley d4965e9df4 Merge pull request #17112 from woocommerce/fix/product-variations-tax-classes
Add 'parent' to the list of valid tax classes for product variations
2017-10-10 12:32:29 +01:00
Jack 24efd36126 set query_vars (#17105)
* query_vars is set

* add changes
2017-10-10 12:14:35 +01:00
Mike Jolley 6098e25c9a Merge pull request #17102 from woocommerce/fix/thanks-page-classname
Classname was using total instead of email
2017-10-10 11:43:47 +01:00
Jeff Stieler 974f402017 Fix duplicate plugin install from wizard. (#17114)
* Setup wizard: avoid duplicate plugin installs.
* Setup wizard: use background flag to determine if Jetpack is being installed.
* Setup wizard: use wc_set_time_limit() instead of set_time_limit() to avoid potential warnings/errors.
2017-10-09 16:45:33 -06:00
Claudio Sanches 87cf8f4809 Merge pull request #17106 from woocommerce/fix/activate-step-when-returning-from-wpcom
Setup wizard: activate step shouldn't be hidden after a successful WPCOM connection.
2017-10-09 16:59:53 -03:00
Rodrigo Primo fa0e5569f8 Add 'parent' to the list of valid tax classes for product variations
Commit d9f9e74bd added a check to `WC_Product::set_tax_class()` to only accept valid tax classes, but this created a bug for product variations as this type of product has an extra tax class called 'parent'.

This commit fixes this problem by adding a new method to `WC_Product` that returns a list of valid tax classes. `WC_Product_Variation` then override this method, returning another list including the tax class 'parent'.

Fixes #17024
2017-10-09 16:20:03 -03:00
Rodrigo Primo d769ef24f2 Coding standards fixes 2017-10-09 16:19:24 -03:00
Mike Jolley 6d7522d864 Merge pull request #17075 from woocommerce/fix/phpunit-excluded-files
Updated tests exclusion list
2017-10-09 19:46:34 +01:00
Mike Jolley 6a035995a6 Merge pull request #17108 from woocommerce/update/phpcs-i18n
PHPCS - Check for translatable strings placeholders
2017-10-09 19:45:47 +01:00
Mike Jolley 30c98aaa2d Merge pull request #17107 from woocommerce/update/phpcs-check-for-xss-and-sanitization
PHPCS - Check for CSRF, sanitization and XSS.
2017-10-09 19:45:18 +01:00
Claudio Sanches 7ce5d9e1c6 Merge pull request #17109 from woocommerce/fix/17085-2
Translate "No matches found" message in filter widget
2017-10-09 15:39:51 -03:00
claudiulodro f1dac6f3ac Use esc_js instead of addslashes 2017-10-09 11:37:09 -07:00
Mike Jolley 6838b00c38 Merge pull request #17110 from woocommerce/fix/17104
Allow display_name to be searchable - this will commonly be full name
2017-10-09 19:29:26 +01:00
Mike Jolley fae130d038 Allow display_name to be searchable - this will commonly be full name
Closes #17104
2017-10-09 19:20:22 +01:00