search

livestreet / livestreet-framework

Фреймворк LiveStreet
23 stars 17 forks source link

@KarelWintersky , @olezhikz #239

Open oleg-demidov opened 6 years ago

oleg-demidov commented 6 years ago

есть вот такая простенькая либа https://github.com/Vasiliy-Makogon/Database попробую для начала ее прикрутить к своему форку ls1 (под php7) если встанет хорошо, осмелюсь предложить ее для официальной ветки

Originally posted by @Xmk in https://github.com/livestreet/livestreet-framework/issues/237#issuecomment-433732854

KarelWintersky commented 5 years ago

Надо было написать мне в ЛС на офсайте.

Эта простенькая либа, точнее это "простенькое изменение" потребует либо написания враппера между ней и нынешним синтаксисом обращения к БД в ливстрит,

ЛИБО

изменения обращения к БД во всём коде и всех плагинах (они и так то несовместимы, а станут еще более).

Куда ни кинь - всюду клин.

Но я не возражаю, попробуйте ;)

Xmk commented 5 years ago

@KarelWintersky "враппер" уже есть, сейчас он работает с либой DbSimple. Все обращения к бд из кода движка и плагинов идут через эту прослойку - поэтому я и написал что изменение простенькое относительно =) мне бы только время найти этим заняться. Ну или может у @olezhikz оно найдется.

KarelWintersky commented 5 years ago

Я, если честно, не вижу смысла переходить с https://github.com/gowebpro/livestreet1_php7/blob/1.0.4-php7/engine/lib/external/DbSimple/Mysqli.php

без каких-то более глобальных изменений.

Xmk commented 5 years ago

@KarelWintersky вот тут ты вроде как ругал эту либу =)

KarelWintersky commented 5 years ago

Да, ругал. Мне (лично мне) она не нравится. Но если вам, как мейнтейнерам LS нравится - попробуйте использовать ,почему нет? =)