受注編集の画面で、金額にエラーがあると、課税区分のカッコ内が表示されなくなる 不具合の修正 #5761
Conversation
chihiro-adachi
changed the title
|
@maianhtu010596 |
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## 4.2 #5761 +/- ##
============================================
+ Coverage 82.54% 82.55% +0.01%
- Complexity 6422 6425 +3
============================================
Files 475 475
Lines 25849 25855 +6
============================================
+ Hits 21337 21345 +8
+ Misses 4512 4510 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
@maianhtu010596 |
|
@chihiro-adachi @kiy0taka |
| @@ -221,6 +221,12 @@ public function index(Request $request, RouterInterface $router, $id = null) | |||
|
|
|||
| $form->handleRequest($request); | |||
| $purchaseContext = new PurchaseContext($OriginOrder, $OriginOrder->getCustomer()); | |||
|
|
|||
| foreach ($TargetOrder->getOrderItems() as $orderItem) { | |||
| if($orderItem->getTaxDisplayType() == null){ | |||
There was a problem hiding this comment.
@shinya
if($orderItem->getTaxDisplayType() == null){
↓
if ($orderItem->getTaxDisplayType() == null) {
| ) { | ||
| $this->entityManager = $entityManager; | ||
| $this->taxRuleRepository = $taxRuleRepository; | ||
| $this->taxRuleService = $taxRuleService; | ||
| $this->orderHelper = $orderHelper; |
There was a problem hiding this comment.
@shinya
phpstanの指摘修正お願いします。
あと空白ずれてる？
| @@ -77,7 +80,7 @@ public function process(ItemHolderInterface $itemHolder, PurchaseContext $contex | |||
| $item->setTaxType($this->getTaxType($OrderItemType)); | |||
| } | |||
| if (!$item->getTaxDisplayType()) { | |||
| $item->setTaxDisplayType($this->getTaxDisplayType($OrderItemType)); | |||
| $item->setTaxDisplayType($this->orderHelper->getTaxDisplayType($OrderItemType)); | |||
There was a problem hiding this comment.
@shinya
phpstanの指摘修正お願いします。
There was a problem hiding this comment.
メンバ変数の宣言が漏れている、という指摘であってますでしょうか？
そちら反映いたしました（認識違いあればおっしゃってください。）
| * | ||
| * @param $OrderItemType | ||
| * | ||
| * @return TaxType |
There was a problem hiding this comment.
TaxType -> TaxDisplayType
| */ | ||
| protected function getTaxDisplayType($OrderItemType) | ||
| { | ||
| if ($OrderItemType instanceof OrderItemType) { |
There was a problem hiding this comment.
処理内容はそのまま記述するのではなく、orderHelperに移譲してください。
return $this->orderHelper->getTaxDisplayType($OrderItemType)
| * - ポイント値引き: 税込 | ||
| * | ||
| * @param $OrderItemType | ||
| * |
There was a problem hiding this comment.
deprecatedのアノテーションを追加お願いします。
以下のように記載いただければ。
@deprecated OrderHelper::getTaxDisplayTypeを使用してください
概要(Overview・Refs Issue)
以下のissueの対応です。
#5753 受注編集の画面で、金額にエラーがあると、課税区分のカッコ内が表示されなくなる
問題を修正した後の結果は次のようになります。
方針(Policy)
実装に関する補足(Appendix)
テスト（Test)
相談（Discussion）
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目