Introduce AsFilter attribute to define filter type and form type

[vasilvestre]

This PR introduces a new attribute, AsFilter, to provide a more concise and structured way of defining filter types and form types for grid filters.

Previously, filter types and form types were defined two methods inherited from ConfigurableFilterInterface.

By using the AsFilter attribute, developers can now explicitly declare the type and form type of a filter directly above the class definition. This approach improves code readability, reduces reliance on tag attributes, and enhances maintainability.

Steps:

• Create the attribute and implement it's logic
• Test
• Documentation

Example usage:

use Sylius\Component\Grid\Filtering\FilterInterface;
use Sylius\Component\Grid\Metadata\AsFilter;

#[AsFilter(type: 'product_name', formType: ProductNameFilterType::class)]
final class ProductNameFilter implements FilterInterface
{
    public function apply(DataSourceInterface $dataSource, string $name, $data, array $options): void
    {
        if (empty($data['value'])) {
            return;
        }

        $queryBuilder = $dataSource->getQueryBuilder();
        $rootAlias = $queryBuilder->getRootAliases()[0];

        $queryBuilder
            ->andWhere(sprintf('%s.name LIKE :name', $rootAlias))
            ->setParameter('name', '%' . $data['value'] . '%');
    }
}

The RegisterFiltersPass compiler pass has been updated to check for the AsFilter attribute and ignore duplicate filter configuration.

Tests are provided in the PR, documentation too.

A new behaviour is that type is now optional as the FQCN is now used by default. Which means you can reference it directly in PHP or YAML .

Future considerations:

In a future PR, we can consider deprecating the ConfigurableFilterInterface as it becomes redundant by introducing the AsFilter attribute. This will further simplify the process of creating grid filters and improve consistency.

[loic425]

@vasilvestre you have two commits with "-" as commit message
https://github.com/Sylius/SyliusGridBundle/pull/348/commits

[lchrusciel]

it seems to be incorrect, or at least misleading

[vasilvestre]

Why ? We can use FQCN as name and also call the create method of the filter.

[lchrusciel]

How is it tested? Does it override nationality filter and is covered by this test:

SyliusGridBundle/src/Bundle/Tests/Functional/GridUiTest.php

Lines 235 to 245 in f500a89

	public function it_includes_all_rows_even_when_sorting_by_a_nullable_path(): void
	{
	$this->client->request('GET', '/authors/');
	$totalItemsCountBeforeSorting = count($this->getAuthorNamesFromResponse());
	$this->client->request('GET', '/authors/?sorting[nationality]=desc');
	$totalItemsCountAfterSorting = count($this->getAuthorNamesFromResponse());
	$this->assertSame($totalItemsCountBeforeSorting, $totalItemsCountAfterSorting);
	}

or is not covered by test and we don't know if it works?

[vasilvestre]

I think it's covered in https://github.com/Sylius/SyliusGridBundle/blob/f500a89f5ea35fd675012c30e55d0def0a751cc3/tests/Application/src/Filter/AttributeNationalityFilter.php and https://github.com/Sylius/SyliusGridBundle/blob/f500a89f5ea35fd675012c30e55d0def0a751cc3/tests/Application/src/Grid/Builder/AttributeNationalityFilter.php