Merge pull request #57 from devaslanphp/dev

Bug-fix: Ticket number button
devaslanphp · Sep 19, 2022 · e83f4e4 · e83f4e4
2 parents fe8718b + e7fa24e
commit e83f4e4
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/app/Http/Livewire/TicketDetails.php b/app/Http/Livewire/TicketDetails.php
@@ -52,10 +52,12 @@ public function ticketSaved(): void
     /**
      * Copy a ticket url
      *
-     * @param Ticket $ticket
+     * @param int $ticketId
      * @return void
      */
-    public function copyTicketUrl(Ticket $ticket): void {
+    public function copyTicketUrl(int $ticketId): void
+    {
+        $ticket = Ticket::where('id', $ticketId)->first();
         Notification::make()
             ->success()
             ->title(__('Ticket url copied'))

diff --git a/app/Http/Livewire/Tickets.php b/app/Http/Livewire/Tickets.php
@@ -2,19 +2,15 @@
 
 namespace App\Http\Livewire;
 
-use App\Models\FavoriteProject;
 use App\Models\Project;
 use App\Models\Ticket;
 use App\Models\User;
 use Filament\Forms\Components\Grid;
 use Filament\Forms\Components\MultiSelect;
-use Filament\Forms\Components\Select;
 use Filament\Forms\Components\TextInput;
 use Filament\Forms\Concerns\InteractsWithForms;
 use Filament\Forms\Contracts\HasForms;
-use Filament\Notifications\Actions\Action;
 use Filament\Notifications\Notification;
-use Illuminate\Support\Str;
 use Livewire\Component;
 
 class Tickets extends Component implements HasForms
@@ -237,10 +233,12 @@ public function ticketDeleted()
     /**
      * Copy a ticket url
      *
-     * @param Ticket $ticket
+     * @param int $ticketId
      * @return void
      */
-    public function copyTicketUrl(Ticket $ticket): void {
+    public function copyTicketUrl(int $ticketId): void
+    {
+        $ticket = Ticket::where('id', $ticketId)->first();
         Notification::make()
             ->success()
             ->title(__('Ticket url copied'))

diff --git a/resources/views/livewire/ticket-details.blade.php b/resources/views/livewire/ticket-details.blade.php
@@ -10,7 +10,7 @@
                     <i class="fa fa-tag"></i>
                     {{ $ticket->project->name }}
                     <span class="text-xs text-gray-300">/</span>
-                    <button type="button" class="font-normal hover:cursor-pointer hover:underline" wire:click="copyTicketUrl({{ $ticket }})" title="@lang('Click to copy url to ticket')">{{ $ticket->ticket_number }}</button>
+                    <button type="button" class="font-normal hover:cursor-pointer hover:underline" wire:click="copyTicketUrl('{{ $ticket->id }}')" title="@lang('Click to copy url to ticket')">{{ $ticket->ticket_number }}</button>
                 </span>
                 <div class="w-full">
                     @livewire('ticket-details.title', ['ticket' => $ticket])

diff --git a/resources/views/livewire/tickets.blade.php b/resources/views/livewire/tickets.blade.php
@@ -69,7 +69,7 @@
                             <i class="fa fa-tag"></i>
                             {{ $ticket->project->name }}
                             <span class="text-xs text-gray-300">/</span>
-                            <button type="button" class="font-normal hover:cursor-pointer hover:underline" wire:click="copyTicketUrl({{ $ticket }})" title="@lang('Click to copy url to ticket')">{{ $ticket->ticket_number }}</button>
+                            <button type="button" class="font-normal hover:cursor-pointer hover:underline" wire:click="copyTicketUrl('{{ $ticket->id }}')" title="@lang('Click to copy url to ticket')">{{ $ticket->ticket_number }}</button>
                         </span>
                         <div class="w-full flex flex-row justify-between items-start gap-5">
                             <a href="{{ route('tickets.details', ['ticket' => $ticket, 'slug' => Str::slug($ticket->title)]) }}" class="text-lg font-medium text-gray-700 hover:underline">