Это нормально, что запрос Model::all() возвращает и удалённые(Soft deleted) модели?

Question

[Евгений Ромашкан]

В документации сказано, что Soft Deleted модели автоматически исключаются из любых выборок.

spoiler

As noted above, soft deleted models will automatically be excluded from query results. However, you may force soft deleted models to appear in a result set using the withTrashed method on the query:

$flights = App\Flight::withTrashed()
->where('account_id', 1)
->get();
https://laravel.com/docs/5.6/eloquent#deleting-models

Однако методы Model::all() и Model::get() мне возвращает все модели, в том числе и удалённые. Это нормальное поведение? И как тогда выбирать все модели кроме удалённых?

Вот он весь мой код:

public function index()
    {
        return Category::all();
    }

Код модели:

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;

class Category extends Model
{
    use SoftDeletes;

    protected $dates = ['deleted_at'];

    public static function boot() {
        static::deleting(function(Category $category) {
            //$subcategories = $category->subcategories;
            //foreach ($subcategories as $subcategory) $subcategory->delete();
        });
    }
}

Comments

[Antonio Solo]

soft delete (на уровне PHP) это зло,

поначалу выглядит красиво,а по-факту нарушает всю стройность реляционной модели, уникальность индексов, внешних ключей.

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

soft delete опять же работает только с элокентом, для всех запросов формируемых другими способами придется про эти софтделиты помнить.

Answer 1

[JhaoDa]

Путём задавания множества вопросов удалось увидеть код, который написал автор. Из кода стало ясно, что он переопределил в модели метод boot, но забыл вызвать родительский метод boot.

Вообще, переопределять boot ради модельных событий — плохая практика. Почитайте документацию на предмет узнавания хорошей практики.

Comments

[Евгений Ромашкан]

JhaoDa, Вы были правы. Я ошибался. Спасибо за ответ

Answer 2

[Александр Аксентьев]

Значит что-то сильно неправильно делаете.

Код из документации ничего не говорит о вашем коде.

Comments

[Евгений Ромашкан]

Вот он весь мой код:

public function index()
    {
        return Category::all();
    }

Вот строки в БД


А вот результат

[{"id":6,"name":"asd","created_at":"2018-05-01 11:27:38","updated_at":"2018-05-01 11:27:38","deleted_at":null},{"id":7,"name":"qwe","created_at":"2018-05-01 11:27:41","updated_at":"2018-05-01 11:27:51","deleted_at":"2018-05-01 11:27:51"}]

[JhaoDa]

Евгений Ромашкан, и из этого кода тоже не видно, как вы заюзали soft delete.

[Евгений Ромашкан]

JhaoDa, Вызвал вот такой метод на категории, что помечена теперь в бд как soft deleted

public function destroy(Category $category)
    {
        $category->delete();
        return back();
    }

[JhaoDa]

Евгений Ромашкан, и? Я разве это спросил? Я спросил — как вы заюзали в модели soft delete? Вы его там вообще заюзали?

[Евгений Ромашкан]

JhaoDa, Вот код самой модели

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;

class Category extends Model
{
    use SoftDeletes;
    protected $dates = ['deleted_at'];
	protected $guarded = [];
    protected $fillable = ['name'];
    
    public function subcategories()
    {
    	return $this->hasMany(Subcategory::class);
    }

    public function products()
    {
        return $this->hasManyThrough(Product::class, Subcategory::class);
    }

    public function addSubcategory($name)
    {
    	$this->subcategories()->create([
    		'name' => $name, 
    		'category_id' => $this->id,
    	]);
    }
    
    public static function boot() {
        static::deleting(function(Category $category) {
            //$subcategories = $category->subcategories;
            //foreach ($subcategories as $subcategory) $subcategory->delete();
        });
    }
}

[JhaoDa]

Евгений Ромашкан, вы забыли вызвать родительский boot. И вообще, переопределять boot ради модельных событий — плохая практика. Почитайте документацию на предмет узнавания хорошей практики.

[Евгений Ромашкан]

JhaoDa, boot удаляет подкатегории. О них сейчас речь не идёт. SoftDelete я вызываю на категории, и получаю тоже категории. Те строки из БД тоже они.

[JhaoDa]

Евгений Ромашкан, вы вообще читаете, что в пишу?

[Евгений Ромашкан]

JhaoDa, Хорошо. Я перечитаю тему событий в документации, после того как разберусь что не так происходит с удалением категорий. А пока что это относится к другой модели, которая в вопросе не фигурирует.

[JhaoDa]

Евгений Ромашкан, я написал, что не так. При чем тут какая-то другая модель? Или вы не понимаете, что делает метод boot? Ну не понимайте, просто попробуйте моё решение.

[Евгений Ромашкан]

JhaoDa, Читаю. Ок. Вот вам код модели:

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;

class Category extends Model
{
    use SoftDeletes;
    protected $dates = ['deleted_at'];
	protected $guarded = [];
    protected $fillable = ['name'];
    
    public function subcategories()
    {
    	return $this->hasMany(Subcategory::class);
    }

    public function products()
    {
        return $this->hasManyThrough(Product::class, Subcategory::class);
    }

    public function addSubcategory($name)
    {
    	$this->subcategories()->create([
    		'name' => $name, 
    		'category_id' => $this->id,
    	]);
    }
}

Вы довольны? Нету там больше метода boot. А теперь я повторю свой вопрос: Почему метод all() выдаёт мне все категории, включая удаленные?

[JhaoDa]

Евгений Ромашкан, а вот теперь и не должен выдавать. Значит, вы чего-то недоговариваете.

[Евгений Ромашкан]

JhaoDa, ОК. Вы правы. Извинияюсь. Спасибо за ответ. Я не знал что за это отвечает метод boot /

[hakkol]

JhaoDa,

И вообще, переопределять boot ради модельных событий — плохая практика.

С чего вы так решили? А вы сами читали доку на eloquent, хоть это?

[JhaoDa]

hakkol, начнём с того, что это не документация. Впрочем, этим и закончим, потому что, во-первых, актуальная версия документации рекомендует иной подход, а во-вторых, я не сказал. что так делать нельзя — я сказал, что так делать не надо. Это как валидация в контроллере вместо форм реквеста. И как написание логики в файлах роутинга. Понимаете разницу?

[hakkol]

JhaoDa, Я это к тому написал, что человек может вас не так понять. Ваш пример про валидацию в контроллере - прям в точку. Было бы уместно пояснить или хоть ссылку дать, что не так, чтобы вас точно поняли.

[JhaoDa]

hakkol, я написал, что надо сделать для узнавания того, как юзать события правильно. Ссылки я уже давно стараюсь не давать, ибо толку от этого ноль — доку надо читать всю, с первой страницы, сплошняком.

[Александр Аксентьев]

hakkol, LoL
20 лайфхаков для тех кто не смог в документацию/логику судя по всему, потому как там ничего нового абсолютно нет.
А переопределение boot как раз правильно сделано с вызовом parent::boot, а не как у автора вопроса.

[hakkol]

JhaoDa, Ок, я понял вашу точку зрения

[hakkol]

Александр Аксентьев, Поверьте, есть много людей, которые с laravel работают и не знают, что "так можно было?"