WhereClauseでfalseとして評価されるパラメータを受け取れるよう変更

[izayoi256]

概要(Overview・Refs Issue)

#4547

実装に関する補足(Appendix)

$yのチェックをnullチェックにしようかと思いましたが、そもそもチェックする意図も不明だったため、チェック自体を削除しました。
何か不都合があれば教えて下さい。

テスト（Test)

'0'やnullを渡した場合のテストを追加。

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

動作確認して問題ありませんでした。

$yのチェックをnullチェックにしようかと思いましたが、そもそもチェックする意図も不明だったため、チェック自体を削除しました。何か不都合があれば教えて下さい。

こちら考えてみましたが、チェックが必要なケースは思いつきませんでした。
私もチェックなしで問題ないと思います。

確認したコード

<?php

namespace Customize\Repository;

use Eccube\Doctrine\Query\WhereClause;
use Eccube\Doctrine\Query\WhereCustomizer;
use Eccube\Repository\QueryKey;

class AdminProductListCustomizer extends WhereCustomizer
{
    public function getQueryKey()
    {
        return QueryKey::PRODUCT_SEARCH_ADMIN;
    }

    protected function createStatements($params, $queryKey)
    {
        return [WhereClause::neq('p.id', ':customizeid', 0)];
    }
}

[okazy]

変更内容問題ありません。
落ちているテストも他でIssueが立っているものですので問題ありません。

[okazy]

ありがとうございます！取り込みました。