Backport(バックポート) #2853
完了#1193 の修正により op_include_form() を使用している箇所で renderHelp() の内容が出力されていない
100%
説明
概要¶
フォームを出力するパーツ apps/pc_frontend/templates/_partsForm.php について、 op_include_form() 関数などを用いてこのパーツを使用すると、フォームクラス内で setHelp() メソッドで設定されているはずの Help テキストが出力されない。
原因¶
#1193 では CAPTCHA に関する renderError() の内容が出力されない問題を対応しているが、 renderError() を明示的に出力するように変更した際に、 renderHelp() の出力を考慮していない。
ただし、この問題について、 member/config?category=pcAddress などの CAPTCHA を含むフォームでは CAPTCHA に関しては Help テキストが出力されている( render 系のメソッドで何が出力されるのかがよく分かっていない)。
http://redmine.openpne.jp/issues/1193#note-12 にもコメントを書いています。
修正方針¶
render 系のメソッドが何を出力するかを確認した上で、 #1193 での修正のような書き換えを行なう。 renderHelp() を明示的に出力する記述を付け加えることで対応できるのではないかと考えている。
影響のあるバージョン¶
#1193 の修正を行なった OpenPNE-3.4.x, 3.6.x, master は全て影響があると思われる。
フォームにおいて Help テキストが出力されていないことで、エンドユーザのフォーム操作の利便性が損なわれ、また開発者側も Help テキストを出力できなくなってしまっているため、この不具合は速やかに修正されることが好ましい。
Minoru Takai さんが12年以上前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 03e64ed4339135493bdfa323df6f8f593f560f56 で適用されました。
Kousuke Ebihara さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Yuma Sakata さんが12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKです。