プロジェクト

全般

プロフィール

Bug(バグ) #2905

完了

OpenPNE2系からのコンバートでデイリーニュースを配信する/しないの設定値がコンバートされない

Mutsumi Imamura さんが12年以上前に追加. 約9年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2012-03-26
期日:
進捗率:

100%

予定工数:
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Unknown (未調査)

説明

Overview (現象)

OpenPNE2系からのコンバートでデイリーニュースを配信する/しないの設定値がコンバートされない。
コンバートされないため、配信を拒否していたメンバーにデイリーニュースが送信されてしまう場合がある。

余談ではあるが、OpenPNE2 と 3 では、デイリーニュースの配信設定値 value の値の違いがあるのでこの問題を修正する時には注意が必要である。

=== OpenPNE2
* value="0" 受け取らない
* value="1" 毎日
* value="2" 週1回 (月)

=== OpenPNE3
* value="0": 送信しない
* value="1": 週2回送信する
* value="2": 毎日送信する

再現バージョン

  • OpenPNE3.6.2

再現手順

  1. OpenPNE2系から3系にコンバートする。
  2. デイリーニュースの設定値を確認する。

Causes (原因)

バグが発生した原因を記入

Way to fix (修正内容)

修正内容を記入

Shouta Kashiwagi さんが12年以上前に更新

  • 対象バージョン261 から OpenPNE 3.8.x に変更

Yuma Sakata さんが12年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.6.4 に変更

Mutsumi Imamura さんが12年以上前に更新

  • 担当者Hidenori Goto にセット

Hidenori Goto さんが12年以上前に更新

問題箇所

Overviewにあるとおり、V2→V3のコンバータでデイリーニュース配信フラグに対する処理が行われていません。

修正方針

コンバータにて

  • /data/upgrade/2/sql/member_config.sql にて、以下のSQLを追加
INSERT INTO member_config (id, member_id, name, value, name_value_hash, created_at, updated_at) (SELECT NULL, c_member_id, "daily_news", 0, MD5(<?php echo $this->conn->expression->concat($this->conn->quote('daily_news'), $this->conn->quote(','), 0) ?>), NOW(), NOW() FROM c_member WHERE is_receive_daily_news = 0);
INSERT INTO member_config (id, member_id, name, value, name_value_hash, created_at, updated_at) (SELECT NULL, c_member_id, "daily_news", 2, MD5(<?php echo $this->conn->expression->concat($this->conn->quote('daily_news'), $this->conn->quote(','), 2) ?>), NOW(), NOW() FROM c_member WHERE is_receive_daily_news = 1);
INSERT INTO member_config (id, member_id, name, value, name_value_hash, created_at, updated_at) (SELECT NULL, c_member_id, "daily_news", 1, MD5(<?php echo $this->conn->expression->concat($this->conn->quote('daily_news'), $this->conn->quote(','), 1) ?>), NOW(), NOW() FROM c_member WHERE is_receive_daily_news = 2);

すでに運営中のコンバート済みSNSについては、まだmember_configにdaily_news設定値がないメンバーに対してのみ、c_memberからis_receive_daily_newsのフラグを取り出して設定するバッチを用意。

Hidenori Goto さんが12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

Hidenori Goto さんが12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

Yuya Watanabe さんが12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

下記コミットを stable-3.6.x ブランチにマージしました.

5ff724065fe82ac7cbdc403a584698b8e24774a7

Yuma Sakata さんが12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

kaoru n さんが約9年前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

他の形式にエクスポート: Atom PDF