-
Notifications
You must be signed in to change notification settings - Fork 17
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
@KarelWintersky , @olezhikz #239
Comments
Надо было написать мне в ЛС на офсайте. Эта простенькая либа, точнее это "простенькое изменение" потребует либо написания враппера между ней и нынешним синтаксисом обращения к БД в ливстрит, ЛИБО изменения обращения к БД во всём коде и всех плагинах (они и так то несовместимы, а станут еще более). Куда ни кинь - всюду клин. Но я не возражаю, попробуйте ;) |
@KarelWintersky "враппер" уже есть, сейчас он работает с либой DbSimple. Все обращения к бд из кода движка и плагинов идут через эту прослойку - поэтому я и написал что изменение простенькое относительно =) мне бы только время найти этим заняться. Ну или может у @olezhikz оно найдется. |
Я, если честно, не вижу смысла переходить с без каких-то более глобальных изменений. |
@KarelWintersky вот тут ты вроде как ругал эту либу =) |
Да, ругал. Мне (лично мне) она не нравится. |
есть вот такая простенькая либа https://github.com/Vasiliy-Makogon/Database
попробую для начала ее прикрутить к своему форку ls1 (под php7)
если встанет хорошо, осмелюсь предложить ее для официальной ветки
Originally posted by @Xmk in #237 (comment)
The text was updated successfully, but these errors were encountered: