Introduce PHP endpoints for product filters

[Quetzacoalt91]

Todo

• Implement condition Does not contain
• Implement product selection from filters with Features

Retrieve possible options of product filters

• {"action":"getProductFilterOptions", "kind": "category"}

Response

[
    {
        "id": "1",
        "value": "Root"
    },
    {
        "id": "2",
        "value": "Catalogue"
    },
    {
        "id": "845",
        "value": "Books"
    }
]

• {"action":"getProductFilterOptions", "kind": "brand"}

Response

[
    {
        "id": "1",
        "value": "International Company"
    }
]

• {"action":"getProductFilterOptions", "kind": "attribute"}

Response

[
    [
        "id": "2",
        "key": "Form",
        "values": [
            ["id": "4", "key": "Form", "value": "Classique", "language": "fr"],
            ["id": "4", "key": "Form", "value": "Classic", "language": "en_gb"],
            ["id": "4", "key": "Form", "value": "Chromatique", "language": "fr"],
            ["id": "4", "key": "Form", "value": "Shiny", "language": "en_gb"],
        ],
    ],
    [
        "id": "1",
        "key": "Gender",
        "values": [
            ["id": "3", "key": "Gender", "value": "Femmes", "language": "fr"],
            ["id": "3", "key": "Gender", "value": "Women", "language": "en_gb"],
            ["id": "3", "key": "Gender", "value": "Hommes", "language": "fr"],
            ["id": "3", "key": "Gender", "value": "Men", "language": "en_gb"],
        ]
    ]
]

• {"action":"getProductFilterOptions", "kind": "feature"}

Response

[
    {
        "id": "25",
        "key": "Bio",
        "values": [
            {
                "id": "3776",
                "key": "Bio",
                "value": "Non",
                "language": "fr"
            },
            {
                "id": "3776",
                "key": "Bio",
                "value": "Nope",
                "language": "gb"
            },
            {
                "id": "3780",
                "key": "Bio",
                "value": "Oui",
                "language": "fr"
            },
            {
                "id": "3780",
                "key": "Bio",
                "value": "Yep",
                "language": "gb"
            }
        ]
    }
]

Count products matching provided filters

Request

{
    "action":"countMatchingProductsFromFilters",
    "filters": [
        {
            "attribute": "id",
            "condition": "contains",
            "value": "79"
        },
        {
            "attribute": "id",
            "condition": "lower",
            "value": "7938"
        },
        {
            "attribute": "price",
            "condition": "greater",
            "value": "0"
        }
    ]
}

Response

{
    "numberOfProducts": 1
}

List products matching provided filters

Request

{
    "action":"listMatchingProductsFromFilters",
    "filters": [
        {
            "attribute": "id",
            "condition": "contains",
            "value": "79"
        },
        {
            "attribute": "id",
            "condition": "lower",
            "value": "7938"
        },
        {
            "attribute": "price",
            "condition": "greater",
            "value": "0"
        }
    ]
}

Response

[
    {
        "id_product": "7937",
        // ...
    },
    {
        // ...
    }
]

[guillaumeArgiles]

Maybe we can call it ManufacturerOptionsProvider ?

[Quetzacoalt91]

That's what I originally thought. But as we're dealing with Brands on the product filter UI, I wanted to put this name inside the Product Filter namespace

[Quetzacoalt91]

Double checking on the back office shows "Brands" on the interface, only Manufacturer is used on the database, probably for backward compatibility.

[tberger-prestashop]

Grâce à mes grandes connaissances PHP & Core, je te donne une bonne étoile pour ce merveilleux travail:
🌟

Toutes mes félicitations 🎉

[kktos]

    $isSingleValue = isset($filter['value']);

    if($isSingleValue) {
        $checkValueIsNumber($filter['value']);
    } else {
        foreach ($filter['values'] as $value) {
            $checkValueIsNumber($value);
        }    
    }

[Quetzacoalt91]

The current version seems more appropriate to me. Before handling each key, we should check it exists first.
The else would make the code less readable as we don't understand easily that we can rely on values if value is not set.