プロジェクト

全般

プロフィール

Bug(バグ) #2905

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

Mutsumi Imamura約12年前に追加. 8年以上前に更新.

ステータス:
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 (修正内容)

修正内容を記入

関係しているリビジョン

リビジョン 5ff72406 (差分)
Hidenori Gotoほぼ12年前に追加

(refs #2905) Added converter SQL for a delivery flag of a daily news

履歴

#1 Shouta Kashiwagi約12年前に更新

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

#2 Yuma Sakataほぼ12年前に更新

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

#3 Mutsumi Imamuraほぼ12年前に更新

  • 担当者Hidenori Goto にセット

#4 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のフラグを取り出して設定するバッチを用意。

#5 Hidenori Gotoほぼ12年前に更新

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

#6 Hidenori Gotoほぼ12年前に更新

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

#7 Yuya Watanabeほぼ12年前に更新

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

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

5ff724065fe82ac7cbdc403a584698b8e24774a7

#8 Yuma Sakataほぼ12年前に更新

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

テストOKです。

#9 kaoru n8年以上前に更新

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

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