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

Прикріплення файлів до приватних повідомлень (bb_attachments) #27

Open
AIlkiv opened this issue Jun 11, 2017 · 5 comments

Comments

@AIlkiv
Copy link

AIlkiv commented Jun 11, 2017

На толоці зараз є така функції. На новому torrentpier нема, треба вирішити чи реалізовуємо.

Довідка по базі толоки:
bb_attachments таблиця зв'язки прикріплених файлів з post (post_id) або ПП (privmsgs_id, user_id_2 - ці дві колонки відсутні у новій версії)

@konfuciusu
Copy link
Contributor

Мало би бути нескладно додати, принцип однаковий, як і в прикріплень до тем.

@konfuciusu konfuciusu changed the title Прікріплення файлів до приватних повідомлень Прикріплення файлів до приватних повідомлень Jun 16, 2017
@yukoff yukoff added this to the v3.0.0 milestone Jul 15, 2017
@yukoff yukoff added the ready label Jul 31, 2017
yukoff added a commit to yukoff/toloka that referenced this issue Aug 15, 2017
yukoff added a commit to yukoff/toloka that referenced this issue Aug 15, 2017
@yukoff
Copy link
Contributor

yukoff commented Aug 15, 2017

@konfuciusu,
В старій схемі таблиця не має PK, і тут трохи незрозуміло, яким він (PK) має бути у випадку з ПП (можливо навіть ще один UNIQUE знадобиться - залежить від того, як має забезпечуватись унікальність). Наприклад (PRIMARY KEY типовий з нової схеми БД):

old new
attach_id attach_id
post_id post_id
privmsgs_id +
user_id_1 user_id_1
user_id_2 +
- KEY attach_id_post_id (attach_id,post_id) PRIMARY KEY (attach_id,post_id) +
- KEY attach_id_privmsgs_id (attach_id,privmsgs_id) UNIQUE KEY attach_id_privmsgs_id (attach_id,privmsgs_id) +
KEY post_id (post_id) +
KEY privmsgs_id (privmsgs_id) +

@konfuciusu
Copy link
Contributor

Для унікальності треба
(attach_id,post_id)
і
(attach_id,privmsgs_id)

Але в такому вигляді, як у вас, це не спрацює, бо post_id може бути 0 (тоді privmsgs_id != 0), або privmsgs_id може бути 0 (тоді post_id != 0).

Може просто (attach_id,post_id,privmsgs_id)?

@yukoff
Copy link
Contributor

yukoff commented Aug 15, 2017

@konfuciusu,
Я теж саме про це думав ;) Хоча було б корисно на тип запитів глянути

@yukoff yukoff changed the title Прикріплення файлів до приватних повідомлень Прикріплення файлів до приватних повідомлень (bb_attachments) Aug 15, 2017
yukoff added a commit to yukoff/toloka that referenced this issue Aug 17, 2017
yukoff added a commit to yukoff/toloka that referenced this issue Aug 20, 2017
yukoff added a commit to yukoff/toloka that referenced this issue Aug 20, 2017
@yukoff
Copy link
Contributor

yukoff commented Aug 23, 2017

З цим по БД все, додав у #74, решта за функціоналом.

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

No branches or pull requests

3 participants