Delete segment

resources/views/segments/index.blade.php

@@ -35,8 +35,9 @@
                             </a>
                         </td>
                         <td>{{ $segment->subscribers_count }}</td>
-                        <td><a class="btn btn-sm btn-light"
-                               href="{{ route('sendportal.segments.edit', $segment->id) }}">{{ __('Edit') }}</a></td>
+                        <td>
+                            @include('sendportal::segments.partials.actions')
+                        </td>
                     </tr>
                 @empty
                     <tr>

resources/views/segments/partials/actions.blade.php

@@ -0,0 +1,7 @@
+<form action="{{ route('sendportal.segments.destroy', $segment->id) }}" method="POST">
+    @csrf
+    @method('DELETE')
+    <a href="{{ route('sendportal.segments.edit', $segment->id) }}"
+       class="btn btn-sm btn-light">{{ __('Edit') }}</a>
+    <button type="submit" class="btn btn-sm btn-light">{{ __('Delete') }}</button>
+</form>

routes/web.php

@@ -82,7 +82,7 @@
         });
 
         // Segments.
-        $appRouter->resource('segments', 'Segments\SegmentsController')->except(['show', 'destroy']);
+        $appRouter->resource('segments', 'Segments\SegmentsController')->except(['show']);
 
         // Workspace User Management.
         $appRouter->namespace('Workspaces')

src/Http/Controllers/Segments/SegmentsController.php

@@ -66,4 +66,14 @@ public function update(int $id, SegmentRequest $request): RedirectResponse
 
         return redirect()->route('sendportal.segments.index');
     }
+
+    /**
+     * @throws Exception
+     */
+    public function destroy(int $id): RedirectResponse
+    {
+        $this->segmentRepository->destroy(auth()->user()->currentWorkspace()->id, $id);
+
+        return redirect()->route('sendportal.segments.index');
+    }
 }

src/Models/Segment.php

@@ -19,6 +19,11 @@ class Segment extends BaseModel
         'subscribers'
     ];
 
+    public function campaigns(): BelongsToMany
+    {
+        return $this->belongsToMany(Campaign::class);
+    }
+
     /**
      * The workspace this segment belongs to.
      */

src/Repositories/SegmentTenantRepository.php

@@ -37,4 +37,17 @@ public function syncSubscribers(Segment $segment, array $subscribers = [])
     {
         return $segment->subscribers()->sync($subscribers);
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function destroy($workspaceId, $id): bool
+    {
+        $instance = $this->find($workspaceId, $id);
+
+        $instance->subscribers()->detach();
+        $instance->campaigns()->detach();
+
+        return $instance->delete();
+    }
 }

tests/Feature/Segments/SegmentsControllerTest.php

@@ -101,5 +101,19 @@ function a_segment_is_updateable_by_an_authenticated_user()
         ]);
     }
 
+    /* @test */
+    public function a_segment_can_be_deleted()
+    {
+        [$workspace, $user] = $this->createUserAndWorkspace();
+
+        $segment = factory(Segment::class)->create(['workspace_id' => $workspace->id]);
+
+        $response = $this->actingAs($user)
+            ->delete(route('sendportal.segments.destroy', $segment->id));
 
+        $response->assertRedirect();
+        $this->assertDatabaseMissing('segments', [
+            'id' => $segment->id,
+        ]);
+    }
 }