Skip to content

Conversation

@nobuhiko
Copy link
Owner

@nobuhiko nobuhiko commented Feb 8, 2026

Summary

  • ECCUBE2Downloadsプラグインがインストール済みの場合、2.xバックアップの mtb_product_type.csv から「ダウンロード」販売種別のIDを検出
  • dtb_product_class 及び dtb_deliverysale_type_id を 222(ECCUBE2Downloadsの販売種別ID)にマッピング
  • down_filename / down_realfilename は既存の汎用マッピングロジックで自動移行される(追加コード不要)

Test plan

  • ECCUBE2Downloadsプラグインを有効化した状態でDataMigration43のテストを実行
  • ダウンロード商品を含む2.xバックアップを移行し、dtb_product_classsale_type_id が222になることを確認
  • 通常商品の sale_type_id が222にならないことを確認
  • dtb_delivery のダウンロード配送の sale_type_id が222になることを確認
  • down_filename, down_realfilename が移行されることを確認

🤖 Generated with Claude Code

nobuhiko and others added 7 commits February 8, 2026 10:06
ECCUBE2Downloadsプラグインがインストール済みの場合、2.xバックアップの
mtb_product_type.csvから「ダウンロード」販売種別のIDを検出し、
dtb_product_class及びdtb_deliveryのsale_type_idを222にマッピングする。
down_filename/down_realfilenameは既存の汎用ロジックで自動移行される。

テストを追加: sale_type_id=222の書き換えとdown_filename移行の検証。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- composer.json: 4.3.6 → 4.3.7
- README: テーブル形式で見やすく整理、ECCUBE2Downloads連携の説明を追加、
  重複していたPostgrSQL注意点を統合、移行できないデータの記述を更新

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
検出ロジックがsaveToP(dtb_product_class)の後にあったため、
sale_type_idマッピング時にdownloadProductTypeIdがnullだった問題を修正。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ECCUBE2Downloadsプラグイン未インストール環境ではdown_filenameカラムが
存在しないため、information_schemaでカラム存在を確認してからアサートする。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- test.yamlにECCUBE2Downloadsのcheckout・アーカイブ・インストールを追加
- テストからdtb_plugin手動INSERTとinformation_schemaガードを削除
- down_filename/down_realfilenameのアサーションを常に実行

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
DataMigration43のtar.gzにECCUBE2Downloadsが含まれると
Symfonyが誤ったnamespaceでクラスを読み込もうとしてエラーになる。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
PostgreSQLではbooleanカラムに整数リテラルで比較できないため
visible = 1 を visible = true に修正。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nobuhiko nobuhiko merged commit a3070d2 into 4.2 Feb 8, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant