Откуда сортировка в $wp_query или как её отключить?

Question

[Крил]

Пытаюсь вывести "свои" товары но они почему-то сортируются неправильно

while($aa = mysql_fetch_array($a)){				
			array_push($p_id, $aa['post_id']);				
		}
		if(count($p_id) == 0){
			***
		} else {
			$args['post__in'] = $p_id;
		}
		$args['post_type'] = 'product';
	}	
// в $wp_query->query сейчас находится только Array ( [post_type] => product ) 
	query_posts(array_merge($args,$wp_query->query));

в массиве $p_id

Array
(
    [0] => 30788
    [1] => 30787
    [2] => 30786
    [3] => 36196
    [4] => 33569
    [5] => 33568
    [6] => 22032
    [7] => 22745
    [8] => 29516
    [9] => 26660
    [10] => 22033
    [11] => 33396
    [12] => 33397
    [13] => 35537
    [14] => 34520
    [15] => 34521
    [16] => 35805
    [17] => 35263
    [18] => 35536
    [19] => 34519
    [20] => 28829
    [21] => 28827
    [22] => 35599
    [23] => 28826
    [24] => 35595
    [25] => 35598
    [26] => 35535
    [27] => 34522
    [28] => 35400
    [29] => 34523
    [30] => 35398
    [31] => 35259
    [32] => 35260
    [33] => 19384
    [34] => 35262
    [35] => 36194
    [36] => 29942
    [37] => 35261
    [38] => 33316
    [39] => 30567
    [40] => 28887
    [41] => 29501
    [42] => 23869
    [43] => 28880
    [44] => 23205
)

Но выводятся:
36194
36196
30788
33316
28880
34519
...

$args['orderby'] = 'none'; не помогает а наоборот сортирует по порядку

перезапись

$wp_query->query_vars->orderby = "";
$wp_query->query_vars->order = "";

ничего не даёт
даже тупо query_posts($args); ничего не меняет
проверял может чего не подхватывает но $args['nopaging'] = 1; работает прекрасно

Answer 1

[Крил]

Это гениально! $args['orderby'] = 'post__in';

Answer 2

[MetaDone]

$args['post__in'] = implode(",",$p_id);
а еще в $args нужно вписать
и вообще все лучше сделать через WP_Query, а не через query_posts
И не очень понятно зачем дергать mysql напрямую, это ж аццкий костыль, для выборки таких товаров используйте wpdb

Comments

[Крил]

это для полнотекстового поиска по произвольным полям и заголовку - массиве лежат самые релевантные отсортированные по релевантности

при implode ругается сам woocomerce

WordPress database error: [You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') AND wp_posts.post_type = 'product' AND (wp_posts.post_status = 'publish' OR wp_posts' at line 1]
SELECT SQL_CALC_FOUND_ROWS wp_posts.ID FROM wp_posts WHERE 1=1 AND wp_posts.ID IN () AND wp_posts.post_type = 'product' AND (wp_posts.post_status = 'publish' OR wp_posts.post_status = 'wc-send-to-client' OR wp_posts.post_status = 'wc-wait-delivery' OR wp_posts.post_status = 'private') ORDER BY wp_posts.post_date DESC LIMIT 0, 15

[Крил]

и кстати вот с wp-kama.ru/function/query_posts#vazhno-ne-ispol-zu... про implode
Не используете строку с ID разделенными запятыми, так работать не будет. Нужно передавать готовый массив:

// не правильно
$exclude_ids = '1,2,3';
query_posts( array( 'post__not_in' => array( $exclude_ids ) ) );

// Правильно
$exclude_ids = array( 1, 2, 3 );
query_posts( array( 'post__not_in' => $exclude_ids ) );

Answer 3

[Mr Crabbz]

Попробуйте сконструировать ваш wp_query через этот сервис: https://generatewp.com/wp_query/ - там в том числе можно выставить orderby = "none".
Возможно вы где-то ошиблись в мелочи или знаке и просто не видите очевидную ошибку. По приведённому коду сказать трудно. Непонятна сама цель такого вывода. Ведь в каком-то же порядке должны посты вывестись всё таки, верно?

Comments

[Крил]

создано на основе dontforget.pro/wordpress/filtr-postov-po-proizvoln...

function go_filter() {
	$args = array(); 
	$args['meta_query'] = array('relation' => 'AND');
	global $wp_query;
/* тут другие параметры которые есть тут http://dontforget.pro/wordpress/filtr-postov-po-proizvolnyim-polyam-v-wordpress/ и они работают */

	if ($_GET['ss'] != '') {
		
               $result = $_GET['ss'];		

		$sdd_db_host='localhost';
		$sdd_db_name='***';
		$sdd_db_user='***';
		$sdd_db_pass='***';

		@mysql_connect($sdd_db_host,$sdd_db_user,$sdd_db_pass);
		@mysql_select_db($sdd_db_name);
		mysql_query("SET NAMES utf8");

		$p_id = array();

		$a = mysql_query("SELECT DISTINCT _postmeta.post_id, _postmeta.meta_key, _postmeta.meta_value FROM _postmeta WHERE MATCH (meta_value) AGAINST ('" . $result . "') and meta_key ='For_search'"); 

		while($aa = mysql_fetch_array($a)){				
			array_push($p_id, $aa['post_id']);// тут сложили все id постов по релевантности		
		}

		if(count($p_id) == 0){
			wp_redirect('***');
		} else {
			$args['post__in'] = $p_id;// тут пытаемся добавить	
		}

		$args['post_type'] = 'product';
		$args['nopaging'] = 0;
		//$args['orderby'] = 'none';
	}	
	query_posts(array_merge($args,$wp_query->query)); 
}

[Игорь Воротнёв]

Крил: см. мой ответ. Этот код - одна сплошная ДЫРА в плане безопасности.

Answer 4

[Игорь Воротнёв]

Крил: боже упаси такой код использовать, у вас там sql-injection! Я уж молчу про опасный output. Это раз. Два - прямой коннект в БД и низкоуровневый sql-запрос внутри среды WP - полный кретинизм. Три - читайте про хуки. У WooCommerce (и у WP) масса полезных для таких задач событий и фильтров.

Сделайте шаг назад, избавьтесь от этого дырявого говнокода. Поставьте четко задачу. Тогда можно будет сделать все правильно.

ЗЫ: По приведенной вами ссылке код нормальный, а у вас получился кошмар.

Comments

[Крил]

про sql знаю, это позже поправится - мне хоть вручную массив задать нужно чтоб он сортировался правильно, запросы сейчас так написаны не для критики и ваш ответ не по теме

[Игорь Воротнёв]

Крил: ну так поставьте массив вручную для отладки, это как раз правильный подход. А костыли с дырками - неправильный и опасный. Вероятность того, что у вас не заложен нормальный бюджет и сроки на рефакторинг слишком высока, и шансы того, что кусочек подобного кода попадет в продакшн - тоже весьма высоки.

Мой ответ - уточняющий. Разбираться в текущем коде - не есть хорошо. Вернитесь в отправную точку, сформулируйте четко задачу, тогда можно будет воссоздать аналогичные условия у себя и помочь вам с отладкой. Сейчас же ваш "временный" код может больше ошибок вносить, чем вы пытаетесь отловить.

[Крил]

Игорь Воротнёв: это мой тестовый проект
<?php
function go_filter() {
$args = array();
$args['meta_query'] = array('relation' => 'AND');
global $wp_query;

function ifget($key, $ge_t){
if ($ge_t != '') {
return array(
'key' => $key,
'value' => $ge_t
);
}
}

$args['meta_query'][] = ifget("Скорость", $_GET["speed"]);
$args['meta_query'][] = ifget("Время", $_GET["time"]);

if ($_GET['price_ot'] != '' || $_GET['price_do'] != '') {
if ($_GET['price_ot'] == '') $_GET['price_ot'] = 0;
if ($_GET['price_do'] == '') $_GET['price_do'] = 9999999;
$args['meta_query'][] = array(
'key' => '_regular_price',
'value' => array( (int)$_GET['price_ot'], (int)$_GET['price_do'] ),
'type' => 'numeric',
'compare' => 'BETWEEN',
);
}

if ($_GET['sort_abc'] != '') {
$args['orderby'] = 'title';
$args['order'] = $_GET['sort_abc'];
}

if ($_GET['sort_prise'] != '') {
$args['order'] = $_GET['sort_prise'];
$args['orderby'] = 'meta_value_num';
$args['meta_key'] = '_regular_price';
}

if ($_GET['na_zakaz'] != '') {
$args['post_status'] = $_GET['na_zakaz'];
}

if ($_GET['ss'] != '') {
$args['post__in'] = array(30220, 29505, 33723, 35600);
//получаю 35600, 29505, 30220, 33723
$args['post_type'] = 'product';
$args['nopaging'] = 0;
//$args['orderby'] = 'none';
}

query_posts(array_merge($args,$wp_query->query));
}
?>

[Игорь Воротнёв]

Крил: ну задача все еще не ясна. Что с вашим куском кода происходит в средах WP и WC - непонятно. Установите для отладки хотя бы Query Monitor. На данный момент, похоже, вы не совсем разобрались как работает WP_Query, и тем более не учли, что WooCommerce использует свой класс WC_Query, который как раз и переопределяет все, что ему нужно, в родном WP_Query. У него есть свое подключение в хук parse_request для анализа запроса (ваши переменные), а также в pre_get_posts для модификации запроса непосредственно перед его выполнением. Посмотрите исходный код класса, чтобы понять их workflow. По тем данным, которые вы предоставляете, я не могу у себя воссоздать аналогичные условия, поэтому могу строить только предположения. Первые из них - либо у вас переменные не парсятся корректно, либо они не передаются корректно (возможно после всех ваших танцев с бубном они перезаписываются WooCommerce'ом на свои, в порядке приоритета).

[Крил]

Игорь Воротнёв: да, я не понимаю некоторых вещей, но стараюсь понять.
Дело в том что всё работает нормально и другие параметры срабатывают кроме $args['post__in'], похоже на то что ничего не перезаписывает wocommerce.