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

DBの文字コードをutf8mb4に対応 #4796

Merged
merged 7 commits into from
Jun 9, 2022

Conversation

k-yamamura
Copy link
Contributor

概要(Overview・Refs Issue)

レンタルサーバでもutf8mb4を扱えるDBが増えてきたため、
MySQLの標準文字コードをutf8mb4でインストールするように対応
#4525

実装に関する補足(Appendix)

2系や3系からutf8の文字コードを利用しているDBを移行してきても特に影響はなさそう。

相談(Discussion)

初期インストールであれば問題ないが、
バージョンアップ時に、
dtb_productの
note
description_list
description_detail
search_word
の型変更は必要かどうか。

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

@okazy okazy added the improvement 機能改善 label Dec 22, 2020
@okazy okazy added this to the 4.0.x milestone Dec 22, 2020
@okazy okazy changed the base branch from 4.0 to 4.1-feature February 17, 2021 05:18
@okazy okazy modified the milestones: 4.0.x, 4.1 Feb 17, 2021
@chihiro-adachi chihiro-adachi modified the milestones: 4.1, 4.1.x Sep 3, 2021
@chihiro-adachi chihiro-adachi changed the base branch from 4.1-feature to 4.1 September 6, 2021 05:20
@nanasess nanasess mentioned this pull request Apr 18, 2022
@chihiro-adachi chihiro-adachi modified the milestones: 4.1.x, 4.2.0 May 30, 2022
@chihiro-adachi chihiro-adachi changed the base branch from 4.1 to 4.2 June 3, 2022 07:18
@chihiro-adachi
Copy link
Contributor

@k-yamamura
ありがとうございます。

一部修正かけています。

@chihiro-adachi
Copy link
Contributor

@nanasess
こちらの内容みていただいてもよいですか?

@nanasess
Copy link
Contributor

nanasess commented Jun 3, 2022

@chihiro-adachi 手元の環境では、大元の charset を utf8mb4 にする必要がありました

diff --git a/app/config/eccube/packages/doctrine.yaml b/app/config/eccube/packages/doctrine.yaml
index e52cd87129..372f428cdb 100644
--- a/app/config/eccube/packages/doctrine.yaml
+++ b/app/config/eccube/packages/doctrine.yaml
@@ -9,7 +9,7 @@ doctrine:
     dbal:
         driver: 'pdo_sqlite'
         server_version: "%env(DATABASE_SERVER_VERSION)%"
-        charset: 'utf8'
+        charset: 'utf8mb4'

         # for mysql only
         default_table_options:

charset も env で設定するか、何とかして動的に設定しないといけないかもしれないです

@chihiro-adachi
Copy link
Contributor

@nanasess
ありがとうございます。
envで設定するように修正してみました。
7ac8bfb

@nanasess
Copy link
Contributor

nanasess commented Jun 7, 2022

@chihiro-adachi 設定できるようになりました
image

@chihiro-adachi
Copy link
Contributor

@nanasess
ありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement 機能改善
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants