Woocommerce attribute issue
-
Привет. Данный плагин супер, но есть один минус в плане работы с WooCommerce и любыми языками кроме англ. Дело в том, что он автоматом таки переводит ВНУТРЕННИЕ атрибуты в товаре, ровно как и глобальные. Хотя не должен этого делать, ибо тогда ломается стандартный вывод вариантов товара в корзине. Т.е допустим я добавил внутренний(!) атрибут “Цвет”. По итогу он будет выводить его как tsvet. Хотя из коробки WooCommerce выводит так, как написали, ибо это не сущность.
Почему is_wc_attribute() ВСЕГДА возвращает false для внутренних атрибутов:is_wc_attribute_taxonomy() → проверяет глобальные атрибуты WooCommerce
Внутренний атрибут “цвет” → НЕ глобальный → falseis_wc_product_not_converted_attribute() → сломанная логика:
php
protected function is_wc_product_not_converted_attribute( string $title ): bool {
global $product; // ← ПРОБЛЕМА 1: часто NULLif ( ! is_a( $product, 'WC_Product' ) ) { return false; // ← Часто срабатывает здесь } $attributes = (array) get_post_meta( $product->get_id(), '_product_attributes', true ); // ... проверяет уже сохранённые атрибуты}
Почему global $product часто NULL:
При сохранении товара из админки WooCommerce не всегда устанавливает эту глобальную переменную. При AJAX-обработке атрибутов она точно не установлена
Метод вызывается из контекста sanitize_title(), а не из страницы товараИтог:
is_wc_attribute_taxonomy(“цвет”) → false
is_wc_product_not_converted_attribute(“цвет”) → false (потому что global $product = NULL)
is_wc_attribute(“цвет”) → false
transliterate(“цвет”) → “tsvet”Ошибка в коде: Метод is_wc_product_not_converted_attribute() пытается определить атрибут по уже сохранённым данным, но не учитывает, что:
НОВЫЙ атрибут ещё не сохранён
Глобальная переменная $product не установлена в нужном контексте
Надо проверять не метаполя, а $_POST данные или контекст сохранения товара
-
Или вовсе так:
public function sanitize_title( $title, $raw_title = '', $context = '' ) {
global $wpdb;
// Не транслитерировать при создании глобальных WC атрибутов
if ( function_exists( 'wc_sanitize_taxonomy_name' ) ) {
$trace = debug_backtrace( DEBUG_BACKTRACE_IGNORE_ARGS );
foreach ( $trace as $call ) {
if (
isset( $call['function'] ) &&
$call['function'] === 'wc_sanitize_taxonomy_name'
) {
return $title;
}
}
}
if (
! $title ||
'query' === $context ||
! $this->transliterate_on_pre_term_slug_filter( (string) $title )
) {
return $title;
}
$title = urldecode( (string) $title );
$pre = apply_filters( 'ctl_pre_sanitize_title', false, $title );
if ( false !== $pre ) {
return $pre;
}
if ( $this->is_term ) {
$this->is_term = false;
if ( $this->is_frontend && class_exists( SitePress::class ) ) {
return $title;
}
$sql = $wpdb->prepare(
"SELECT slug FROM $wpdb->terms t LEFT JOIN $wpdb->term_taxonomy tt
ON t.term_id = tt.term_id
WHERE t.slug = %s",
rawurlencode( $title )
);
if ( $this->taxonomies ) {
$sql .= ' AND tt.taxonomy IN (' . $this->prepare_in( $this->taxonomies ) . ')';
}
$term = $wpdb->get_var( $sql );
if ( ! empty( $term ) ) {
return $term;
}
}
return $this->is_wc_attribute( $title ) ? $title : $this->transliterate( $title );
}}
нене, я сделал вообще иначе, без debug_backtrace . Твой плагин не трогал. И всё что было в прошлых коммах тоже не использовал.
Вот если интересно – https://github.com/uxtora/cyr-to-lat-wp-personal-.git
Проверил, работает. Что думаешь?
Если поможет в фиксе – буду рад.Привет. Нашёл ещё одну очень сильную проблему. Возможно она появилась после обновлений – хз. Если сохранить страницу и потом поменять слаг напрямую – он возвращает старый после сохранения. Т,е допустим ты создал страницу “Главная”, он создал ей слаг “glavnaya”. Нажал сохранить. Страница сохранилась. Потом ты решил поменять слаг вручную на “home”. Идешь редачить, AJAX улетает, вроде всё окей, но при сохранении получаешь опять старый слаг. Т.е пост нейм у него почему-то пустой. Я проверил несколько плагинов и везде такая же проблема. + тестировал, просто делал ключи+санитайз = та же проблема. Я решил лишь при прямой перезаписи в БД, но это не совсем правильно т.к обходятся стандартные хуки и тд. Но вроде как при аяксе пофигу… Не знаю. Там есть дебаг файлы, попробуй убери $wpdb->update и поменяй на ванилу, тебе вернёт старый слаг. Короче вп..
https://github.com/uxtora/cyr-to-lat-wp-personal-/blob/main/record%20to%20DB%20AJAX
You must be logged in to reply to this topic.
