Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SEO теги на каждую раздачу. #9

Closed
wants to merge 1 commit into from

Conversation

VictoRD11
Copy link
Contributor

Тест проводил вроде бы багов не было.

@yunasc
Copy link
Owner

yunasc commented Feb 25, 2014

Склеил.

@yunasc yunasc closed this Feb 25, 2014
@VictoRD11
Copy link
Contributor Author

Осталось только теги, да?
Сам или мне попробивать?

@yunasc
Copy link
Owner

yunasc commented Feb 25, 2014

Надо подумать.

Есть наработки, но они от хдклуба.
Не хочу один в один оттуда брать.

@VictoRD11
Copy link
Contributor Author

Та их легко реализировать, вроде бы.
Ну хз.

@yunasc
Copy link
Owner

yunasc commented Feb 26, 2014

Странно...

А поиск по ним? Не хочу кривой вариант с like %%

@VictoRD11
Copy link
Contributor Author

хмм.. а как тогда? MATCH AGAINST ?

@yunasc
Copy link
Owner

yunasc commented Feb 26, 2014

Тоже коряво, надо спроектировать это дело так, чтобы можно было сделать Where t.visible = 'yes' and (xx.tag = 'string' or xx.tag = 'string2')

Для этого надо таблица-индекс как справочник. Формата где то

Tid Tag
Ну и потом outer join на нее.

Надо еще думать, у этого подхода тоже свои минусы - надо писать обвязку которая будет индекс держать в порядке + если торрентов будет много, и тегов тоже - еще одна проблема в размере такого индекса.
Далее проблема усложнения запроса в browse.php т.к из-за (... OR ...) и того что в индексе два тега, которые ссылаются на t.id - будет две строки, значит надо делать group by t.id, а тут уже проблемы производительности... Не хотелось-бы такого монстра пихать, а MATCH AGAINST -> это только myISAM, что для разросшегося трекера тоже не прикольно.

@yunasc
Copy link
Owner

yunasc commented Feb 27, 2014

#10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants