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

Расширенный ?cols #44

Closed
vasa-c opened this issue Mar 20, 2016 · 11 comments
Closed

Расширенный ?cols #44

vasa-c opened this issue Mar 20, 2016 · 11 comments
Assignees
Milestone

Comments

@vasa-c
Copy link
Owner

vasa-c commented Mar 20, 2016

db
table
col
func
as

vasa-c added a commit that referenced this issue Mar 20, 2016
@vasa-c
Copy link
Owner Author

vasa-c commented Mar 20, 2016

?cols, Table & maps

@mkedo
Copy link
Contributor

mkedo commented Mar 23, 2016

Раз пошла такая пьянка, может добавишь в Table чтоб можно было схему/бд указать к таблице? :)
$table = new Table($db, array('schema', 'table'));
Ну ?t соответсвенно.

@vasa-c
Copy link
Owner Author

vasa-c commented Mar 23, 2016

@mkedo, по подробнее

@mkedo
Copy link
Contributor

mkedo commented Mar 23, 2016

В PostgreSQL часто приходится схему указывать.
Например, есть таблица users в схеме web.
Я сейчас не могу написать $t = new \go\DB\Table($db, array('web', 'users')); $t->insert(...),
чтоб получить потом INSERT INTO "web"."users" ..., т.к там
используется плейсхолдер ?t INSERT INTO ?t ..., а он не разворачивается из массива.

@apolev
Copy link
Contributor

apolev commented Mar 26, 2016

#48

@apolev
Copy link
Contributor

apolev commented Mar 26, 2016

Кстати там в composer.json заявлена поддержка PHP выше 5.3.0, но библиотека в нем не будет работать: в 5 местах в разных файлах используется short array syntax. Мне эти места поправить или может быть поднимем версию до 5.5.0? 5.3.0 уже скоро 2 года как снят с поддержки.

@vasa-c
Copy link
Owner Author

vasa-c commented Mar 26, 2016

Кто-то раньше просил оставить совместимость с 5.3, мне было не жалко, так как кроме коротких массивов ничего нового не использовалось. Сейчас, наверное, уже не актуально.

А в каких местах short syntax? Я в самой библиотеке стараюсь без него обходиться, а в тестах внимания не обращаю.

@vasa-c vasa-c added this to the 2.3 milestone Mar 26, 2016
@vasa-c vasa-c self-assigned this Mar 26, 2016
@apolev
Copy link
Contributor

apolev commented Mar 26, 2016

Если Fakes не учитывать всего одно вхождение:
goDB/Fakes/Helpers/FakeQueries.php:26
goDB/Fakes/Helpers/FakeQueries.php:70
goDB/Fakes/Helpers/FakeQueries.php:87
goDB/Fakes/Helpers/Sort.php:27
goDB/Helpers/Templater.php:291

@vasa-c
Copy link
Owner Author

vasa-c commented Mar 26, 2016

Ну, раз немного, то исправил

vasa-c added a commit that referenced this issue Mar 27, 2016
@vasa-c
Copy link
Owner Author

vasa-c commented Mar 27, 2016

@vasa-c vasa-c closed this as completed Mar 27, 2016
@vasa-c
Copy link
Owner Author

vasa-c commented Mar 27, 2016

@mkedo @apolev https://github.com/vasa-c/go-db/wiki/table-format

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

No branches or pull requests

3 participants