Error in extending tables

I would like to extend from Tables for creating my own class (in filament 3)
I created an empty class:

namespace App\Tables\Components;
use Filament\Tables\Table;

class MyTable extends Table{

}

namespace App\Tables\Components;
use Filament\Tables\Table;

class MyTable extends Table{

}

but if I try to use MyTable in this way in a resource file:

public static function table(MyTable $table): MyTable

public static function table(MyTable $table): MyTable

I have this error:

Declaration of App\Filament\Resources\ExhibitionResource::table(App\Tables\Components\MyTable $table): App\Tables\Components\MyTable must be compatible with Filament\Resources\Resource::table(Filament\Tables\Table $table):

Declaration of App\Filament\Resources\ExhibitionResource::table(App\Tables\Components\MyTable $table): App\Tables\Components\MyTable must be compatible with Filament\Resources\Resource::table(Filament\Tables\Table $table):

And I cannot understand why.

Solution

The

abstract

abstract

Resource names the specific

Table

Table

class which must be implemented in that particular scenario, instead of specifying a contract. https://github.com/filamentphp/filament/blob/aa609c8f1214e6c49f1625d006a4d6739bcc2e5c/packages/panels/src/Resources/Resource.php#L151-L154

GitHub

filament/packages/panels/src/Resources/Resource.php at aa609c8f1214...

A collection of beautiful full-stack components for Laravel. The perfect starting point for your next app. Using Livewire, Alpine.js and Tailwind CSS. - filamentphp/filament

Jump to solution

krekas•10/7/23, 4:23 PM

why do you want to extend it?

Zziolupo I would like to extend from Tables for creating my own class (in filament 3) I c...

Filament•10/7/23, 4:50 PM

Please ask about the actual problem you're trying to solve, instead of your attempted solution. https://xyproblem.info

Kkrekas why do you want to extend it?

ziolupoOP•10/8/23, 7:27 AM

It just an experiment to see how flexible can be using FIlament.

krekas•10/8/23, 7:29 AM

experiment of what? what you try to achieve?

Solution

Zziolupo I would like to extend from Tables for creating my own class (in filament 3) I c...

DrByte•10/8/23, 9:52 PM

The

abstract

abstract

Resource names the specific

Table

Table

GitHub

filament/packages/panels/src/Resources/Resource.php at aa609c8f1214...

A collection of beautiful full-stack components for Laravel. The perfect starting point for your next app. Using Livewire, Alpine.js and Tailwind CSS. - filamentphp/filament

DrByte•10/8/23, 9:52 PM

But it still begs the question: what would you want to do in your extended version of that particular class?

ziolupoOP•10/9/23, 6:13 AM

Honestly speaking I don't have anything that I want to do with the new class...b ut I can easily find.
For example a class "compact", or a class where I can dynamically change the padding.
But the idea was just having a new class for playing.

DrByte•10/10/23, 1:56 AM

Understandable. Unfortunately in this specific case, that's not the right approach.

DrByte•10/10/23, 1:57 AM

Instead, Filament lets you set global configurations on many of its components, so that's one thing you can explore.

DrByte•10/10/23, 1:59 AM

Another thing could be to set up some Traits that you add to the various classes if you want to add the same functionality to several classes with one line of code.
But I've not run into a need for that, at least not when it comes to major components like Resources and Tables. (I do use traits for reusable Form fields, since some of those are used in multiple resources and the consistency is super handy.)

DrByte•10/10/23, 2:00 AM

If it's padding you're interested in controlling, you have a lot of control in the CSS, whether by targeting individual named elements with css selectors in your own custom css, or by recompiling the css overall.

DDrByte If it's padding you're interested in controlling, you have a lot of control in t...

ziolupoOP•10/10/23, 5:46 AM

Thanks DrByte. I investigate further "under the hood" and it's exactly as you are saying. About the result, I already achieved that with CSS or by changing the original code (I know that is something not to do...

... but as I was saying from the beginning, I was more interested in the possibility, without a particular ussue or problem to be solved.
Thanks again