Project

General

Profile

Bug(バグ) #2905

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

Added by Mutsumi Imamura almost 8 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-03-26
Due date:
% Done:

100%

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

Description

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 (修正内容)

修正内容を記入

Associated revisions

Revision 5ff72406 (diff)
Added by Hidenori Goto over 7 years ago

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

History

#1 Updated by Shouta Kashiwagi almost 8 years ago

  • Target version changed from 261 to OpenPNE 3.8.x

#2 Updated by Yuma Sakata almost 8 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.6.4

#3 Updated by Mutsumi Imamura over 7 years ago

  • Assignee set to Hidenori Goto

#4 Updated by Hidenori Goto over 7 years ago

問題箇所

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 Updated by Hidenori Goto over 7 years ago

  • Status changed from New(新規) to Accepted(着手)

#6 Updated by Hidenori Goto over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#7 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

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

5ff724065fe82ac7cbdc403a584698b8e24774a7

#8 Updated by Yuma Sakata over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#9 Updated by kaoru n over 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF