Bug（バグ） #3190
2系から3系にコンバートする際に正しくフォームタイプがコンバートされない
開始日:
2012-08-30
期日:
進捗率:
100%
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
No (いいえ)
説明
■詳細
2系にはあって3系にはないプロフィールのフォームタイプが存在し、それらが設定されていると正しくコンバートされない。
例）
text,textlong
■環境
PC / win7 Firefox14 / OpenPNE3.8.0
関係しているリビジョン
fixed upgrade strategy to convert form_type correctly (fixes #3190)
履歴
#1
Yuya Watanabe が11年以上前に更新
- 説明 を更新 (diff)
#2
ichikawa tatsuya が11年以上前に更新
２系でフォームタイプがtext及びtextlongのものをtextareaに変更するよう修正しました。
https://github.com/ichikawatatsuya/OpenPNE3/commit/4e63aff39207518daa8df62f5cda118f4ded5162
diff --git data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
index a3eaa83..82e0934 100644
--- data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
+++ data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
@@ -72,6 +72,11 @@ class opUpgradeFrom2MemberProfileStrategy extends opUpgradeAbstractStrategy
break;
}
+ if ('text' === $profile['form_type'] || 'textlong' === $profile['form_type'])
+ {
+ $profile['form_type'] = 'textarea';
+ }
+
$this->conn->execute('INSERT INTO profile (id, name, is_required, is_unique, is_edit_public_flag, default_public_flag, form_type, value_type, is_disp_regist, is_disp_config, is_disp_search, value_regexp, value_min, value_max, sort_order, created_at, updated_at) VALUES (?, ?, ?, 0, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, NOW(), NOW())', array(
$profile['c_profile_id'] , $profile['name'] , $profile['is_required'] , $profile['public_flag_edit'] , $publicFlagDefault ,
$profile['form_type'] , $profile['val_type'] , $profile['disp_regist'] , $profile['disp_config'] , $profile['disp_search'] ,
#3
ichikawa tatsuya が11年以上前に更新
- ステータス を New（新規） から Accepted（着手） に変更
#4
ichikawa tatsuya が11年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
#5
ichikawa tatsuya が11年以上前に更新
- 担当者 を Eitarow Fukamachi にセット
#7
Chiharu Nakajima がほぼ11年前に更新
- 対象バージョン を OpenPNE 3.6.x から OpenPNE 3.6.10 に変更
#8
Chiharu Nakajima がほぼ11年前に更新
- 3.8 で発生するか を Yes (はい) から No (いいえ) に変更
#9
Yuya Watanabe がほぼ11年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
2.14 でのフォームタイプ¶
| 値 | 管理画面での表記 |
|---|---|
| text | テキスト |
| textlong | テキスト(長) |
| textarea | テキスト(複数行) |
| select | 単一選択(プルダウン) |
| radio | 単一選択(ラジオボタン) |
| checkbox | 複数選択(チェックボックス) |
3.6 でのフォームタイプ¶
| 値 | 管理画面での表記 |
|---|---|
| input | テキスト |
| textarea | テキスト(複数行) |
| select | 単一選択(プルダウン) |
| radio | 単一選択(ラジオボタン) |
| checkbox | 複数選択(チェックボックス) |
| date | 日付 |
2.14 と 3.6 の差異¶
| 2.14 の値 | 3.6 の値 | 管理画面での表記 |
|---|---|---|
| text | input | テキスト |
| textlong | テキスト(長) |
管理画面での表記との対応を見る限りだと修正した内容は正しくないようですが，大丈夫でしょうか？
確認のため差し戻します．
#10
pnetan がほぼ11年前に更新
2系で「text」のものを「input」へ、「textlong]のものを「textarea」に変更しました。
diff --git data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
index d76268e..a3eaa83 100644
--- data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
+++ data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
@@ -72,15 +72,6 @@ class opUpgradeFrom2MemberProfileStrategy extends opUpgradeAbstractStrategy
break;
}
+ if ('text' === $profile['form_type'])
+ {
+ $profile['form_type'] = 'input';
+ }
+ elseif ('textlong' === $profile['form_type'])
+ {
+ $profile['form_type'] = 'textarea';
+ }
+
$this->conn->execute('INSERT INTO profile (id, name, is_required, is_unique, is_edit_public_flag, default_public_flag, form_type, value_type, is_disp_regist, is_disp_config, is_disp_search, value_regexp, value_min, value_max, sort_order, created_at, updated_at) VALUES (?, ?, ?, 0, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, NOW(), NOW())', array(
$profile['c_profile_id'] , $profile['name'] , $profile['is_required'] , $profile['public_flag_edit'] , $publicFlagDefault ,
$profile['form_type'] , $profile['val_type'] , $profile['disp_regist'] , $profile['disp_config'] , $profile['disp_search'] ,
#12
ichikawa tatsuya がほぼ11年前に更新
- ステータス を Rejected（差し戻し） から Accepted（着手） に変更
- 進捗率 を 50 から 0 に変更
#13
ichikawa tatsuya がほぼ11年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
#14
Yuya Watanabe がほぼ11年前に更新
パッチ適用時に失敗するようです．
$ wget https://redmine.openpne.jp/attachments/download/472/3190_2.patch $ patch -p0 --dry-run < 3190_2.patch patching file data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php patch: **** malformed patch at line 12: + }
#15
Yuya Watanabe がほぼ11年前に更新
正しいパッチは下記の通りです．
diff --git data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
index d76268e..a3eaa83 100644
--- data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
+++ data/upgrade/2/opUpgradeFrom2MemberProfileStrategy.class.php
@@ -72,6 +72,15 @@ class opUpgradeFrom2MemberProfileStrategy extends opUpgradeAbstractStrategy
break;
}
+ if ('text' === $profile['form_type'])
+ {
+ $profile['form_type'] = 'input';
+ }
+ elseif ('textlong' === $profile['form_type'])
+ {
+ $profile['form_type'] = 'textarea';
+ }
+
$this->conn->execute('INSERT INTO profile (id, name, is_required, is_unique, is_edit_public_flag, default_public_flag, form_type, value_type, is_disp_regist, is_disp_config, is_disp_search, value_regexp, value_min, value_max, sort_order, created_at, updated_at) VALUES (?, ?, ?, 0, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, NOW(), NOW())', array(
$profile['c_profile_id'] , $profile['name'] , $profile['is_required'] , $profile['public_flag_edit'] , $publicFlagDefault ,
$profile['form_type'] , $profile['val_type'] , $profile['disp_regist'] , $profile['disp_config'] , $profile['disp_search'] ,
パッチファイルの差分． 3190_2.patch が修正したもの．3190_2.patch.1 が添付ファイル．
$ diff 3190_2.patch 3190_2.patch.1 5c5 < @@ -72,6 +72,15 @@ class opUpgradeFrom2MemberProfileStrategy extends opUpgradeAbstractStrategy --- > @@ -72,15 +72,6 @@ class opUpgradeFrom2MemberProfileStrategy extends opUpgradeAbstractStrategy
#16
Yuya Watanabe がほぼ11年前に更新
更新履歴 e7bb1545032e65658633951a5830b76917a18c4a で適用されました。
#17
Yuya Watanabe がほぼ11年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
#18
Chiharu Nakajima がほぼ11年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テスト完了しました。問題ありません。