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

マイグレーション時のエラーを修正 #4309

Merged

Conversation

@kiy0taka
Copy link
Contributor

commented Sep 5, 2019

概要(Overview・Refs Issue)

#4281 で追加しているレコードがidを固定にしているため、プラグインで追加されたCSVカラム設定がある場合にエラーになるのを修正

方針(Policy)

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
@@ -32,7 +32,10 @@ public function up(Schema $schema) : void
return;
}
$this->addSql("INSERT INTO dtb_csv (id, csv_type_id, creator_id, entity_name, field_name, reference_field_name, disp_name, sort_no, enabled, create_date, update_date, discriminator_type) VALUES (204, 1, null , 'Eccube\\\\Entity\\\\ProductClass', 'TaxRule', 'tax_rate', '税率', 31, false, CURRENT_TIMESTAMP, CURRENT_TIMESTAMP,'csv')");
$taxRateExists = $this->connection->fetchColumn("SELECT COUNT(*) FROM dtb_csv WHERE csv_type_id = 1 AND entity_name = 'Eccube\\\\Entity\\\\ProductClass' AND field_name = 'TaxRule' AND reference_field_name = 'tax_rate'");
if ($taxRateExists === 0) {

This comment has been minimized.

Copy link
@chihiro-adachi

chihiro-adachi Sep 5, 2019

Contributor

@kiy0taka
$taxRateExistsがstringの場合があるようです。

mysql 5.7 -> string
postgres 9.6 -> integer

DBの差異なのかバージョンによって異なるのかは見れてないです。

This comment has been minimized.

Copy link
@chihiro-adachi

chihiro-adachi Sep 5, 2019

Contributor

mysql/postgresともに動作確認しました。
Eccube\\\\Entity\\\\ProductClassの値も正常に登録されています。

@kiy0taka kiy0taka force-pushed the kiy0taka:dev/fix_migration_20190821081036 branch from 8a1ea19 to 8f763d4 Sep 5, 2019

@chihiro-adachi chihiro-adachi added this to the 4.0.3 milestone Sep 5, 2019

@chihiro-adachi chihiro-adachi merged commit 7b267c5 into EC-CUBE:4.0 Sep 5, 2019

3 checks passed

Inspection Summary
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.