russian stemmer module

Аватар пользователя Heihachi88

Скачал модуль от сюда - http://www.drupal.ru/files/rustemmer.zip
Скачал апдейт для модуля от сюда - http://www.drupal.ru/files/rustemmer_002.zip

В папке модуля rustemmer заменил файл rustemmer.module из архива версии 002, включил модуль, но поиск всеравно работает некорректно, т.е. в определенной статье имеется слово масимов, ввожу в поиске масим - нет результатов, может быть я что-то не правильно делаю? Помогите, пожалуйста.

Версия Drupal:
0 Thanks

Комментарии

Аватар пользователя Vladimir_Z
Vladimir_Z 7 лет назад

Может нужно переиндексировать сайт после включения модуля?

Аватар пользователя Vladimir_Z
Vladimir_Z 7 лет назад
Azerot написал:
Возьмите отсюда:
http://drupalist.org.ru/modules/rustemmer
стоит и работает

Вот там как раз и написано
«Единственное что необходимо после включения модуля на странице модулей admin/modules, это выполнение переиндексации сайта. Вы должны нажать на кнопку "Переиндексировать сайт" на странице admin/settings/search и дождаться пока содержимое сайта будет полностью...»

Аватар пользователя graker
graker 7 лет назад

Там к сожалению в INSTALL.txt не написано, что надо прилагающийся в архиве патч применить к модулю search. А без этого, похоже, не будет работать.

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад

Может таки кто-нибудь выложит сией модуль на drupal.org? Думаю, что там его быстрее бы допилили.

Аватар пользователя graker
graker 7 лет назад
andypost@drupal.org написал:
Может таки кто-нибудь выложит сией модуль на drupal.org? Думаю, что там его быстрее бы допилили.

А разве дают выкладывать на D.org модули, для работы которых нужно патчить ядро?

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад
graker написал:
А разве дают выкладывать на D.org модули, для работы которых нужно патчить ядро?

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

Аватар пользователя graker
graker 7 лет назад
andypost@drupal.org написал:
Безусловно, и таких примеров море. Здесь вопрос лиш в лицензировании библиотеки, которая делает стемминг

А. Это хорошо.
Но насчет лицензирования неясно, есть ли проблемы. Там алгоритм приведен в открытом источнике одним человеком (без лицензии), реализация на php взята из открытого форума, из поста другого человека (тоже без лицензии) и адаптирована под Друпал третьим человеком. Весь код живет внутри файла модуля, без внешних библиотек. Копия GPL 2.0 лежит в архиве вместе с модулем.

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад
graker написал:
Копия GPL 2.0 лежит в архиве вместе с модулем.

Ну коль так, все отлично! Можно смело заводить проект!
У тебя есть CVS доступ?

Аватар пользователя graker
graker 7 лет назад
andypost@drupal.org написал:
У тебя есть CVS доступ?

Нету.

2 Azerot: как так? У меня без патча никакого стеммера не было. Пропатчил - сразу заработало.

Аватар пользователя Heihachi88
Heihachi88 7 лет назад
Vladimir_Z написал:
Может нужно переиндексировать сайт после включения модуля?

Переиндексация и запуск крона делал - 0 результатов

graker написал:
Там к сожалению в INSTALL.txt не написано, что надо прилагающийся в архиве патч применить к модулю search. А без этого, похоже, не будет работать.

Подскажите пожалуйста, как эти патчи применять вообще? :)

Azerot написал:
Возьмите отсюда:
http://drupalist.org.ru/modules/rustemmer
стоит и работает

Какие действия вы проделывали? Какая версия друпала? - У меня не работает, ввожу масим - пишет нет результатов, хотя в статье есть слово масимов. Переиндексацию, запуск крона, очистку кеша делал.

Аватар пользователя graker
graker 7 лет назад
Heihachi88 написал:
Подскажите пожалуйста, как эти патчи применять вообще? :)

Копируешь патч в директорию модуля search, а дальше из консоли:
patch search.module stemmer_search.module.patch.
И всё :)

А вообще - вот тут написано: http://drupal.org/patch/apply

Аватар пользователя Azerot
Azerot 7 лет назад
Цитата:
Какие действия вы проделывали? Какая версия друпала

Никаких не проделывал, кроме стандартных по установке модуля.
Версия начиналась с 6.14, сейчас последняя 6.17

Цитата:
Может таки кто-нибудь выложит сией модуль на drupal.org? Думаю, что там его быстрее бы допилили.

На drupal.org делают только то, что считают нужным делать. Частая практика - ноль ответов на вопросы, ноль реакции на баги. Выложить простой смертный тоже ничего не может - доступ не дают и вообще чихать они хотели. Это моё личное мнение, основанное на моих попытках работать с drupal.org

Аватар пользователя graker
graker 7 лет назад
Azerot написал:
Никаких не проделывал, кроме стандартных по установке модуля.

А патч применять не пришлось?

Цитата:
Выложить простой смертный тоже ничего не может - доступ не дают и вообще чихать они хотели. Это моё личное мнение, основанное на моих попытках работать с drupal.org

Да в принципе дают доступ, но процесс его получения долог, нуден и не очень приятен :) В результате десять раз подумаешь, а надо ли тебе это вообще.

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад
Azerot написал:
На drupal.org делают только то, что считают нужным делать.

Есть правила и их нужно соблюдать!

Azerot написал:
ноль реакции на баги

Зависит от багов и преложеного решения. Немаловажно еще и то, кто и как сообщает о баге

Azerot написал:
доступ не дают и вообще чихать они хотели

Доступ дают и довольно оперативно, просто следят за качеством и стилем кода тех, кто получает доступ.

Аватар пользователя Azerot
Azerot 7 лет назад
Цитата:
А патч применять не пришлось?

Нет!

Цитата:
Да в принципе дают доступ, но процесс его получения долог, нуден и не очень приятен :)

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

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

Аватар пользователя Azerot
Azerot 7 лет назад
Цитата:
Зависит от багов и преложеного решения. Немаловажно еще и то, кто и как сообщает о баге

Вот я и говорю, зависит от того хотят они им заниматься или нет.
Какая разница кто сообщает о баге, если он имеет место? Что, так сложно проверить, если предоставлены все необходимые данные? Если не прав тот, кто предоставил, напишите, отклоните.

Аватар пользователя Azerot
Azerot 7 лет назад
Цитата:
2 Azerot: как так? У меня без патча никакого стеммера не было. Пропатчил - сразу заработало.

Я не помню, чтобы что-то патчил. Патч, насколько я помню был предназначен для d5, хотя могу ошибаться, конечно.

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад

Баги бывают разные, в стабильную версию попадают практически только изменени связанные с безопасностью и стабильностью работы. Существует масса багов, которые нельзя исправить не внеся изменения в текущий АПИ - они переходят в следущую версию.

Что касается багов в сторонних модулях (контрибе), тут все уже зависит от автора модуля. Вот типичный пример бага, который известен, но исправлен не будет http://drupal.org/node/194197

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад

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

Аватар пользователя graker
graker 7 лет назад
andypost@drupal.org написал:
Насколько я помню, патч нужен для правильной подсветки слова в результатах поиска. Ибо подсвечиваться должно то слово, которое прошло стемминг, а не то, которое выводится на странице

А, да, похоже что так и есть.
Тогда значит должно работать и без патча, я просто не понял сразу.

Аватар пользователя Heihachi88
Heihachi88 7 лет назад

Ребята, помогите пропатчить search.module патчем stemmer_search.module.patch через GnuWin32! Сделал все как написано тут http://www.indiawebsearch.com/content/how-to-apply-a-patch-to-a-drupal-m... , но при вводе команды patch < stemmer_search.module.patch он открывает новое окно с командной строкой и ничего не происходит!

Аватар пользователя Azerot
Azerot 7 лет назад
Цитата:
Насколько я помню, патч нужен для правильной подсветки слова в результатах поиска. Ибо подсвечиваться должно то слово, которое прошло стемминг, а не то, которое выводится на странице

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

Цитата:
Баги бывают разные, в стабильную версию попадают практически только изменени связанные с безопасностью и стабильностью работы. Существует масса багов, которые нельзя исправить не внеся изменения в текущий АПИ - они переходят в следущую версию.

Хорошо. Давайте предметно. Как вам такой баг?
http://drupal.org/node/769698
Запощен мной более 3-х месяцев назад. Не только не исправлен, и не принят в работу, но даже ни ответа ни привета.
Есть всё - описание, патч, казалось бы чего ещё надо?

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад

Ну так что, кто заведет проект на d.o?

Azerot написал:
Запощен мной более 3-х месяцев назад. Не только не исправлен, и не принят в работу, но даже ни ответа ни привета. Есть всё - описание, патч, казалось бы чего ещё надо?

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

Основная ошибка при публикации бага - он был опубликован в 6.16 (а нужно было как минимум в 6-dev) но и там его маловероятно кто-нить подхватил бы, так как все проблемы сначала решаются в HEAD (сейчас это 7) и только после решения в текущей версии они портируются в более младшие версии.

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

Аватар пользователя graker
graker 7 лет назад
andypost@drupal.org написал:
Ну так что, кто заведет проект на d.o?

Кому охота вставать в очередь на доступ к CVS и отвечать на каверзные вопросы kiamlaluno с чужим проектом, в коде которого еще предстоит разбираться? :)

Аватар пользователя graker
graker 7 лет назад
andypost@drupal.org написал:
Завел проект http://drupal.org/project/rustemmer

Для начала думаю заслать текущую версию, после того как причешем, можно сделать релиз

Круто.
Кажется, у меня есть уже issue по приложенному патчу. Попозже ночью запощу, или завтра.

Аватар пользователя Azerot
Azerot 7 лет назад
Цитата:
Основная ошибка при публикации бага - он был опубликован в 6.16 (а нужно было как минимум в 6-dev) но и там его маловероятно кто-нить подхватил бы, так как все проблемы сначала решаются в HEAD (сейчас это 7) и только после решения в текущей версии они портируются в более младшие версии.

А я считаю, что основная ошибка не в том, что я опубликовал в 6.16, а в том что система отслеживания ошибок неграмотно спроектирована. Такое мнение вытекает из того, что на баг даже никто не удосужился ответить. Если никого не интересуют баги в 6.16 не надо давать постить баги в этот раздел. Оставили бы две ветки тогда: 6.x и 7.x. Почему-то в других проектах, такого нет.

И ещё прочитал ваш коммент на баг. Проблема в том, что баг не в коде модуля locale, как в дублирующем баге, а код в include файле ядра locale.inc. Видимо поэтому я его и не нашёл, когда искал

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад

Поправил и почистил большую часть кода, сделал релиз (кандитат 2) http://drupal.org/project/rustemmer

Похоже есть возможность избежать патча, но пока не уверен

ЗЫЖ Azerot, просто опубликовать патч или баг - не эффективно, нужно об этом оповестить! Из всех десятков тысяч пользователей эта проблема досаждает единицам, так что им стоит координировать свои усилия, чтобы на них обратили внимание. Прекращаем пустую полемику... удачи.

Аватар пользователя Heihachi88
Heihachi88 7 лет назад
andypost@drupal.org написал:
Поправил и почистил большую часть кода, сделал релиз (кандитат 2) http://drupal.org/project/rustemmer


Как успехи с модулем? Установил его, всеравно придется патчить ядро?

Аватар пользователя andypost@drupal.org
andypost@drupal.org 7 лет назад

Ядро патчить нужно в любом случае, но теперь приоритетная задача, портировать сией модуль на Drupal 7

http://drupal.org/project/porterstemmer уже вышел релиз для D7

Вот issue по портированию