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_warnings*) #41

Open
AIlkiv opened this issue Jun 16, 2017 · 9 comments
Open

Жовті картки (bb_warnings*) #41

AIlkiv opened this issue Jun 16, 2017 · 9 comments

Comments

@AIlkiv
Copy link

AIlkiv commented Jun 16, 2017

В torrentPier 0.35 відсутні таблиці bb_warnings і bb_warnings_text, а на толоці вони є.
@konfuciusu Що це за таблиці та що з ними робимо?

@konfuciusu
Copy link
Contributor

Це жовті картки, виглядає на те, що такого функціоналу в torrentpier2 немає.

@konfuciusu
Copy link
Contributor

Можна було би взяти за основу https://torrentpier.me/forum/threads/36747/

@konfuciusu konfuciusu changed the title Міграція bb_warnings Жовті картки Jun 17, 2017
@yukoff yukoff added this to the v3.0.0 milestone Jul 15, 2017
@yukoff
Copy link
Contributor

yukoff commented Jul 15, 2017

@konfuciusu Я так розумію, вони залишаються?

@yukoff
Copy link
Contributor

yukoff commented Jul 15, 2017

@konfuciusu Тут так само - попередня назва давала контекст, може вже краще "Міграція жовтих карток (bb_warnings*)"?

@konfuciusu
Copy link
Contributor

Залишаються, не проти зміни назви.

@yukoff yukoff added the ready label Jul 16, 2017
@yukoff yukoff changed the title Жовті картки Жовті картки (bb_warnings & bb_warnings_text) Jul 17, 2017
@yukoff yukoff changed the title Жовті картки (bb_warnings & bb_warnings_text) Жовті картки (bb_warnings*) Jul 17, 2017
@yukoff yukoff added in progress and removed ready labels Jul 27, 2017
@yukoff
Copy link
Contributor

yukoff commented Aug 3, 2017

А хтось брався за код самих "жовтих карток"? Я глянув ту модифікацію з форуму - додав поки у себе. Загалом я такого "спагетті" не бачив вже давно (це не саме про тозширення, TP, принаймні щодо роботи з шаблонами, геть скрізь цим страдає) :( Один суцільний антишаблон... Але думаю це просто "спадщина" - у develop гілці були зрушення в сторону Twig, але чомусь їх немає в experimental

@yukoff
Copy link
Contributor

yukoff commented Aug 17, 2017

Тут питання виникло - судячи з дампу install/sql/toloka_old.sql зараз у bb_warnings_text.bbcode_uid дані у latin1 зберігаються:

CREATE TABLE `bb_warnings_text` (
...
 `bbcode_uid` varchar(10) CHARACTER SET latin1 NOT NULL DEFAULT '',
...
) ... DEFAULT CHARSET=utf8;

Хоча на інших подібних -- bb_posts_text.bbcode_uid, bb_privmsgs_text.privmsgs_bbcode_uid та bb_users.user_sig_bbcode_uid -- кодування типове для таблиці, тобто utf8 😕 Загалом має нормально конвертуватись, але краще перевірити -- @konfuciusu, чи можливо якийсь дамп bb_warnings_text зробити? В принципі, warning_text не потрібен.

@konfuciusu
Copy link
Contributor

@yukoff подивився, bb_warnings_text.bbcode_uid взагалі не використовується.

@yukoff
Copy link
Contributor

yukoff commented Aug 18, 2017

Окей, тоді з цим наче все в частині змін у БД - лишається власне код

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