dbalの3バージョンから削除されたfetchColumn関数をfetchOneに更新。

[sai-gillingham]

概要(Overview・Refs Issue)

ECCUBEの最新バージョンで以下のコマンドを実行すると、
以下のエラーが発生するため、修正PRを作成いたしました。
bin/console doctrine:migrations:migrate --no-interaction

Call to undefined method Doctrine\DBAL\Connection::fetchColumn()"

原因としては、👇のPRが作成された時点ではECCUBEはdbalの2バージョンを利用していたため、その時点では問題がなかったのかもしれません。しかし、4.3ではdbalの3バージョンを利用しているため、dbalの3以降で削除された関数が最新のECCUBEのコードに含まれています。

• Googleアナリティクス機能を追加 #4912
• 規格・規格分類：「CSV登録」作成, 「CSVダウンロード・CSV出力項目設定」追加 #5043

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

テスト＃１) 最新のECCUBEで下記のコマンドを実行したら、エラーが出るかどうかを確認すること。
bin/console doctrine:migrations:migrate --no-interaction

テスト＃１が行った後、テスト ＃2)

1. doctrine_migration_versionsテーブルからversion='DoctrineMigrations\Version20210216120000'レコードを削除する
2. bin/console doctrine:migrations:migrate --no-interactionコマンドを実行する。
3. dtb_blockテーブルでblock_name='Googleアナリティクス'をクエリーして、１レコードがあるか確認すること。
   • ２つ以上のレコードはNG。

相談（Discussion）

• 1点ご提案があります。GitHub Actionsにマイグレーションテストを追加することで、より安全になると思いますが、いかがでしょうか？
• 修正したマイグレーションファイルには、取り消し用down()関数にSQLが含まれていませんが、それで本当に大丈夫でしょうか？

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

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

レビュワー確認項目

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.79%. Comparing base (6116449) to head (9b0786c).

Additional details and impacted files

@@           Coverage Diff           @@
##              4.3    #6229   +/-   ##
=======================================
  Coverage   82.79%   82.79%           
=======================================
  Files         480      480           
  Lines       26419    26419           
=======================================
  Hits        21873    21873           
  Misses       4546     4546           

Flag	Coverage Δ
E2E	82.79% <ø> (ø)
Unit	82.79% <ø> (ø)

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.

[nanasess]

LGTM