Steve-Dee-Designs
/
woocommerce
mirror of https://github.com/woocommerce/woocommerce.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use get_the_terms() instead of wp_get_post_terms() 

wp_get_post_terms() is a wrapper around wp_get_object_terms() which does not
use the object cache, and generates a database query every time it is used.

get_the_terms() however can use data from the object cache if present.
This commit is contained in:
Lee Willis 2016-11-09 11:27:24 +00:00
parent ad37a68ffb
commit 2c2239a3b0
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
includes/abstracts/abstract-wc-product.php
View File

@ -1235,7 +1235,11 @@ class WC_Product extends WC_Abstract_Legacy_Product {
* @return array of terms
*/
protected function get_term_ids( $taxonomy ) {
return wp_get_post_terms( $this->get_id(), $taxonomy, array( 'fields' => 'ids' ) );
$terms = get_the_terms( $this->get_id(), $taxonomy );
if ( false === $terms || is_wp_error( $terms ) ) {
return array();
}
return wp_list_pluck( $terms, 'term_id' );
}
/**