@laravel_pro

Страница 812 из 2014
F01134H
09.10.2017
12:13:10
не, хочешь новое окружение, пили новый конфиг )
в случае с ларой - новое приложение

кек

ну или как

Денис
09.10.2017
12:14:04
хз, сливаешь с гита, делаешь инсталл, я в композере на инсталл делаю копирование .env

Google
Денис
09.10.2017
12:14:47
инсталл своего проекта конечно же, прежде создав проект на дев

F01134H
09.10.2017
12:17:01
т.е. по такой логике для своего проекта мне надо иметь копию под каждое окружение? :D

Anton
09.10.2017
12:17:55
т.е. по такой логике для своего проекта мне надо иметь копию под каждое окружение? :D
При смене окружения как минимум все джобы и прочее надо рестартить...

ну то есть это не самый простой проесс и я считаю что ничего сложного нет в том чтобы написать свой скрипт, который переключает .env на нужный + рестартует все что надо вокруг

F01134H
09.10.2017
12:19:01
благо есть докер

Денис
09.10.2017
12:19:01
или я не так понял

F01134H
09.10.2017
12:21:13
хм, вот у меня две бд крутятся

а как миграции на второй запустить :D

неудобно блэт

Денис
09.10.2017
12:24:21
контейнеры )

F01134H
09.10.2017
12:24:34
докер)

Денис
09.10.2017
12:25:09
именно )

Google
Dmitriy
09.10.2017
12:30:26
а как миграции на второй запустить :D
в миграциях можно изменить бд

Anton
09.10.2017
12:30:26
Я тут чет вспомнил... работал в одном достаточно известном стартапе... у них не было тестовой БД - была только боевая ::)

Dmitriy
09.10.2017
12:48:58
как это?
Schema::connection('db2')->create ...

F01134H
09.10.2017
12:49:12
Хуя ты Димон

в каких вещах шаришь

не ожидал от тебя

Anton
09.10.2017
12:50:00
но только это опять же не имеет отношения к смене .env - тебе может просто понадобится 2 БД для одного проекта

даже на разных хостах

?? Vladimir
09.10.2017
12:50:55
Всем привет. В laravel же работа с куками ведется только через стандартные функции php?

F01134H
09.10.2017
12:51:35
нет

Anton
09.10.2017
12:53:16
Мне одному кажется что если тебе надо в куки что-то большее кроме SID, то лучше уж тогда на JWT переходить?

Или это предвзято?

Dzhamil
09.10.2017
13:53:14
кто нибудьт работаь с firebase ?

Anton
09.10.2017
13:57:19
Чуть не спутал это с древней БД ?

Артем
09.10.2017
13:58:07
я работал

Dzhamil
09.10.2017
14:00:46
кастомную авторизацию делал?

Артем
09.10.2017
14:10:44
Это нет

F01134H
09.10.2017
15:11:37
Блин, вот интересно, надо ли выносить действия с бд в отдельный слой? У меня сейчас все в методах контроллера...

Google
F01134H
09.10.2017
15:11:55
получается и протестить эти методы можно только как целиковый кусок логики

Alex
09.10.2017
15:13:05
для этого создают репозитории

Anton
09.10.2017
15:13:43
то есть идея - если у тебя действие обновляет несколько моделей - хуячь репу для этого?

F01134H
09.10.2017
15:14:18
Alex
09.10.2017
15:15:28
а для чего?

F01134H
09.10.2017
15:15:44
для хранения и фильтрации сущностей

в ларке для коллекций например

не уверен, но вроде они его даже реализуют

Alex
09.10.2017
15:16:44
я видел реализацию репозиториев чисто для запросов к бд

F01134H
09.10.2017
15:16:55
мне не это нужно

Alex
09.10.2017
15:16:58
чтоб не делать запросы в контроллере

F01134H
09.10.2017
15:17:04
для запросов к бд я уже юзаю eloquent

в смысле не делать запросы в контроллере?

типо в репу вынести квери билдинг? Репа не для этого же лол

Alex
09.10.2017
15:17:55
типа, в репозитории вызывались методы eloquent

F01134H
09.10.2017
15:18:02
она для хранения сущностей, не для бизнес логики

Alex
09.10.2017
15:18:09
а в контроллере - методы репозитория

F01134H
09.10.2017
15:18:26
не должно в репозитории быть реализации запросов к бд

Google
F01134H
09.10.2017
15:19:04
а в контроллере - методы репозитория
вот убери из этого всего репозитории, оставь сервисы для запросов к бд - и это и будет мой вопрос

Anton
09.10.2017
15:21:37
не должно в репозитории быть реализации запросов к бд
Ну это спорное решение... Везде где я читал говорят - можно, но лучше все решать на уровне команды...

F01134H
09.10.2017
15:21:50
я нагуглил

и первый попавшийся вариант говорит что так делать не надо

Синоним слова репозиторий - коллекция

коллекция не должна билдить запросы к бд

и уж тем-более их выполнять

Anton
09.10.2017
15:27:09
первый не значит верный ?

F01134H
09.10.2017
15:27:28
:D

Anton
09.10.2017
15:27:31
Просто оба варианта имеют право на жизнь... я пока для себя не решил какой лучше

Enjoy the
09.10.2017
15:27:32
первый не значит верный ?
С девушками вроде так же работает

F01134H
09.10.2017
15:27:48
ну потому что это коллекция

Anton
09.10.2017
15:28:08
ну не зря же там два разных слова

F01134H
09.10.2017
15:28:09
вот есть библиотека (настоящая из реальной жизни), в ней работает библиотекарь

книги не должны сами переставляться и еще что то делать

Anton
09.10.2017
15:28:37
если бы была полное соотвествие то наверное одно слово применялось...

Вобщем я пока не готов к более глубоким спорам ?

и если в коде увижу что репозиторий делает запрос к БД то на ревью скорее всего пока пропущу ?

Google
Anton
09.10.2017
15:48:12
я пока для себя не решил - зло или нет... посмотрю кто какие еще печеньки предложит ?

Денис
09.10.2017
15:54:43
/link@yetanotherstatsbot

Yet Another Stats
09.10.2017
15:54:44
https://yasb.exileed.com/t/laravel_pro

Anton
09.10.2017
16:21:16
Стремительно ворвался в 5ку ?

Денис
09.10.2017
16:39:53
Вроде как сервисы должны делать запросы в базу

Взять симфони3. Там колекции автоматически генерятся

В случае с симфони, репозиторий только описывает свойства сущности и добавляет геттеры сеттеры, плюс может быть бизнес логика

Берешь этот репозиторий и кидаешь в сервис для записи в базу

Давайте пообсуждаем что нибудь ))))

А так взять тот же ангуляр 2, аналогично, сервисы отвечают за общение с бэком, а репы названы моделями

F01134H
09.10.2017
16:54:03
Это же фронт

Денис
09.10.2017
16:56:33
Принципы то одни

Репа не для запросов

F01134H
09.10.2017
16:58:14
Угу

Денис
09.10.2017
17:01:09
Встроенными средствами можно сделасть json-api? Не рест, отдавать всегда 200 {success: bool, result:[]}

Денис
09.10.2017
17:02:46
На сайте ларавель есть примеры?

Александр
09.10.2017
17:04:52
Просто по пути отдавать json, без авторизации?

Страница 812 из 2014