feat: sync, webhook, tokens refresh, fillable, key config

README.md

@@ -7,9 +7,6 @@ The Bluesquare Connect package allows you to use its OAuth server and sync its r
 Update your `composer.json`:
 
 ```
-"require": {
-    "bluesquare/laravel-connect": "dev-master"
-}
 "repositories": [
     {
         "type": "vcs",
@@ -21,7 +18,7 @@ Update your `composer.json`:
 Install the package:
 
 ```bash
-composer update bluesquare/laravel-connect
+composer require bluesquare/laravel-connect "1.2"
 ```
 
 Finally, update your `.env` with your client's credentials:

composer.json

@@ -35,8 +35,8 @@
         }
     },
     "require": {
-        "guzzlehttp/guzzle": "^7.2",
+        "guzzlehttp/guzzle": "^7.3",
-        "php": "^7.2"
+        "php": "^7.3|^8.0"
     },
     "prefer-stable": true
 }

config/bconnect.php

@@ -5,7 +5,7 @@ return [
     /**
      * OAuth model
      */
-    'model' => \App\User::class,
+    'model' => \App\Models\User::class,
 
     /**
      * Route that redirects to Bluesquare Connect
@@ -13,18 +13,19 @@ return [
     'login_url' => '/connect/authorize',
 
     /**
+     * Use post-login remember cookie
+     */
+    'login_remember' => true,
+
+    /**
      * OAuth callback URL
      */
     'redirect_url' => env('BCONNECT_REDIRECT', 'http://localhost:8000/connect/callback'),
 
     /**
-     * OAuth client id
+     * OAuth client identifiers
      */
     'client_id' => env('BCONNECT_CLIENT_ID', null),
-
-    /**
-     * OAuth client secret
-     */
     'client_secret' => env('BCONNECT_CLIENT_SECRET', null),
 
     /**

src/Commands/RefreshTokens.php

@@ -0,0 +1,45 @@
+<?php
+
+namespace Bluesquare\Connect\Commands;
+
+use Bluesquare\Connect\Connect;
+use Bluesquare\Connect\ConnectException;
+use Bluesquare\Connect\Traits\HasConnectTokens;
+use Illuminate\Console\Command;
+
+class RefreshTokens extends Command
+{
+    protected $signature = 'connect:refresh';
+
+    protected $description = 'Refresh Bluesquare Connect tokens';
+
+    public function handle(Connect $connect)
+    {
+        $class = config('bconnect.model');
+
+        $has_fields = in_array(HasConnectTokens::class, class_uses($class));
+
+        if (! $has_fields) {
+            throw new ConnectException("$class does not implement HasConnectTokens");
+        }
+
+        $class::query()->chunks(10, function ($models) use ($connect) {
+            $models->each(function ($model) use ($connect) {
+                if (! empty($model->connect_refresh_token) && $model->connect_expires_at <= now()->addHour()) {
+                    try {
+                        $tokens = $connect->getAccessTokenFromRefreshToken($model->connect_refresh_token);
+                        $connect->updateUserConnectData($model, $tokens);
+                        $model->save();
+                    }
+                    catch (\Exception $exception) {
+                        $this->warn("Failed to refresh model tokens", $model->toArray());
+                    }
+                }
+            });
+        });
+
+        $this->info("Tokens refreshed");
+
+        return 0;
+    }
+}

src/Commands/Sync.php

@@ -3,41 +3,45 @@
 namespace Bluesquare\Connect\Commands;
 
 use Bluesquare\Connect\Connect;
+use Bluesquare\Connect\ConnectException;
+use Bluesquare\Connect\Traits\HasConnectTokens;
 use Illuminate\Console\Command;
 
 class Sync extends Command
 {
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
     protected $signature = 'connect:sync';
 
-    /**
+    protected $description = 'Sync Bluesquare Connect users';
-     * The console command description.
-     *
-     * @var string
-     */
-    protected $description = 'Synchronize Bluesquare Connect resources';
 
-    /**
-     * Create a new command instance.
-     *
-     * @return void
-     */
-    public function __construct()
-    {
-        parent::__construct();
-    }
-
-    /**
-     * Execute the console command.
-     *
-     * @return mixed
-     */
     public function handle(Connect $connect)
     {
-        $connect->syncAll();
+        $this->call('connect:refresh');
+
+        $class = config('bconnect.model');
+
+        $has_fields = in_array(HasConnectTokens::class, class_uses($class));
+
+        if (! $has_fields) {
+            throw new ConnectException("$class does not implement HasConnectTokens");
+        }
+
+        $class::query()->chunks(10, function ($models) use ($connect) {
+            $models->each(function ($model) use ($connect) {
+                try {
+                    if (! empty($model->connect_access_token)) {
+                        $data = $connect->getUserData($model->connect_access_token);
+                        $connect->updateUserData($model, $data);
+                        $model->save();
+                    }
+                }
+                catch (\Exception $exception) {
+                    $this->warn("Failed to sync model data", $model->toArray());
+                }
+            });
+        });
+
+        $this->info("Models synced");
+
+        return 0;
     }
 }

src/Connect.php

@@ -2,8 +2,9 @@
 
 namespace Bluesquare\Connect;
 
-use Bluesquare\Connect\Traits\HasConnectSync;
+use Bluesquare\Connect\Traits\HasConnectData;
 use Bluesquare\Connect\Traits\HasConnectTokens;
+use Bluesquare\Connect\Traits\HasConnectWebhook;
 use GuzzleHttp\Client;
 use Illuminate\Http\Request;
 use Illuminate\Routing\Router;
@@ -14,59 +15,13 @@ use Psr\Http\Message\StreamInterface;
 
 class Connect
 {
-    protected static $resources = [
-        'Role',
-        'Company',
-        'Team',
-        'User',
-        'UserTeam'
-    ];
-
-    protected static $foreignKeys = [
-        'role_id' => 'Role',
-        'company_id' => 'Company',
-        'team_id' => 'Team',
-        'user_id' => 'User',
-        'user_teams_id' => 'UserTeam'
-    ];
-
     protected $app;
-    protected $synchronized = [];
 
     public function __construct($app)
     {
         $this->app = $app;
     }
 
-    // User config
-
-    public function setSynchronized($models)
-    {
-        $items = [];
-
-        foreach ($models as $model)
-        {
-            if (!in_array(HasConnectSync::class, class_uses($model)))
-                throw new ConnectException("$model does not implement HasConnectSync trait.");
-
-            $class = explode('\\', $model);
-            $resource = $model::$connectResource ?? end($class);
-
-            $items[$resource] = $model;
-        }
-
-        $synchronized = [];
-
-        foreach (self::$resources as $resourceType) { // Re-ordering
-            foreach ($items as $resource => $model) {
-                if ($resource == $resourceType)
-                    $synchronized[$resource] = $model;
-            }
-        }
-
-        $this->synchronized = $synchronized;
-    }
-
     // API
 
     /**
@@ -76,7 +31,7 @@ class Connect
      * @return array
      * @throws ConnectException
      */
-    public function request($method, $uri, $data = null, $auth = true): array
+    public function request($method, $uri, $data = null, $access_token = null): array
     {
         $url = $this->getUrl();
         $url = $url . '/' . trim($uri, '/');
@@ -89,14 +44,11 @@ class Connect
             ]
         ];
 
-        if ($auth === true) {
+        if (! is_null($access_token)) {
-            $config['headers']['Authorization'] = 'Bearer ' . $this->getAccessToken();
+            $config['headers']['Authorization'] = 'Bearer ' . $access_token;
-        }
-        elseif ($auth !== false) {
-            $config['headers']['Authorization'] = 'Bearer ' . $auth;
         }
 
-        if (!is_null($data)) {
+        if (! is_null($data)) {
             $config['form_params'] = $data;
         }
 
@@ -110,12 +62,13 @@ class Connect
             return json_decode($body, true);
 
         } catch(\Exception $e) {
-            $this->deleteAccessToken();
+            $this->flushTokens();
+
             throw new ConnectException($e->getMessage());
         }
     }
 
-    // OAuth (user)
+    // Authorization flow
 
     public function redirect($state = null)
     {
@@ -123,9 +76,12 @@ class Connect
             $state = Str::random();
 
         $states = session()->get('connect_states');
-        if (!is_array($states))
+
+        if (! is_array($states))
             $states = [];
+
         $states[] = $state;
+
         session()->put('connect_states', $states);
 
         $query = http_build_query([
@@ -137,106 +93,80 @@ class Connect
         ]);
 
         $url = $this->getUrl() . '/oauth/authorize?' . $query;
+
         return redirect()->to($url);
     }
 
     public function checkState(Request $request)
     {
-        if (!session()->has('connect_states')) {
+        if (! session()->has('connect_states'))
-            Log::debug("Missing session states");
             return false;
-        }
 
         $states = session()->get('connect_states');
 
-        if (!is_array($states)) {
+        if (! is_array($states))
-            Log::debug("Invalid session state");
             return false;
-        }
 
-        if (!$request->has('state') || !in_array($request->state, $states)) {
+        if (! $request->has('state') || ! in_array($request->state, $states))
-            Log::debug("Missing valid state in request");
             return false;
-        }
 
         unset($states[array_search($request->state, $states)]);
 
         session()->put('connect_states', $states);
+
         return true;
     }
 
     public function loginFromCallback(Request $request, $redirect_to = '/')
     {
-        if (!$this->checkState($request))
+        if (! $this->checkState($request) || ! $request->has('code'))
             return redirect('/');
 
-        // Code check
-
-        if (!$request->has('code')) {
-            Log::debug("Missing authorization code");
-            return redirect('/');
-        }
-
         // Access token
+
         $expires_at = now();
-        $connect_data = $this->getAccessTokenFromAuthorizationCode($request->code);
+        $connect_data = $this->getAccessTokenFromAuthorizationCode($request->get('code'));
         $connect_data['expires_at'] = $expires_at->addSeconds($connect_data['expires_in']);
 
-        $model_data = $this->getUserData($connect_data['access_token']);
+        // User data
-        $model = config('bconnect.model');
 
-        if (in_array($model, $this->synchronized)) {
+        $user_data = $this->getUserData($connect_data['access_token']);
-            $user = $model::findConnectResource($model_data['id']) ?? new $model;
-        }
-        else {
-            $user = $model::where('email', $model_data['email'])->first() ?? new $model;
-        }
-
-        $model_data = $this->convertForeignKeys($model_data);
-
-        $user->fill($model_data);
-
-        if (in_array($model, $this->synchronized))
-            $user->{$model::$connectColumnId} = $model_data['id'];
 
+        $user = $this->sync('create', $user_data);
+        $this->updateUserConnectData($user, $connect_data);
         $user->save();
 
-        if (in_array($model, $this->synchronized))
+        // Login
-            $user = $model::findConnectResource($model_data['id']);
 
-        $this->updateUserConnectData($user, $connect_data);
+        auth()->login($user, config('bconnect.login_remember', true));
-
-        auth()->login($user, true);
 
         return redirect($redirect_to);
     }
 
+    // OAuth methods
+
     public function getAccessTokenFromAuthorizationCode($code)
     {
-        $data = $this->request('post', 'oauth/token', [
+        return $this->request('post', 'oauth/token', [
             'grant_type' => 'authorization_code',
             'client_id' => config('bconnect.client_id'),
             'client_secret' => config('bconnect.client_secret'),
             'scope' => config('bconnect.user_scopes'),
             'redirect_uri' => config('bconnect.redirect_url'),
             'code' => $code
-        ], false);
+        ]);
-
-        return $data;
     }
 
     public function getAccessTokenFromRefreshToken($refresh_token)
     {
-        $data = $this->request('post', 'oauth/token', [
+        return $this->request('post', 'oauth/token', [
             'grant_type' => 'refresh_token',
             'client_id' => config('bconnect.client_id'),
             'client_secret' => config('bconnect.client_secret'),
             'scope' => config('bconnect.user_scopes'),
             'redirect_uri' => config('bconnect.redirect_url'),
             'refresh_token' => $refresh_token
-        ], false);
+        ]);
-
-        return $data;
     }
 
     public function getUserData($access_token)
@@ -244,164 +174,98 @@ class Connect
         return $this->request('get', 'api/user', null, $access_token);
     }
 
-    public function updateUserConnectData($user, $data)
+    public function getUserAccessToken($model)
     {
-        if (!in_array(HasConnectTokens::class, class_uses(get_class($user))))
+        $class = get_class($model);
-            return false;
+        $has_fields = in_array(HasConnectTokens::class, class_uses($class));
 
-        $user->connect_access_token = $data['access_token'];
+        if (! $has_fields) {
-        $user->connect_refresh_token = $data['refresh_token'];
+            throw new ConnectException("$class does not implement HasConnectTokens");
-        $user->connect_expires_at = $data['expires_at'];
-
-        return $user->save();
-    }
-
-    public function getUserAccessToken($user)
-    {
-        $has_fields = in_array(HasConnectTokens::class, class_uses(get_class($user)));
-
-        if (!$has_fields) {
-            throw new ConnectException("User class does not implement HasConnectTokens");
         }
 
-        if ($user->connect_expires_at <= now()) {
+        if ($model->connect_expires_at <= now()->addHour()) {
-            $connect_data = $this->getAccessTokenFromRefreshToken($user->connect_refresh_token);
+            $connect_data = $this->getAccessTokenFromRefreshToken($model->connect_refresh_token);
-            $this->updateUserConnectData($user, $connect_data);
+            $this->updateUserConnectData($model, $connect_data);
             return $connect_data['access_token'];
         }
 
-        return $user->connect_access_token;
+        return $model->connect_access_token;
     }
 
-    // OAuth (client)
+    // Sync
 
-    public function getAccessToken()
+    public function sync(string $event, array $data)
     {
-        $access_token = cache()->get('bconnect.access_token');
+        $class = config('bconnect.model');
-        $access_token_expiration = cache()->get('bconnect.access_token_expiration');
 
-        if ($access_token && $access_token_expiration > time() + 60) {
+        if (in_array($event, ['update', 'delete'])) {
-            return $access_token;
+            $model = $this->resolveUser($data);
+
+            if (! $model->exists() && $event === 'delete')
+                return $model;
         }
+        else {
+            $hasSoftDeletes = in_array(\Illuminate\Database\Eloquent\SoftDeletes::class, class_uses($class));
+            $model = $this->resolveUser($data, $hasSoftDeletes);
 
-        $data = $this->request('post', '/oauth/token', [
+            if ($model->exists()) {
-            'grant_type' => 'client_credentials',
+                if ($hasSoftDeletes) {
-            'client_id' => config('bconnect.client_id'),
+                    $event = 'restore';
-            'client_secret' => config('bconnect.client_secret'),
+                } else {
-            'scope' => config('bconnect.client_scopes')
+                    $event = 'update';
-        ], false);
+                }
-
-        cache()->set('bconnect.access_token', $data['access_token']);
-        cache()->set('bconnect.access_token_expiration', time() + $data['expires_in']);
-
-        return $data['access_token'];
-    }
-
-    public function deleteAccessToken()
-    {
-        cache()->delete('bconnect.access_token');
-        cache()->delete('bconnect.access_token_expiration');
-    }
-
-    // Webhook handler
-
-    /**
-     * @param Request $request
-     * @return bool
-     */
-    public function handleWebhook(Request $request)
-    {
-        $data = $request->validate([
-            'connectEventType' => 'required|in:created,updated,deleted',
-            'connectResourceType' => 'required',
-            'connectResourceTable' => 'required',
-            'connectResourceData' => 'required|array',
-            'connectResourceData.id' => 'required'
-        ]);
-
-        if (!array_key_exists($data['connectResourceType'], $this->synchronized))
-            return false;
-
-        $model = $this->synchronized[$data['connectResourceType']];
-        $method = $this->getEventMethod($data['connectEventType']);
-
-        $data = $data['connectResourceData'];
-
-        try {
-            $data = $this->get($data['connectResourceType'], $data['connectResourceData']['id']);
-            if ($data['connectEventType'] == 'deleted') {
-                abort(403, "This resource still exists.");
-            }
-        } catch (\Exception $e) {
-            if ($data['connectEventType'] != 'deleted') {
-                abort(404, "Could not retrieve this resource.");
             }
         }
 
-        $data = $this->convertForeignKeys($data);
+        if (in_array(HasConnectWebhook::class, class_uses($class))) {
-
+            $method = 'onConnect' . ucfirst($event);
-        $model::$method($data['id'], $data);
+            $model->$method($data);
-
+        } else {
-        return true;
+            $this->updateUserData($model, $data);
-    }
+            $model->save();
-
-    // Resources getters
-
-    public function getAll($resourceType)
-    {
-        return $this->request('get', "api/resources/$resourceType");
-    }
-
-    public function get($resourceType, $resourceId)
-    {
-        return $this->request('get', "api/resources/$resourceType/$resourceId");
-    }
-
-    // Resources syncing
-
-    public function syncAll($resourceTypes = null)
-    {
-        $resourceTypes = $resourceTypes ?? $this->synchronized;
-
-        foreach ($resourceTypes as $resourceType)
-        {
-            $resourceType = $this->resolveResourceType($resourceType);
-
-            if (!array_key_exists($resourceType, $this->synchronized))
-                throw new ConnectException("Resource $resourceType not declared as synchronized.");
-
-            $resources = $this->getAll($resourceType);
-            $model = $this->synchronized[$resourceType];
-            $identifiers = [];
-
-            foreach ($resources as $data)
-            {
-                $identifiers[] = intval($data['id']);
-                $this->sync($resourceType, $data['id'], $data);
-            }
-
-            foreach ($model::all() as $item)
-            {
-                if (!in_array(intval($item->{$model::$connectColumnId}), $identifiers))
-                    $model::onConnectResourceDoesNotExist($item);
-            }
-        }
-    }
-
-    public function sync($resourceType, $resourceId, $resourceData = null)
-    {
-        $resourceType = $this->resolveResourceType($resourceType);
-
-        if (is_null($resourceData)) {
-            $resourceData = $this->get($resourceType, $resourceId);
         }
 
-        $model = $this->synchronized[$resourceType];
+        return $model;
-        $item = $model::findConnectResource($resourceId);
+    }
-        $method = $this->getEventMethod($item ? 'updated' : 'created');
 
-        $data = $this->convertForeignKeys($resourceData);
+    public function updateUserConnectData($user, $data)
-        $model::$method($resourceId, $data);
+    {
+        if (in_array(HasConnectTokens::class, class_uses(get_class($user))))
+            $user->fillConnectTokens($data);
+    }
+
+    public function updateUserData($user, $data)
+    {
+        if (in_array(HasConnectData::class, class_uses(get_class($user))))
+            $user->fillConnectData($data);
+    }
+
+    protected function resolveUser($data, $withTrashed = false)
+    {
+        $class = config('bconnect.model');
+        $query = $class::query();
+
+        if ($withTrashed)
+            $query->withTrashed();
+
+        $model = new $class;
+
+        if (in_array(HasConnectData::class, class_uses($class))) {
+            $id = $model->getConnectIdentifier();
+
+            $origin = is_array($id) ? $id[0] : $id;
+            $target = is_array($id) ? $id[1] : $id;
+
+            $model = $query->where($target, $data[$origin])->first() ?? $model;
+            $model->$target = $data[$origin];
+        } else {
+            $model = $query->where('email', $data['email'])->first() ?? $model;
+
+            if (! $model->exists())
+                $model->email = $data['email'];
+        }
+
+        return $model;
     }
 
     // Routing
@@ -425,44 +289,14 @@ class Connect
 
     // Misc
 
-    protected function resolveResourceType($class)
+    protected function flushTokens()
     {
-        if (in_array($class, $this->synchronized))
+        session()->forget('bconnect.access_token');
-            return array_flip($this->synchronized)[$class];
+        session()->forget('bconnect.access_token_expiration');
-
-        return $class;
-    }
-
-    protected function resolveResourceModel($class)
-    {
-        if (array_key_exists($class, $this->synchronized))
-            return $this->synchronized[$class];
-
-        return $class;
-    }
-
-    protected function getEventMethod($event)
-    {
-        return 'onConnectResource' . ucfirst($event);
     }
 
     protected function getUrl()
     {
         return config('bconnect.url') ?? 'https://connect.bluesquare.io';
     }
-
-    protected function convertForeignKeys($data)
-    {
-        foreach (self::$foreignKeys as $key => $resourceType)
-        {
-            if (!array_key_exists($key, $data)) continue;
-            if (!array_key_exists($resourceType, $this->synchronized)) continue;
-
-            $model = $this->resolveResourceModel($resourceType);
-            $record = $model::findConnectResource($data[$key]);
-            $data[$key] = $record ? $record->id : null;
-        }
-
-        return $data;
-    }
 }

src/ConnectServiceProvider.php

@@ -2,70 +2,54 @@
 
 namespace Bluesquare\Connect;
 
+use Bluesquare\Connect\Commands\RefreshTokens;
 use Bluesquare\Connect\Commands\Sync;
 use Illuminate\Support\ServiceProvider;
 
 class ConnectServiceProvider extends ServiceProvider
 {
-    /**
-     * Register any application services.
-     *
-     * @return void
-     */
     public function register()
     {
-        // Config
+        $this->mergeConfigFrom($this->path('config/bconnect.php'), 'bconnect');
-
-        $this->mergeConfigFrom(
-            __DIR__ . '/../config/bconnect.php',
-            'bconnect'
-        );
-
-        // Singletons
 
         $this->app->singleton(Connect::class, function ($app) {
             return new Connect($app);
         });
     }
 
-    /**
-     * Bootstrap any application services.
-     *
-     * @return void
-     */
     public function boot()
     {
-        // Config
+        $config_path = $this->path('config/bconnect.php');
+        $views_path = $this->path('resources/views/connect');
 
         $this->publishes([
-            __DIR__ . '/../config/bconnect.php' => config_path('bconnect.php')
+            $config_path => config_path('bconnect.php'),
+            $views_path => resource_path('views/vendor/connect'),
         ]);
 
-        // Translations
+        $this->loadTranslationsFrom($this->path('resources/translations'), 'connect');
 
-        $this->loadTranslationsFrom(__DIR__.'/../resources/translations', 'connect');
+        $this->loadViewsFrom($this->path('resources/views/connect'), 'connect');
 
-        // Views
+        if ($this->app->runningInConsole()) {
-
+            $this->commands([
-        $this->loadViewsFrom(__DIR__.'/../resources/views/connect', 'connect');
+                RefreshTokens::class,
-
+                Sync::class,
-        $this->publishes([
+            ]);
-            __DIR__.'/../resources/views/connect' => resource_path('views/vendor/connect'),
+        }
-        ]);
 
+        // Laravel 7+
         if (method_exists($this, 'loadViewComponentsAs')) {
-            // Laravel 7+
             $this->loadViewComponentsAs('connect', [
                 \Bluesquare\Connect\View\Components\Button::class
             ]);
         }
+    }
 
-        // Commands
+    // Misc
 
-        if ($this->app->runningInConsole()) {
+    private function path($path = '')
-            $this->commands([
+    {
-                Sync::class
+        return __DIR__ . "/../$path";
-            ]);
-        }
     }
 }

src/Controllers/ConnectController.php

@@ -20,6 +20,19 @@ class ConnectController extends Controller
 
     public function webhook(Request $request, Connect $connect)
     {
-        return $connect->handleWebhook($request);
+        $hash = sha1(config('bconnect.client_secret') . date('Y-m-d'));
+
+        if ($request->header('x-connect-hash') !== $hash)
+            abort(403);
+
+        $data = $request->validate([
+            'event_type' => 'required|in:create,update,delete',
+            'connect_data' => 'required|array',
+            'connect_data.*' => 'nullable',
+            'connect_data.id' => 'required',
+            'connect_data.email' => 'required_if:event_type,create|required_if:event_type,update',
+        ]);
+
+        $connect->sync($data['event_type'], $data['connect_data']);
     }
 }

src/Traits/HasConnectData.php

@@ -0,0 +1,47 @@
+<?php
+
+namespace Bluesquare\Connect\Traits;
+use Illuminate\Support\Facades\Log;
+
+trait HasConnectData
+{
+    public function getConnectIdentifier()
+    {
+        return $this->connectIdentifier ?? 'connect_id';
+    }
+
+    public function fillConnectData(array $data)
+    {
+        $touched = [];
+
+        $fillable = $this->connectFillable ?? [];
+
+        foreach ($fillable as $origin => $targets) {
+            $value = is_string($origin) ? $data[$origin] : $data[$targets];
+            $targets = is_string($origin) && is_array($targets) ? $targets : [$targets];
+
+            foreach ($targets as $target) {
+                $parts = explode('|', $target);
+                $target = $parts[0];
+                $currentValue = $value ?? ($parts[1] ?? null);
+                $target_model = $this;
+                $parts = explode('.', $target);
+
+                foreach ($parts as $i => $property) {
+                    if ($i < count($parts) - 1) {
+                        $target_model = $target_model->$property;
+                        continue;
+                    }
+
+                    if ($target_model !== $this)
+                        $touched[] = $target_model;
+
+                    $target_model->$property = $currentValue;
+                }
+            }
+        }
+
+        foreach ($touched as $model)
+            $model->save();
+    }
+}

src/Traits/HasConnectSync.php

@@ -1,43 +0,0 @@
-<?php
-
-namespace Bluesquare\Connect\Traits;
-
-trait HasConnectSync
-{
-    abstract function fill(array $attributes);
-    abstract function save();
-    abstract function delete();
-
-    public static $connectResource;
-
-    public static $connectColumnId = 'connect_resource_id';
-
-    public static function findConnectResource($id)
-    {
-        return self::query()->where(self::$connectColumnId, $id)->first();
-    }
-
-    public static function onConnectResourceCreated($id, $data)
-    {
-        $record = self::findConnectResource($id) ?? new self;
-        $record->fill($data); // TODO
-        $record->{self::$connectColumnId} = $id;
-        return $record->save();
-    }
-
-    public static function onConnectResourceUpdated($id, $data)
-    {
-        return self::onConnectResourceCreated($id, $data);
-    }
-
-    public static function onConnectResourceDeleted($id, $data = null)
-    {
-        $record = self::findConnectResource($id);
-        return $record ? $record->forceDelete() : false;
-    }
-
-    public static function onConnectResourceDoesNotExist($record)
-    {
-        return $record->forceDelete();
-    }
-}

src/Traits/HasConnectTokens.php

@@ -4,5 +4,10 @@ namespace Bluesquare\Connect\Traits;
 
 trait HasConnectTokens
 {
-    //
+    public function fillConnectTokens(array $data)
+    {
+        $this->connect_access_token = $data['access_token'];
+        $this->connect_refresh_token = $data['refresh_token'];
+        $this->connect_expires_at = $data['expires_at'];
+    }
 }

src/Traits/HasConnectWebhook.php

@@ -0,0 +1,41 @@
+<?php
+
+namespace Bluesquare\Connect\Traits;
+
+trait HasConnectWebhook
+{
+    public function onConnectCreate(array $data)
+    {
+        $this->onConnectUpdate($data);
+    }
+
+    public function onConnectUpdate(array $data)
+    {
+        if (in_array(HasConnectData::class, class_uses(self::class))) {
+            $this->fillConnectData($data);
+        } else {
+            $this->email = $data['email'];
+        }
+
+        $this->save();
+    }
+
+    public function onConnectDelete(array $data)
+    {
+        $this->onConnectUpdate($data);
+
+        if (in_array(\Illuminate\Database\Eloquent\SoftDeletes::class, class_uses(self::class))) {
+            $this->delete();
+        } elseif (array_key_exists('remember_token', $this->attributes)) {
+            $this->remember_token = null;
+            $this->save();
+        }
+    }
+
+    public function onConnectRestore(array $data)
+    {
+        $this->restore();
+
+        $this->onConnectUpdate($data);
+    }
+}