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,
This commit is contained in:
grosssmutttr 2017-10-11 14:47:00 +02:00 committed by GitHub
parent ce47fa65c2
commit 1ccec7d156
1 changed files with 1 additions and 0 deletions

View File

@ -510,6 +510,7 @@ class WC_Widget_Layered_Nav extends WC_Widget {
if ( 'or' === $query_type && ! ( 1 === sizeof( $current_filter ) && $option_is_set ) ) {
$link = add_query_arg( 'query_type_' . sanitize_title( str_replace( 'pa_', '', $taxonomy ) ), 'or', $link );
}
$link = str_replace('%2C', ',', $link);
}
if ( $count > 0 || $option_is_set ) {