Jump to content

Featured Replies

9 часов назад, Zero108 сказал:

Кстати, можно было бы исключить из логов вот такие запросы

Я не могу получить такую запись. Пустой запрос быстрым поиском не отправляется, а если нажимать кнопку с пустым запросом или жать Enter, то запись у меня не добавляется.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195433
Share on other sites
  • Replies 140
  • Created
  • Последний ответ
Expand topic overview

Лучшие авторы в теме

Лучшие авторы в теме

Most Popular Posts

edmsl
edmsl

Хороший плагин, но на 4.7.9 не работает. Поэтому решил его адаптировать. Это моя первая работа, так что ошибки очень вероятны. Перевод мой. На версиях ниже 4.7.9 не тестировал. (BIM) Quick S

Zero108
Zero108

edmsl На 4.7.11.1 работает:  Осталось только в CSS подправить стиль для темной темы. Попробую для Nexxe темы сделать.

edmsl
edmsl

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

Posted Images

Expand topic overview

edmsl У меня тоже не получается получить такую запись намеренно.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195436
Share on other sites

Zero108 Может боты балуются? У меня гости исключены из логирования.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195437
Share on other sites

edmsl возможно, боты, так как нечасто и давно не было.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195438
Share on other sites

Zero108 В пользу этой теории говорит еще то, что текст запроса - это текст из плейсхолдера (заглушки) поля поиска.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195439
Share on other sites
9 часов назад, edmsl сказал:

Sorry, but no. I did't work with cripto coins and I can't even test this plugin.

Ok, no problem

9 часов назад, edmsl сказал:

Yes, but he did not answer

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195440
Share on other sites

Обнаружил еще одну проблему. Если, допустим, у файла указан скриншот по url, то в поиске он те отображается. Буду разбираться.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195475
Share on other sites

edmsl Уже пора переименовать плагин в Smart Search 1.1.0 и выложить отдельным файлом. От изначального кода там осталось 10%.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195478
Share on other sites
2 минуты назад, Zero108 сказал:

От изначального кода там осталось 10%

Нет, как раз-таки нового кода там процентов 10. Но то, что его лучше бы сделать отдельным файлом, я согласен. Все же плагин и сейчас развивается автором, и наличие моей ответвленной версии может сбить с толку.

Буду выпускать его под именем Smart Search, название мне нравится, хотя и не совсем подходит.

И да, проблему со скриншотом выше решил. Вечером скину новый файл сразу с исправлением.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195479
Share on other sites

edmsl Добавишь новую раздачу файла отдельно? Там удобнее будет не смешитвать. Плагин вьетнамского автора, если честно, кажется не таким уж хорошим после твоих нововведений и информации, что я узнал насчет защиты через php от флдуда. 

40 минут назад, edmsl сказал:

название мне нравится

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

Edited by Zero108

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195480
Share on other sites
10 минут назад, Zero108 сказал:

Добавишь новую раздачу файла отдельно?

Да, добавлю.

10 минут назад, Zero108 сказал:

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

В общем, есть куда развивать плагин если что.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195481
Share on other sites

edmsl могу прислать папку умного модуля Голубева. Может что из php оттуда взять можно? Хотя, что создано в Битриксе, умирает в нём же, так как система специфическая.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195482
Share on other sites

Zero108 Взглянуть-то можно, а там видно будет, сгодится он или нет.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195483
Share on other sites

edmsl Напиши мне в телегу. 

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195484
Share on other sites

@Exception Нужна твоя консультация.

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

В коде плагина есть такие строки:

$term = preg_replace('/^(\'(.*)\'|"(.*)")$/', '$2$3', $term);

$temp = preg_split('//u', $term, 0, PREG_SPLIT_NO_EMPTY);

foreach ($temp as $a) {
	$cnt = 0;
	if (preg_match('/[^\w\s]+/u', $a)) {
		$cnt++;
		if ($cnt >= 2) {
			\IPS\Output::i()->json(array('type' => 'OK', 'html' => "", 'term' => $term));
		}
	} else {
		$cnt = 0;
	}
}

где $term = preg_replace('/^(\'(.*)\'|"(.*)")$/', '$2$3', $term); убирает кавычки в начале и конце строки. Из-за этого точный поиск не срабатывает.

Собственно, мне не понятно, для чего это делается. Какая-то защита? Вроде из инпута в любом случае передается строка.

И проверка в цикле мне тоже не понятна. Разбивается вся строка на символы, а потом проверка на длину символа, где считаются все НЕ буквы и НЕ пробелы. Видимо, это какие-то специфические проверки безопасности, о которых я не знаю, но хотелось бы понять это все.

Edited by edmsl

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195497
Share on other sites

Ответа я так и не нашел, поэтому решил делать, полагаясь на логику. В итоге написал так:

// Удалил эту строку отсюда и сделал удаление кавычек только для $temp для проверки в цикле (1).
//$term = preg_replace('/^(\'(.*)\'|"(.*)")$/', '$2$3', $term);

// Здесь проверка, что кавычки есть. Нужно для того, чтобы не не добавлялась * для поиска по неполному слову.
$isStrictSearch = preg_match('/^(\'(.*)\'|"(.*)")$/', $term);

$temp = preg_split(
	'//u',
	// Перенес сюда (1)
	preg_replace('/^(\'(.*)\'|"(.*)")$/', '$2$3', $term),
	0,
	PREG_SPLIT_NO_EMPTY
);

foreach ($temp as $a) {
	$cnt = 0;
	if (preg_match('/[^\w\s]+/u', $a)) {
		$cnt++;
		if ($cnt >= 2) {
			\IPS\Output::i()->json(array('type' => 'OK', 'html' => "", 'term' => $term));
		}
	} else {
		$cnt = 0;
	}
}

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

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195523
Share on other sites
В 11.08.2023 в 10:53, edmsl сказал:

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

Можно и в одинарные ковычки обернуть. Кто как привый искать. Я оборачиваю на гугле в "поисковая фраза".

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195530
Share on other sites

Zero108 Одинарные не работают на движке. Попробуй в стандартном поиске с одинарными и с двойными обернуть по очереди.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195531
Share on other sites

edmsl пока не заметил разницы между поисковая фраза, "поисковая фраза" и 'поисковая фраза'. Может у тебя получится? https://a108.net/search

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195532
Share on other sites

Zero108 Поиск Эластик? Как-то странно он себя ведет все же. Но даже так разница очевидна.

Спойлер

firefox_2023-08-12_14-36-19.thumb.jpg.025367e7c687a22f3e06f17e259da1fb.jpg

firefox_2023-08-12_14-36-56.thumb.jpg.7af1c21fdc09e1bc7ac09abd67b6527c.jpg

firefox_2023-08-12_14-37-54.thumb.jpg.dac761eec6f30903038486f2de30ff6d.jpg

В последнем варианте, с двойными кавычками, по идее, должен найти только те строки, где слова строго в указанном порядке. Но 3-я выдача не соответствует ему. Внутри в статье тоже нет такой фразы.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195535
Share on other sites

edmsl Да, эластик. В настройках стоит: "Both apples and bananas".

Такое ощущение, что вьетнамский автор плагина понял, что любой вариант поиска (эластик или mysql) плевать хотел на ковычки, и забил на это.

Edited by Zero108

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195538
Share on other sites

Выложил новую версию под новым именем.

Ждет одобрения.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195550
Share on other sites

edmsl а что там поменялось с последнего раза?

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195551
Share on other sites

Zero108 Точный поиск с кавычками, пара новых опций поиска для виджета.

Новое название, следовательно изменены все классы CSS и id.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195554
Share on other sites

edmsl скинь в телегу, побуду тестером.

Link to comment
https://ipbmafia.ru/topic/23648-bim-quick-search/?&do=findComment&comment=195556
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.


Guest
Ответить в этой теме...

Последние посетители 0

  • No registered users viewing this page.