f9webltd / laravel-deletable Goto Github PK

Implement method to disable deletion checks, i.e. so the result of isDeletable() is always true and any checks within isDeletable() are skipped. The API would look something along the lines of: the following.

A user may have a "deletable" restriction:

namespace App;

use F9Web\LaravelDeletable\Traits\RestrictsDeletion;
use Illuminate\Database\Eloquent\Model;

class User extends Model
{
  use RestrictsDeletion;
  
  public function isDeletable() : bool
  {
    return $this->orders()->doesntExist();
  }  
}

Assuming the user had orders, the following would fail and the model would not be deleted:

$user->delete();

The following would allow the deletable check to be skipped and the model would be deleted:

$user->withoutDeletableRestrictions()->delete()

Hello,

Can you update the composer.json to support Laravel 8?

Ads a simple trait called NoneDeletableModel to the package. The trait could then be used in Eloquent models to deny all deletion attempts:

declare(strict_types=1);

namespace F9Web\LaravelDeletable\Traits\Traits;

use F9Web\LaravelDeletable\Traits\RestrictsDeletion;

trait NoneDeletableModel
{
    use RestrictsDeletion;

    public function isDeletable() : bool
    {
        return false;
    }
}

To prevent all deletion requests for a model:

declare(strict_types=1);

namespace App\Models;

use F9Web\LaravelDeletable\Traits\NoneDeletableModel;
use Illuminate\Database\Eloquent\Model;

class Order extends Model
{
   use NoneDeletableModel;
}

Now, orders cannot be deleted.

It would be nice to have support of mass delete, e.g. User::where('is_archived', true)->delete() . Something like isMassDeletable(): bool. Laravel SoftDeletes Trait supports mass delete, so its code may be helpful.

PHP 8 compatibility isn't included in the composer.json.