Skip to content

Commit

Permalink
Merge pull request #21 from sander3/user-model-bugfix
Browse files Browse the repository at this point in the history
Resolves #20
  • Loading branch information
sander3 committed Nov 10, 2018
2 parents c98e961 + 9e7015f commit 03bfb49
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 17 deletions.
8 changes: 4 additions & 4 deletions src/Events/GdprDownloaded.php
Expand Up @@ -2,25 +2,25 @@

namespace Soved\Laravel\Gdpr\Events;

use App\User;
use Illuminate\Queue\SerializesModels;
use Illuminate\Contracts\Auth\Authenticatable;

class GdprDownloaded
{
use SerializesModels;

/**
* @var \App\User
* @var \Illuminate\Contracts\Auth\Authenticatable
*/
public $user;

/**
* Create a new event instance.
*
* @param \App\User $user
* @param \App\Authenticatable $user
* @return void
*/
public function __construct(User $user)
public function __construct(Authenticatable $user)
{
$this->user = $user;
}
Expand Down
8 changes: 4 additions & 4 deletions src/Events/GdprInactiveUser.php
Expand Up @@ -2,25 +2,25 @@

namespace Soved\Laravel\Gdpr\Events;

use App\User;
use Illuminate\Queue\SerializesModels;
use Illuminate\Contracts\Auth\Authenticatable;

class GdprInactiveUser
{
use SerializesModels;

/**
* @var \App\User
* @var \Illuminate\Contracts\Auth\Authenticatable
*/
public $user;

/**
* Create a new event instance.
*
* @param \App\User $user
* @param \Illuminate\Contracts\Auth\Authenticatable $user
* @return void
*/
public function __construct(User $user)
public function __construct(Authenticatable $user)
{
$this->user = $user;
}
Expand Down
8 changes: 4 additions & 4 deletions src/Events/GdprInactiveUserDeleted.php
Expand Up @@ -2,25 +2,25 @@

namespace Soved\Laravel\Gdpr\Events;

use App\User;
use Illuminate\Queue\SerializesModels;
use Illuminate\Contracts\Auth\Authenticatable;

class GdprInactiveUserDeleted
{
use SerializesModels;

/**
* @var \App\User
* @var \Illuminate\Contracts\Auth\Authenticatable
*/
public $user;

/**
* Create a new event instance.
*
* @param \App\User $user
* @param \App\Authenticatable $user
* @return void
*/
public function __construct(User $user)
public function __construct(Authenticatable $user)
{
$this->user = $user;
}
Expand Down
10 changes: 5 additions & 5 deletions src/Jobs/Cleanup/Strategies/DefaultStrategy.php
Expand Up @@ -2,9 +2,9 @@

namespace Soved\Laravel\Gdpr\Jobs\Cleanup\Strategies;

use App\User;
use Carbon\Carbon;
use Illuminate\Database\Eloquent\Collection;
use Illuminate\Contracts\Auth\Authenticatable;
use Soved\Laravel\Gdpr\Events\GdprInactiveUser;
use Soved\Laravel\Gdpr\Jobs\Cleanup\CleanupStrategy;
use Soved\Laravel\Gdpr\Events\GdprInactiveUserDeleted;
Expand Down Expand Up @@ -48,11 +48,11 @@ private function notifyInactiveUsers(
Collection $users
) {
$users->filter(
function (User $user) use ($inactivity, $notificationThreshold) {
function (Authenticatable $user) use ($inactivity, $notificationThreshold) {
return $user->last_activity->diffInDays($inactivity)
=== $notificationThreshold;
}
)->each(function (User $user) {
)->each(function (Authenticatable $user) {
event(new GdprInactiveUser($user));
});
}
Expand All @@ -68,9 +68,9 @@ private function deleteInactiveUsers(
Carbon $inactivity,
Collection $users
) {
$users->filter(function (User $user) use ($inactivity) {
$users->filter(function (Authenticatable $user) use ($inactivity) {
return $user->last_activity < $inactivity;
})->each(function (User $user) {
})->each(function (Authenticatable $user) {
$user->delete();

event(new GdprInactiveUserDeleted($user));
Expand Down

0 comments on commit 03bfb49

Please sign in to comment.