Support for auditSync and auditDetach on Auditable custom Pivot class

src/Auditable.php

@@ -5,6 +5,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\Relations\MorphMany;
+use Illuminate\Database\Eloquent\Relations\Pivot;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Collection;
@@ -13,6 +14,7 @@
 use Illuminate\Support\Facades\Event;
 use OwenIt\Auditing\Contracts\AttributeEncoder;
 use OwenIt\Auditing\Contracts\AttributeRedactor;
+use OwenIt\Auditing\Contracts\Auditable as ContractsAuditable;
 use OwenIt\Auditing\Contracts\Resolver;
 use OwenIt\Auditing\Events\AuditCustom;
 use OwenIt\Auditing\Exceptions\AuditableTransitionException;
@@ -746,7 +748,16 @@ public function auditDetach(string $relationName, $ids = null, $touch = true, $c
         }
 
         $old = $relationCall->get($columns);
-        $results = $relationCall->detach($ids, $touch);
+
+        $pivotClass = $relationCall->getPivotClass();
+
+        if($pivotClass !== Pivot::class && is_a($pivotClass, ContractsAuditable::class, true)) {
+            $results = $pivotClass::withoutAuditing(function () use ($relationCall, $ids, $touch) {
+                return $relationCall->detach($ids, $touch);
+            });
+        } else {
+            $results = $relationCall->detach($ids, $touch);
+        }
         $new = $relationCall->get($columns);
 
         $this->dispatchRelationAuditEvent($relationName, 'detach', $old, $new);
@@ -772,9 +783,18 @@ public function auditSync(string $relationName, $ids, $detaching = true, $column
         if ($callback instanceof \Closure) {
             $this->applyClosureToRelationship($relationCall, $callback);
         }
-
+        
         $old = $relationCall->get($columns);
-        $changes = $relationCall->sync($ids, $detaching);
+
+        $pivotClass = $relationCall->getPivotClass();
+
+        if($pivotClass !== Pivot::class && is_a($pivotClass, ContractsAuditable::class, true)) {
+            $changes = $pivotClass::withoutAuditing(function () use ($relationCall, $ids, $detaching) {
+                return $relationCall->sync($ids, $detaching);
+            });
+        } else {
+            $changes = $relationCall->sync($ids, $detaching);
+        }
 
         if (collect($changes)->flatten()->isEmpty()) {
             $old = $new = collect([]);

tests/Functional/AuditingTest.php

@@ -20,6 +20,7 @@
 use OwenIt\Auditing\Tests\Models\ArticleCustomAuditMorph;
 use OwenIt\Auditing\Tests\Models\ArticleExcludes;
 use OwenIt\Auditing\Tests\Models\Category;
+use OwenIt\Auditing\Tests\Models\Group;
 use OwenIt\Auditing\Tests\Models\User;
 
 class AuditingTest extends AuditingTestCase
@@ -1092,4 +1093,72 @@ public function canAuditCustomAuditModelImplementation()
         $this->assertNotEmpty($audit);
         $this->assertSame(get_class($audit), \OwenIt\Auditing\Tests\Models\CustomAudit::class);
     }
+
+    /**
+     * @test
+     * @return void
+     */
+    public function itWillAuditSyncWithAuditablePivotClass()
+    {
+        $group = factory(Group::class)->create();
+        $user = factory(User::class)->create();
+
+        $no_of_audits_before = Audit::where('auditable_type', User::class)->count();
+
+        $user->auditSync('groups', [$group->getKey() => ["role" => "admin"]]);
+
+        $no_of_audits_mid = Audit::where('auditable_type', User::class)->count();
+        $memberRole = $user->groups()->first()->pivot->role;
+
+        $user->auditSync('groups', []);
+
+        $no_of_audits_after = Audit::where('auditable_type', User::class)->count();
+
+        $this->assertSame("admin", $memberRole);
+        $this->assertGreaterThan($no_of_audits_before, $no_of_audits_mid);
+        $this->assertGreaterThan($no_of_audits_mid, $no_of_audits_after);
+    }
+
+    /**
+     * @test
+     * @return void
+     */
+    public function itWillAuditAttachWithAuditablePivotClass()
+    {
+        $group = factory(Group::class)->create();
+        $user = factory(User::class)->create();
+
+        $no_of_audits_before = Audit::where('auditable_type', User::class)->count();
+
+        $user->auditAttach('groups', $group);
+
+        $attachedGroup = $user->groups()->first()->getKey();
+        $no_of_audits_after = Audit::where('auditable_type', User::class)->count();
+
+        $this->assertSame($group->getKey(), $attachedGroup);
+        $this->assertGreaterThan($no_of_audits_before, $no_of_audits_after);
+    }
+
+    /**
+     * @test
+     * @return void
+     */
+    public function itWillAuditDetachWithAuditablePivotClass()
+    {
+        $group = factory(Group::class)->create();
+        $user = factory(User::class)->create();
+
+        $user->groups()->attach($group);
+
+        $attachedGroup = $user->groups()->first()->getKey();
+        $no_of_audits_before = Audit::where('auditable_type', User::class)->count();
+
+        $detachedGroups = $user->auditDetach('groups', $group);
+
+        $no_of_audits_after = Audit::where('auditable_type', User::class)->count();
+
+        $this->assertSame($group->getKey(), $attachedGroup);
+        $this->assertSame(1, $detachedGroups);
+        $this->assertGreaterThan($no_of_audits_before, $no_of_audits_after);
+    }
 }

tests/Models/Group.php

@@ -0,0 +1,7 @@
+<?php
+
+namespace OwenIt\Auditing\Tests\Models;
+
+class Group extends \Illuminate\Database\Eloquent\Model
+{
+}

tests/Models/GroupMember.php

@@ -0,0 +1,17 @@
+<?php
+
+namespace OwenIt\Auditing\Tests\Models;
+
+use Illuminate\Database\Eloquent\Relations\Concerns\AsPivot;
+use OwenIt\Auditing\Auditable;
+use OwenIt\Auditing\Contracts\Auditable as ContractsAuditable;
+
+class GroupMember extends \Illuminate\Database\Eloquent\Model implements ContractsAuditable
+{
+    use AsPivot;
+    use Auditable;
+
+    public $fillable = [
+        "role"
+    ];
+}

tests/Models/User.php

@@ -29,4 +29,9 @@ public function getFirstNameAttribute(string $value): string
     {
         return ucfirst($value);
     }
+
+    public function groups()
+    {
+        return $this->belongsToMany(Group::class, 'group_members', 'user_id', 'group_id')->using(GroupMember::class)->withPivot('id','role');
+    }
 }

tests/database/factories/GroupFactory.php

@@ -0,0 +1,16 @@
+<?php
+
+use Faker\Generator as Faker;
+
+/*
+|--------------------------------------------------------------------------
+| Article Factories
+|--------------------------------------------------------------------------
+|
+*/
+
+$factory->define(\OwenIt\Auditing\Tests\Models\Group::class, function (Faker $faker) {
+    return [
+        'name' => $faker->unique()->colorName(),
+    ];
+});

tests/database/migrations/0000_00_00_000004_create_groups_test_table.php

@@ -0,0 +1,41 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+class CreateGroupsTestTable extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::create('groups', function (Blueprint $table) {
+            $table->increments('id');
+            $table->string('name');
+            $table->timestamps();
+        });
+
+        Schema::create('group_members', function (Blueprint $table) {
+            $table->increments('id');
+            $table->unsignedBigInteger('user_id');
+            $table->unsignedBigInteger('group_id');
+            $table->enum('role',['member','admin'])->default('member');
+            $table->timestamps();
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::drop('group_members');
+        Schema::drop('groups');
+    }
+}