Project

General

Profile

Bug(バグ) #1745

ポイント・マイニュース・自己紹介・生年月日以外のプロフィール項目が存在しないと、2→3系のプロフィールのコンバートに失敗する

Added by Kiwa Sakai about 9 years ago. Updated about 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2010-10-27
Due date:
% Done:

100%

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

Description

再現バージョン

  • OpenPNE2: OpenPNE2.14.8-dev
  • OpenPNE3: OpenPNE3.6beta7-dev

概要

OpenPNE2 時点でプロフィール項目を以下の項目以外存在しないようなデータにしてOpenPNE3へコンバートを行うと、コンバート時にエラーメッセージが表示される。
コンバート後にOpenPNE3環境を確認すると、プロフィール(/member/profile)やプロフィール編集(/member/edit/profile )にニックネーム以外のプロフィール項目(生年月日・自己紹介) が表示されない。

残すプロフィール項目
  • self_intro
  • PNE_POINT
  • PNE_MY_NEWS
  • PNE_MY_NEWS_DATETIME
  • 生年月日
コンバート時に表示されるエラーメッセージ
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') LIMIT 16' at line 1. Failing Query: "SELECT c_profile_id, name, is_required, public_flag_edit, public_flag_default, form_type, val_type, disp_regist, disp_config, disp_search, val_regexp, val_min, val_max, sort_order FROM c_profile WHERE c_profile_id IN () LIMIT 16" 

Related issues

Related to OpenPNE 3 - Backport(バックポート) #2180: ポイント・マイニュース・自己紹介・生年月日以外のプロフィール項目が存在しないと、2→3系のプロフィールのコンバートに失敗する Fixed(完了) 2011-06-10

Associated revisions

Revision dba31e32 (diff)
Added by Hiroshi Kato over 8 years ago

skip the member profile convert proccess, if it does not have profile.(fixes #1745)

Revision 745757bb (diff)
Added by Hiroshi Kato over 8 years ago

refactored some processes. (fixes #1745)

History

#1 Updated by Kiwa Sakai about 9 years ago

  • Subject changed from 2系でポイント・マイニュース・自己紹介・生年月日以外のプロフィール項目が存在しないと、プロフィールのコンバートに失敗する to ポイント・マイニュース・自己紹介・生年月日以外のプロフィール項目が存在しないと、2→3系のプロフィールのコンバートに失敗する

#2 Updated by Kousuke Ebihara about 9 years ago

  • Target version set to OpenPNE 3.7.0
  • 3.6 で発生するか set to Yes

#3 Updated by Hiroshi Kato over 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Hiroshi Kato

#4 Updated by Hiroshi Kato over 8 years ago

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

更新履歴 dba31e32e235d8805ef2a107366f66dc7871ae5f で適用されました。

#5 Updated by Kousuke Ebihara over 8 years ago

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

#6 Updated by Kousuke Ebihara over 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

修正自体は構わないのですが、 dba31e32e235d8805ef2a107366f66dc7871ae5f のコミットがどのブランチにも見当たりません。

目で 7/14 周辺の master のログを確認しても当該コミットが確認できませんでした。 https://github.com/openpne/OpenPNE3/commits/master

master への確実な取り込みをお願いします。

あと、差し戻しついでに指摘しますが、プロフィールのコンバートの処理をまるまるスキップするのであれば、この部分を別メソッドとして抽出して、 doRun() メソッドでは単にそのメソッドをコールするかどうかの判定(今回追加された判定)のみをおこなわせるようにしたほうが可読性が上がると思いますので、ご検討ください。

#7 Updated by Hiroshi Kato over 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 745757bbbdd482b155c1269d3e2528d40ecfc632 で適用されました。

#8 Updated by Kousuke Ebihara over 8 years ago

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

#9 Updated by Shingo Yamada over 8 years ago

  • 360対象 set to beta13

#10 Updated by Fumie Toyooka about 8 years ago

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

テスト無事終了しました。

#11 Updated by kaoru n about 4 years ago

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

Also available in: Atom PDF