ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 13 years ago

Last modified 12 years ago

#2905 new defect

「0」と入力すると空欄扱いになるフォームが多くある

Reported by: kiwa Owned by: nobody
Priority: trivial Milestone:
Component: その他 Version: 2.12.x & 2.14.x
Keywords: Cc:

Description (last modified by mori)

■現象

PC・携帯ともに、テキストエリアに「0」と入力すると空欄扱いになるフォームが多くある

  • 最新日記検索(pc_page_h_diary_list_all)
    • キーワード検索
  • メンバー検索(pc_page_h_search)
    • 検索

修正した関数・アクション

  • p_h_album_list_all_search_c_album4c_album関数
  • p_h_diary_list_all_search_c_diary4c_diary関数
  • db_review_search_result4keyword_category関数
  • monitor_diary_list関数
  • monitor_diary_comment_list関数
  • monitor_commu_list関数
  • monitor_topic_comment_list関数
  • monitor_topic_list関数
  • monitor_review_list関数
  • db_admin_album_list関数
  • ktai_page_fh_diary_list関数
  • pc_page_fh_album_list関数
  • pc_page_fh_diary_list関数
  • h_review_addページアクション
  • biz_getGroupList関数

■原因

意図しないキャスト処理が行われている箇所がある。

■修正内容

文字列型の"0"に対する考慮のない判定処理を修正した。({if $item.nickname})を({if $item.nickname !== ''})にするなど。これにより、 #2874 を除き、入力フォームで"0"を入力時に正しく"0"と識別するようになった。

■関連情報

  • #2874 ニックネームなどが「0」のみだと、表示される部分に表示されない場合がある

Change History (16)

comment:1 Changed 13 years ago by kiwa

Description: modified (diff)

comment:2 Changed 13 years ago by mori

Owner: changed from nobody to mori
Status: newassigned

担当します。

comment:3 Changed 13 years ago by mori

Keywords: OpenPNE2.10.12 OpenPNE2.13.4 added
Milestone: OpenPNE2.12.6

comment:4 Changed 13 years ago by mori

Keywords: OpenPNE2.13.4 確認中 added; OpenPNE2.13.4 removed

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

但し、関連情報であげられている #2874 ニックネームなどが「0」のみだと、表示される部分に表示されない場合がある   は、別にコミットします。

comment:5 Changed 13 years ago by mori

Description: modified (diff)

comment:6 Changed 13 years ago by kiwa

Keywords: OpenPNE2.13.4 差し戻し added; OpenPNE2.13.4 確認中 removed

Descriptionについての指摘です。下記の修正をお願いします。

  1. ■修正内容について、ユーザ側から見てどのような対応になったのかも記載してほしいです
    • 例: #2910 「絞込みをすると1ページ目に移動するように修正。」

※ 今回#2910を例に挙げてますが、条件を満たしたものを適当に選んだだけなので「これが最良例」というわけではありません

comment:7 Changed 13 years ago by mori

Description: modified (diff)
Keywords: OpenPNE2.13.4 テスト待ち added; OpenPNE2.13.4 差し戻し removed

comment:8 Changed 13 years ago by ebihara

Keywords: OpenPNE2.13.4 確認中 added; OpenPNE2.13.4 テスト待ち removed

comment:9 Changed 13 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

$keyword が登場する判定処理しか修正の対象としていないようですが、このチケットの範囲は明らかにそれを超えています。

comment:10 Changed 13 years ago by ebihara

Owner: changed from mori to nobody
Status: assignednew

comment:11 Changed 13 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

海老原の担当とします。

comment:12 Changed 13 years ago by ebihara

開発談義での以下のやり取りにより、今回は見送ります。

海老原昂輔 14:02
んーちょっと今回の開発版では以下のチケット見送りたいです。

#2905:「0」と入力すると空欄扱いになるフォームが多くある
http://trac.openpne.jp/ticket/2905

理由としては以下です。
・mori さんの修正が不足している(必要な修正の1割くらいしかおこなわれていない)
・本日中に充分な品質での修正をおこなうのは難しい
・重大なバグであるとはいえない
・元々の対応項目には入っていなかった(moriさんが自主的にマイルストンをつけたものである)

どうでしょう?

きわのすけ 14:07
動作テストもかなり骨が折れるので、見送ってきちんと検証+修正+テストを行ったほうがいいですね。自分は見送りに賛成です。

海老原昂輔 14:26
では今回は見送ります

きわのすけ 14:27
了解です

comment:13 Changed 13 years ago by ebihara

Keywords: OpenPNE2.10.12 OpenPNE2.13.4 差し戻し removed
Milestone: OpenPNE2.12.6

以下のリビジョンで変更を取り消しました。今回の対応項目から外します。

comment:14 Changed 12 years ago by ebihara

Owner: changed from ebihara to nobody
Status: assignednew

comment:15 Changed 12 years ago by ebihara

Version: 2.10.x & 2.12.x & 2.13.x2.12.x & 2.13.x

comment:16 Changed 12 years ago by ebihara

Priority: minortrivial
Note: See TracTickets for help on using tickets.