プロフィール変更で必須でない項目を空白にした場合、その項目の公開範囲が反映されない

[imamura623]

■現象

プロフィールの変更において、必須ではない項目を空白にした場合、その項目の公開範囲が反映されない。

■原因

必須でない項目を選択しなかった場合や自己紹介を記入しなかった場合、公開範囲を選択してもデータベース上に書き込まれなかったため。

■修正内容

今までwebapp/lib/db/memberのdb_member_update_c_member_profileでは

itemvalue?に値が入っている場合書き込むという記述でnullの場合書き込まれないようになっていたのをnullの時""の値を返し記入するようにした。

この修正で必須でない項目を選択しなかった場合や自己紹介文を無記入の場合でも公開範囲のみ変更することが出来るようになりました。

■関連情報

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=16294 より転記

プロフィールのテキスト入力エリアを空白、公開範囲を"公開しない"に設定し、登録した場合に
再度プロフィール変更画面を開くとテキストエリアは空白だが公開範囲が"全員に公開"になっている。

公開したくないのに何度"公開しない"に設定しても"全員に公開"にコンボボックスが設定されてしまうと言われました。
実際は公開されていないのですが紛らわしいのでここ修正した方がいいかと思います。

[akamine]

対応します。

[akamine]

以下のリビジョンを修正しました。

• 2.10.x - r9111
• 2.12.x - r9110

[akamine]

以下のリビジョンを修正しました。

• trunk - r9131

[shingo]

◆2.12・2.13
今回の修正で以下の問題が生じてます。

• 「page_h_prof」のプロフィール欄に未記入の項目も表示されるようになってしまっています。（「公開しない」以外の場合）
• 「page_h_prof」のプロフィール欄に未記入の項目にも「マイフレンドまで公開」が表示されてしまっています。

◆2.10
今回の変更が反映されていないようです。

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9201
• 2.12.x - r9202
• trunk - r9203

[shingo]

• 今回の修正で追記された以下の箇所の「==」の後ろに空白を入れたほうがいいと思います。
  
• テンプレートファイル内でも処理に関わる部分の文字の引用は「'(シングルコーテーション)」のほうがいいと思います。
  

webapp/modules/ktai/templates/inc_f_home_h_prof.tpl

86	({if !$item.value ==""}) 

※他の箇所も同様

• (2.10：携帯)フォームタイプが「textarea」の場合以外の未入力の項目が表示されています。

• bizモードONの時、未入力の項目でも表示されてしまう。

[akamine]

以下のリビジョンを再度修正しました。

• 2.10.x - r9213,r9210
• 2.12.x - r9212,r9209
• trunk - r9214,r9211

[shingo]

訂正します。

• 2.10.x - r9212,r9215
• 2.12.x - r9211,r9214
• trunk - r9213,r9216

[kiwa]

以下の現象の確認をお願いします。

version

• 2.10.x (r9294の状態)
• 2.12.x (r9294の状態)
• trunk (r9294の状態)

概要

プロフィール項目「複数選択(チェックボックス)」の公開範囲の変更が反映されない

再現手順

1. 管理画面にて「複数選択(チェックボックス)」のプロフィール項目を作成する
2. メンバー画面からプロフィール項目変更(pc_page_h_config_prof)にアクセスする
3. 1で作成したプロフィール項目を、何もチェックせずに公開範囲だけ変更してプロフィール変更を完了する
4. 再度プロフィール項目変更(pc_page_h_config_prof)にアクセスする
5. 3で行った変更が保持されず、1のプロフィール項目がデフォルトの公開範囲になっている

[shingo]

以下のリビジョンで comment:14 の事項を修正しました。

• 2.10.x - r9307
• 2.12.x - r9304
• trunk - r9306

[shingo]

以下のリビジョンで comment:14 の事項を修正しました。

• 2.10.x - r9307
• 2.12.x - r9304
• trunk - r9306

以下のリビジョンでSlavePNEに対応しました。

• 2.10.x - r9311
• 2.12.x - r9310
• trunk - r9309

以下のリビジョンでテンプレートを修正しました。ご確認ください。

• 2.10.x - r9314
• 2.12.x - r9312
• trunk - r9313

[ebihara]

• if ($valuec_profile_option_id? == 0) は if (!$valuec_profile_option_id?) と同じ意味なので特に必要がなければそのように記述をお願いします
• db_member_update_c_member_profile() の、cache_drop_c_member_profile()直下の空行を消したのは何か意図があるのでしょうか
• 以下のアクションについては値が空文字列でないかどうかのチェックをおこなわなくても大丈夫なのでしょうか
  • admin_page_delete_c_member_confirm
  • admin_page_list_c_member
  • ktai_page_fh_profile_detail (チェックはおこなっているようですが型チェックが漏れています)

[shingo]

以下のリビジョンで修正しました。ご確認ください。

• 2.10.x - r9318
• 2.12.x - r9315 r9316
• trunk - r9317

[ebihara]

以下の修正は適切ではありません（valuとはなんでしょうか）

 49 ({if $item.form_type == 'checkbox' && $item.valu !== ''})

[ebihara]

comment:22 については海老原が修正を引き受けます。

[ebihara]

comment:22について以下のリビジョンで修正しました。

• 2.10.x - r9333
• 2.12.x - r9332
• trunk - r9331

[ebihara]

comment:24 でリビジョンを誤って表記していました。正しくは以下です。

• 2.10.x - r9333
• 2.12.x - r9331
• trunk - r9330

[kiwa]

確認しました。問題ありません。