EC-CUBE / EC-CUBE/ec-cube

OrderRepositoryのgetQueryBuilderBySearchDataForAdminにおける受注IDについて

Open
#5,973 1 comment 0 reactions 0 assignees View on GitHub
Status: needs-review
Dominant language
PHP
Stars
788
Forks
719
Avg merge
4d 4h
Merged PRs (30d)
39

Description

### 概要(Overview)

OrderRepositoryのgetQueryBuilderBySearchDataForAdmin関数内で$searchDataの以下のキーによる分岐が存在します。
- order_id
- order_id_start
- order_id_end

ですが、現状$searchData内にそれぞれのキーが渡ることは無いように見受けられます。
もし、不要であればこの部分は削除したほうがいいかと思いました。

#### 該当コード

https://github.com/EC-CUBE/ec-cube/blob/4.2/src/Eccube/Repository/OrderRepository.php#L131
https://github.com/EC-CUBE/ec-cube/blob/4.2/src/Eccube/Repository/OrderRepository.php#L145
https://github.com/EC-CUBE/ec-cube/blob/4.2/src/Eccube/Repository/OrderRepository.php#L168

### 期待する内容(Expect) or 要望(Requirement)

OrderRepositoryのgetQueryBuilderBySearchDataForAdmin関数の以下のキーによる分岐を削除
- order_id
- order_id_start
- order_id_end

また、私の勘違いであればどの部分で該当項目による検索が行われているか教えていただきたいです。

よろしくお願いします。

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.