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.

Sign up for GitHub

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

Jump to bottom

[4.2] インボイス対応 #5382

Merged
merged 18 commits into from
Jun 23, 2022
Merged

[4.2] インボイス対応 #5382

merged 18 commits into from
Jun 23, 2022

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Jun 6, 2022 •
edited
Loading

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

  • 注文確認画面(フロント)、注文完了メール、注文編集画面(管理画面)に消費税額を表示
  • お支払い金額から、消費税額を算出するよう変更
    • 非課税、不課税明細(ポイント明細など)は、税率に対する売上金額の割合で按分して税額を算出するよう変更
      invoice_front
      invoice_admin
      invoice_mail
  • ショップ管理→基本情報設定に適格請求書発行事業者登録番号のフォームを追加
    invoice_registration_num
  • 納品書に適格請求書発行事業者登録番号、消費税額を表示
    invoice_document

テスト（Test)

以下計算例と同様のユニットテストを追加
#4268 (comment)

相談（Discussion）

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか
    • - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ] - [ ]

@chihiro-adachi chihiro-adachi added the enhancement 機能追加 label Jun 8, 2022
@chihiro-adachi chihiro-adachi added this to the 4.2.0 milestone Jun 8, 2022
@nanasess nanasess changed the title インボイス対応 [4.2] インボイス対応 Jun 13, 2022
@nanasess nanasess marked this pull request as draft June 16, 2022 13:58
@nanasess nanasess marked this pull request as ready for review June 21, 2022 01:12
@chihiro-adachi chihiro-adachi mentioned this pull request Jun 21, 2022
Closed
chihiro-adachi
chihiro-adachi suggested changes Jun 21, 2022
View reviewed changes
src/Eccube/Entity/BaseInfo.php Outdated Show resolved Hide resolved
src/Eccube/Form/Type/Admin/ShopMasterType.php Outdated Show resolved Hide resolved
@chihiro-adachi
Copy link
Contributor

@nanasess
マイページも修正必要かもです。
image

@nanasess
Copy link
Contributor Author

@chihiro-adachi マイページも対応しました

@chihiro-adachi
Copy link
Contributor

@nanasess
修正ありがとうございます。

1点質問なのですが端数処理ってどこで実行されてます？
実装箇所がぱっとみあたらずでして。。

@nanasess
Copy link
Contributor Author

@chihiro-adachi コードを整理したときに処理がぬけてしまったみたいです。。。
確認しますので少々おまちください

@nanasess
Copy link
Contributor Author

@chihiro-adachi 処理追加しました

@chihiro-adachi
Copy link
Contributor

@nanasess
ありがとうございます。修正確認しました。

一点懸念として、dtb_order_itemのデータ構造上、同一税率であっても課税規則は異なるものを持つことは可能です。
ただ、レアケースかと思いましたので、仕様ということで扱いたいと思います。
もしそのようなデータができた場合は、dtb_order_item.rounding_type_idの値をデータ修正してもらうことで解消可能です。

@chihiro-adachi chihiro-adachi merged commit 66c8d4c into EC-CUBE:4.2 Jun 23, 2022
@chihiro-adachi chihiro-adachi mentioned this pull request Jul 5, 2022
Closed
@nanasess nanasess deleted the improve-invoice branch October 3, 2022 07:53
@xuelian311 xuelian311 mentioned this pull request Jul 5, 2023
Merged
15 tasks
@nanasess nanasess mentioned this pull request Sep 1, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chihiro-adachi chihiro-adachi chihiro-adachi requested changes

Assignees
No one assigned
Labels
enhancement 機能追加 Status: ready-for-merge
Projects
None yet
Milestone
4.2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@nanasess @chihiro-adachi