受注のマルチ検索で、会社名はスペースを除去せず検索 #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
okazy
force-pushed
the
feature/#4716_fix_adminpage_order_search
branch
from
1833275 to
a8592c4
Compare
11 tasks
|
テストも足しておきたいので少々お待ちください。 |
|
お待たせしました！OrderRepositoryの名前の検索に関するテストを追加しました。 |
|
なるほど会社名はスペースいれられるんですね。仕様を考慮できていませんでした。 アプルーブします! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要(Overview・Refs Issue)
EC-CUBE#4766 のテストが落ちていた件について
会社名はスペースの入力ができます。
例えば「株式会社 ほげほげ」という会社名だった場合に、前処理でスペースが除去され、「株式会社ほげほげ」で検索されてしまっていたため、検索結果がヒットしなくなっていました。
受注のマルチ検索で、会社名はスペースを除去せず検索できるようにしました。
方針(Policy)
テストを正としてプロダクトコードを修正しています。
実装に関する補足(Appendix)
テスト（Test)
落ちていたテスト
Eccube\Tests\Web\Admin\Order\OrderControllerTest::testSearchOrderByNameが通ることを確認