Как правильно делать remove_action()?

Question

[Сергей Шилов]

Всем привет! Я купил шаблон одному клиенту и пытаюсь его хоть как-то оптимизировать по скорости. Так вот, по дефолту тема дает возможность быстрого просмотра у картчоки товара, но я хочу её убрать, предполагаю, что это делается через remove_action();

Вот как выглядит код в файле темы:

<?php
/**
 * Cherry WooCommerce quick view module
 *
 *
 * @author 		Cherry Team
 * @category 	Core
 * @package 	cherry-woocommerce-package/functions
 * @version     1.0.0
 */

if ( ! defined( 'ABSPATH' ) ) {
	exit();
}

/**
 * Menu badges management class
 *
 * @since 1.1.0
 */
class cherry_wc_quick_view {

	function __construct() {
		add_action( 'wp_enqueue_scripts', array( $this, 'assets' ), 99 );
		add_action( 'woocommerce_before_shop_loop_item_title', array( $this, 'append_open_wrap' ), 0 );
		add_action( 'woocommerce_before_shop_loop_item_title', array( $this, 'append_close_wrap' ), 100 );
		add_action( 'woocommerce_before_shop_loop_item_title', array( $this, 'append_button' ), 99 );

		if ( is_admin() ) {
			add_action( 'wp_ajax_cherry_wc_quick_view', array( $this, 'process_ajax_request' ) );
			add_action( 'wp_ajax_nopriv_cherry_wc_quick_view', array( $this, 'process_ajax_request' ) );
		}
	}

	/**
	 * Enqueue necessary CSS and JS
	 *
	 * @since 1.1.0
	 */
	function assets() {
		global $cherry_woocommerce;
		if ( !wp_script_is( 'magnific-popup', 'enqueued' ) ) {
			wp_enqueue_script( 'magnific-popup', $cherry_woocommerce->url( 'jquery.magnific-popup.min.js' ), array( 'jquery' ), $cherry_woocommerce->version, true );
		}
		if ( !wp_style_is( 'magnific-popup', 'enqueued' ) ) {
			wp_enqueue_style( 'magnific-popup', $cherry_woocommerce->url( 'assets/css/magnific-popup.css' ), '', $cherry_woocommerce->version, 'all' );
		}
		if ( !wp_script_is( 'cherry_prettyPhoto', 'enqueued' ) ) {
			wp_enqueue_script( 'prettyPhoto', $cherry_woocommerce->url( 'assets/js/jquery.prettyPhoto.min.js' ), '', $cherry_woocommerce->version, 'all' );
		}
		if ( !wp_style_is( 'cherry_prettyPhoto_css', 'enqueued' ) ) {
			wp_enqueue_style( 'cherry_prettyPhoto_css', $cherry_woocommerce->url( 'assets/css/prettyPhoto.css' ), '', $cherry_woocommerce->version, 'all' );
		}
	}

	/**
	 * Append open wrapper for quick view
	 *
	 * @since 1.1.0
	 */
	function append_open_wrap() {
		echo '<div class="cherry-thumb-wrap">';
	}

	/**
	 * Append open wrapper for quick view
	 *
	 * @since 1.1.0
	 */
	function append_close_wrap() {
		echo '</div>';
	}

	/**
	 * Append quick view button to product listing template
	 *
	 * @since 1.1.0
	 */
	function append_button() {
		global $post, $product;

		$btn_tex = apply_filters( 'cherry_wc_quick_view_text', __( 'посмотреть', 'cherry-woocommerce-package' ) );

		echo '<span class="btn cherry-quick-view" data-product="' . $product->id . '">' . $btn_tex . '</span>';

		wp_enqueue_script( 'prettyPhoto' );
	}

	/**
	 * Process AJAX request and return quick view popup content
	 *
	 * @since 1.1.0
	 */
	function process_ajax_request() {

		$verify_nonce = check_ajax_referer( 'cherry_wc_data', '_wpnonce', false );

		$error_message = array( 'content' => __( 'Error!', 'cherry-woocommerce-package' ) );

		if ( !$verify_nonce ) {
			wp_send_json( $error_message );
		}

		$product_id = isset( $_REQUEST['product'] ) ? $_REQUEST['product'] : false;

		if ( !$product_id ) {
			wp_send_json( $error_message );
		}

		if ( !class_exists( 'WC_Product_Factory' ) ) {
			wp_send_json( $error_message );
		}
		global $product, $woocommerce, $post;
		$product_factory = new WC_Product_Factory();
		$post    = get_post( $product_id );
		$product = $product_factory->get_product( $product_id );
		setup_postdata( $post );

		ob_start();
		cherry_wc_get_template_part( 'quick-view-content' );
		$result = ob_get_clean();
		wp_reset_postdata();
		wp_send_json( array( 'content' => $result ) );
	}
}
new cherry_wc_quick_view();

Надо что-то сотворить с "class cherry_wc_quick_view" ?

Answer 1

[WP Panda]

1. Вы ее не оптимизируете, тема на основе CherryFramework по сути своей не разгоняется до приемлемых результатов.
2. Плюс у вас сильно старая версия сейчас он выглядит так
https://github.com/CherryFramework/cherry-woocomme... вашу версию убили еще в 15 году

3. На вашем месте, я бы отключил инклуд этого файла если в теме он подключен хуком или условием, если нет то

на init или даже на wp_head c высоким приоритетом вешаете функцию в которой отключаете все, что в классе подключено

таким образом

remove_action( 'woocommerce_before_shop_loop_item_title', array( 'cherry_wc_quick_view', 'append_open_wrap' ), 0 );

Comments

[Сергей Шилов]

Странно, я купил её неделю назад на template monster.

Оптимизировать её я планирую путем отключения кучи ненужных скриптов и конечной сборки файла в grunt. Хотя бы на клиентской части я могу сделать всё возможное.

В основном, спасибо, решение помогло, осталось понять как его внедрить

[Сергей Шилов]

Ну вот я ремовнул класс, ничего не изменилось

[WP Panda]

Сергей Шилов: remove_actuion работает так, как я указал

на init или даже на wp_head c высоким приоритетом вешаете функцию в которой отключаете все, что в классе подключено

таким образом

remove_action( 'woocommerce_before_shop_loop_item_title', array( 'cherry_wc_quick_view', 'append_open_wrap' ), 0 );

Как это сделали вы я не знаю, но раз не работает значит сделали не так.

[Сергей Шилов]

WP Panda: ну я её прям вот так и вставил. в functions.php

[WP Panda]

Сергей Шилов: а теперь внимательно прочитайте как я уже два раза написал как надо делать

[Сергей Шилов]

WP Panda: верно ли я делаю? В пхп не силен

remove_action( 'wp_head', array( 'cherry_wc_quick_view', 'append_open_wrap' ), 30 );

[WP Panda]

Сергей Шилов:

function toster_397749(){
   remove_action( 'wp_enqueue_scripts', array('cherry_wc_quick_view', 'assets' ), 99 );
   remove_action( 'woocommerce_before_shop_loop_item_title', array( 'cherry_wc_quick_view', 'append_open_wrap' ), 0 );
   remove_action( 'woocommerce_before_shop_loop_item_title', array( 'cherry_wc_quick_view', 'append_close_wrap' ), 100 );
    remove_action( 'woocommerce_before_shop_loop_item_title', array( 'cherry_wc_quick_view', 'append_button' ), 99 );
}

add_action( 'init','toster_397749', 500);<code>