Как не допускать повторного добавления записей в БД при раскатке?

Question

[lexstile]

1. Использую passport, при миграции всегда добавляются 2 клиента в таблицу oauth_clients, как этого избежать?
2. Добавляю тарифы через сиды.
php artisan db:seed
Приходится проверять, а есть ли уже записи в таблице (не очень нравится подход). Нет ли штатных методов проверки или запуска только один раз?

class DatabaseSeeder extends Seeder
{
    public function run()
    {
        $this->call([
            TariffSeeder::class,
        ]);
    }
}

class TariffSeeder extends Seeder
{
    public function run()
    {
        if (Tariff::count()) {
            return;
        }

        $tariffs = collect([...]);

        $tariffs->each(fn($tariff) => Tariff::create($tariff));
    }
}

Comments

[Дмитрий]

Что мешает запускать конкретный класс php artisan db:seed --class=UserSeeder

Answer 1

[Антон Антон]

Запихай в миграции

Comments

[lexstile]

С тарифами так и сделаю, благодарю.
Но что делать с oauth_clients? (они же из коробки генерируются)

[Антон Антон]

lexstile, они делаются тоже миграциями. в любом случае можно сделать миграцию БЕЗ создания/изменения структуры таблиц, а только с добавлением/обработкой данных (и обратными действиями при откате)

[lexstile]

Антон Антон, то есть, нужно разобрать команду passport:install на составные?

composer install
php artisan migrate
php artisan passport:install
php artisan key:generate
php artisan storage:link

[Антон Антон]

lexstile, нет, нужно после инсталляции паспорта и выполнения его миграций добавить свою миграцию и в ней прописать добавление данных и запустить её.

[lexstile]

Антон Антон, не очень понял (может я туплю, конечно), как это поможет ограничить добавление данных в таблицу oauth_clients при повторных миграциях?

То есть, у меня уже более 200 записей в этой таблице, по 2 записи каждый раз добавляется при старте контейнера docker.

А я бы хотел, чтобы оставалось 2 первые.

[Антон Антон]

lexstile, одна миграция выполняется один раз.

[lexstile]

Антон Антон, я не понимаю, нужно заменить миграцию таблицы oauth_clients на свою?
Извините

[Антон Антон]

lexstile, не обязательно. инсталляция паспорта добавляет свои миграции.
можно либо её изменить, либо после неё нужно добавить свою миграцию. я за добавление новой миграции, так будет сразу понятно, что есть доработки относительно стандартного кода от вендора.

[Дмитрий]

Антон Антон, lexstile, Запихивать модели в миграции плохая практика

[Антон Антон]

Дмитрий, по ссылке все про джанго. Но вообще да, очевидно что модели не надо использовать, ибо проект развивается, код растет, добавляются вские скоупы, связи и прочие штуки. Только модели - не единственный способ добавления данных. Например есть query builder или вообще прямые запросы. В любом случае это должен быть способ, учитывающий конкретную структуру БД на момент после выполнения предыдущих миграций.

[Дмитрий]

Антон Антон, по ссылке гугл, там я думаю можно и про ларку найти. Но правило вообщем работает и для миграций ларки и джанго.

[lexstile]

Дмитрий, сделали run.sh при запуске контейнера, но тоже такое себе получилось:

# Проверка наличия записей в таблице oauth_clients
oauthClientsCount=$(php artisan db:query "SELECT COUNT(*) FROM oauth_clients" --no-ansi | grep -Eo '[0-9]+')
if [ "$oauthClientsCount" == "0" ]; then
    # Установка пакета Passport
    php artisan passport:install
fi

# Проверка наличия записей в таблице tariffs
tariffsCount=$(php artisan db:query "SELECT COUNT(*) FROM tariffs" --no-ansi | grep -Eo '[0-9]+')
if [ "$tariffsCount" == "0" ]; then
    # Заполнение таблицы tariffs сидами
    php artisan db:seed
fi

[Дмитрий]

lexstile, ну зачем так сложно то? Пишете класс TarifSeeder. В нем проверяете если нет - добавляем, если есть нет. После чего не паримся и просто шарашим командой:

php artisan db:seed --class=TarifSeeder

[lexstile]

Дмитрий, пожалуй, вы правы насчет сидов, спасибо!
но passport install иначе, наверно, никак

[Дмитрий]

lexstile,
Я не помню что там passport:install делает - ну и лениво разбираться. Но что может вам помешать взять код команды https://github.com/laravel/passport/blob/8cae3ef2c...

И сделать из нее свою которая будет делать только то что вам нужно?