Как добавить дополнительное условие при авторизации laravel?

Question

[maks78945]

Добрый вечер подскажите как можно изменить метод авторизации пользователя.
У меня есть 2 таблицы User с полем company_id и другими страндартными полями, а так же талица Company, в которой есть id, name, active.
Мне нужно авторизировать пользователя только если у его компании active=1.
У меня получилось нагуглить решения как изменить метод Login если active в таблице User.
Но какие изменения мне необходимо внести что бы получилось реализовать мой вариант, где связь по company_id и active находится в другой таблице?

Answer 1

[Алексей]

Миграция - создания дополнительного, связывающего поля в существующую таблицу с внешним ключом.

public function up()
{
Schema::table('user', function (Blueprint $table) {
$table->integer('company_id')->unsigned()->default(1);
$table->foreign('company_id')->references('id')->on('company');
});
}
public function down()
{
Schema::table('user', function ($table) {
$table->dropForeign('user_company_id_foreign');
$table->dropColumn('company_id');
});
}

В модели User
public function company()
{
return $this->belongsTo('App\Company');
}

Проверка в login
$user->company->active

Answer 2

[jazzus]

company_id в users это неправильно. Нужно удалить и добавить user_id в companies. И создать связь user hasMany companies и company belongsTo user. Если компании также могут иметь много юзеров, то использовать manyToMany связь между User и Company. Запрашивать так

User::whereHas('companies', function ($query) {
    $query->where('active', true);
})->get();

Comments

[maks78945]

Подскажите, а почему нельзя использовать company_id в users.
я просто планировал использовать company_id во всех таблицах, products, orders и т.д.?

[jazzus]

maks78945, потому что тогда ты не сможешь написать user hasMany orders/companies/projects. Юзер создает сущности и они ему принадлежат. User hasMany orders и order belongsTo users, а не order hasMany users и user belongsTo order. Во всех таблицах не нужно использовать id т.к. сущности связываются по цепочке, в этом и смысл. Юзер с компаниями, компании с заказами, заказы с продуктами.

$user->companies()
    ->with('orders.products')
    ->get();

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

[Алексей]

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

Вот только в этом случае, если пользователи могут быть во множестве компаний, нужно создавать manyToMany. Если нет, то связь manyToMany - это усложнение приложения.

Во всех таблицах не нужно использовать id т.к. сущности связываются по цепочке, в этом и смысл.

Что за бред. Во всех таблицах нужно использовать id.

[jazzus]

Алексей, бред это писать $table->integer('company_id')->unsigned()->default(1); а здесь обсуждался company_id для связи со всеми таблицами, что делать нет смысла. manyTomany это не "усложение" приложения, а отражение реального мира где учредители могут иметь компании, а компании иметь сотрудников и сущности могут пересекаться.

[Алексей]

jazzus, это ваши домыслы и усложнение ситуации. Вас конкретно спросили

У меня есть 2 таблицы User с полем company_id и другими страндартными полями, а так же талица Company, в которой есть id, name, active.
Мне нужно авторизировать пользователя только если у его компании active=1.

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

manyTomany это не "усложение" приложения, а отражение реального мира где учредители могут иметь компании, а компании иметь сотрудников.

Это отражение вашего реального мира, в какой-то конкретной вашей ситуации.

[jazzus]

Алексей, я и дал правильный рабочий ответ. А не подпорку для костыля. Плюс автор явно начинащий который собирается company_id добавлять во все таблицы и ему лучше сразу понять что не нужно собирать связные id в таблице users, а использовать логичные и правильные решения.

Это отражение вашего реального мира, в какой-то конкретной вашей ситуации.

В моем мире (на планете Земля) отношения между компаниями и юзерами это manyToMany.