プロジェクト

全般

プロフィール

Bug(バグ) #1745

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

Kiwa Sakai13年以上前に追加. 8年以上前に更新.

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

100%

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

説明

再現バージョン

  • 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" 

関連するチケット

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

関係しているリビジョン

リビジョン dba31e32 (差分)
Hiroshi Kato12年以上前に追加

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

リビジョン 745757bb (差分)
Hiroshi Kato12年以上前に追加

refactored some processes. (fixes #1745)

履歴

#1 Kiwa Sakai13年以上前に更新

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

#2 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.7.0 にセット
  • 3.6 で発生するかYes にセット

#3 Hiroshi Kato12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Hiroshi Kato にセット

#4 Hiroshi Kato12年以上前に更新

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

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

#5 Kousuke Ebihara12年以上前に更新

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

#6 Kousuke Ebihara12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Rejected(差し戻し) に変更
  • 進捗率70 から 50 に変更

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

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

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

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

#7 Hiroshi Kato12年以上前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#8 Kousuke Ebihara12年以上前に更新

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

#9 Shingo Yamada12年以上前に更新

  • 360対象beta13 にセット

#10 Fumie Toyooka12年以上前に更新

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

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

#11 kaoru n8年以上前に更新

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

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