Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

管理画面:ソート機能追加(商品一覧・受注一覧・会員一覧) #5038

Merged
merged 10 commits into from Jan 27, 2022
Merged

Conversation

ghost
Copy link

@ghost ghost commented May 11, 2021

概要(Overview・Refs Issue)

Issue番号:#4627
上記issueと関連している

状況が分からないので一旦プルリクエストあげます。

方針(Policy)

・既存のソースに影響しないよう実装
・Repositoryに、CONST追加:[TABLEカラム名 => select名]
・項目追加やソート追加をし易いように実装

実装に関する補足(Appendix)

paginatorの第3引数:array('wrap-queries' => true)
CustomerControllerは現状の項目では、不要でしたので追加していません

テスト(Test)

受注一覧

商品一覧

会員一覧

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

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

レビュワー確認項目

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

@tao-s
Copy link
Contributor

tao-s commented May 28, 2021

受注一覧、商品一覧のソートは欲しかった

@chihiro-adachi chihiro-adachi added this to the 4.1 milestone Aug 26, 2021
@chihiro-adachi chihiro-adachi added the enhancement 機能追加 label Aug 26, 2021
@chihiro-adachi chihiro-adachi modified the milestones: 4.1, 4.1.x Sep 3, 2021
@matsuoshi matsuoshi changed the base branch from 4.0 to 4.1 November 15, 2021 06:27
@matsuoshi matsuoshi modified the milestones: 4.1.x, 4.1.1 Nov 15, 2021
@nanasess
Copy link
Contributor

#5030 出荷IDの対応もあると良さそうです

@nanasess
Copy link
Contributor

@matsuoshi
充実!!商品ソート(4.0対応版) への影響確認が必要かも

@matsuoshi matsuoshi added the affected:plugin プラグインに影響する可能性のある実装変更 label Nov 22, 2021
@kiy0taka kiy0taka removed this from the 4.1.1 milestone Dec 1, 2021
@chihiro-adachi chihiro-adachi added this to the 4.1.x milestone Dec 1, 2021
@matsuoshi matsuoshi modified the milestones: 4.1.x, 4.1.2 Jan 14, 2022
@matsuoshi
Copy link
Contributor

@refine-kenshi ありがとうございます。
こちら、 @tsumu-tsumu よりデザインアイディアをいただいておりまして、この形でフロントの実装に変更を入れさせてもらおうかと考えております

スクリーンショット 2022-01-21 12 44 23

  • 黒の矢印: 現在のソート状況を表す
  • グレーの矢印: ソートボタン
  • 丸い白抜きの矢印: マウスホバー時の表示

@ghost
Copy link
Author

ghost commented Jan 21, 2022

@matsuoshi 確認しました。
@tsumu-tsumu デザインでOKです。

@matsuoshi
Copy link
Contributor

@refine-kenshi 確認ありがとうございます、ではフロントの実装のほう、こちらで変更加えさせていただきます

@codecov-commenter
Copy link

codecov-commenter commented Jan 24, 2022

Codecov Report

Attention: Patch coverage is 42.02899% with 40 lines in your changes missing coverage. Please review.

Project coverage is 68.24%. Comparing base (0782e30) to head (2e5a796).
Report is 288 commits behind head on 4.1.

Files with missing lines Patch % Lines
.../Eccube/Controller/Admin/Order/OrderController.php 33.33% 8 Missing ⚠️
...ube/Controller/Admin/Product/ProductController.php 33.33% 8 Missing ⚠️
src/Eccube/Form/Type/Admin/SearchCustomerType.php 50.00% 4 Missing ⚠️
src/Eccube/Form/Type/Admin/SearchOrderType.php 50.00% 4 Missing ⚠️
src/Eccube/Form/Type/Admin/SearchProductType.php 50.00% 4 Missing ⚠️
src/Eccube/Repository/CustomerRepository.php 42.85% 4 Missing ⚠️
src/Eccube/Repository/OrderRepository.php 42.85% 4 Missing ⚠️
src/Eccube/Repository/ProductRepository.php 42.85% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                4.1    #5038      +/-   ##
============================================
- Coverage     68.45%   68.24%   -0.21%     
- Complexity     6126     6143      +17     
============================================
  Files           461      461              
  Lines         25168    25237      +69     
============================================
- Hits          17228    17223       -5     
- Misses         7940     8014      +74     
Flag Coverage Δ
E2E 56.47% <42.02%> (-0.27%) ⬇️
Unit 76.12% <64.44%> (-0.09%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@matsuoshi matsuoshi self-assigned this Jan 24, 2022
@matsuoshi
Copy link
Contributor

フロントのコード (twig, js, scss) を修正し、デザインを反映しました

@chihiro-adachi chihiro-adachi added the affected:admin_template 管理画面テンプレートのDOMに影響のある変更 label Jan 25, 2022
@chihiro-adachi
Copy link
Contributor

@matsuoshi
少し修正いれたのでみていただいてもよいですか?
ソート順が一意になるように第二ソート追加しています。
その他コメント追加、cs fixかけています。

@xuelian311 xuelian311 assigned matsuoshi and unassigned matsuoshi Jan 27, 2022
@chihiro-adachi chihiro-adachi merged commit 0e60516 into EC-CUBE:4.1 Jan 27, 2022
@chihiro-adachi
Copy link
Contributor

@refine-kenshi
ありがとうございます、マージしました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:admin_template 管理画面テンプレートのDOMに影響のある変更 affected:plugin プラグインに影響する可能性のある実装変更 enhancement 機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants