Bug(バグ) #2847
#1193 の修正により op_include_form() を使用している箇所で renderHelp() の内容が出力されていない
0%
Description
概要¶
フォームを出力するパーツ 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 テキストを出力できなくなってしまっているため、この不具合は速やかに修正されることが好ましい。
Related issues
Associated revisions
(fixes #2847) use renderError() and render() instead of renderRow() ONLY captcha field.
History
#1
Updated by Minoru Takai almost 11 years ago
- Assignee set to Minoru Takai
#2
Updated by Minoru Takai almost 11 years ago
- Target version set to OpenPNE 3.7.0
3.7.x 向けであることを示すため OpenPNE 3.7.0 を対象バージョンとして指定しておきます。
#3
Updated by Minoru Takai almost 11 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 09ee89fdf1e3de64de93bfc063c9ea14607316b2 で適用されました。
#4
Updated by Kousuke Ebihara almost 11 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
#5
Updated by Shouta Kashiwagi almost 11 years ago
- Target version changed from OpenPNE 3.7.0 to 252
#6
Updated by Yuya Watanabe almost 11 years ago
- Target version changed from 252 to OpenPNE 3.8.x
#7
Updated by Yuya Watanabe over 10 years ago
- Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.1
まず stable-3.8.x に含まれるかどうかをまず確認する必要がある.
#8
Updated by Yuma Sakata over 10 years ago
- Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
- % Done changed from 70 to 50
修正が取り込まれていないので差し戻します。
#9
Updated by Yuya Watanabe over 10 years ago
- Status changed from Rejected(差し戻し) to Pending Fixing(修正待ち)
- % Done changed from 50 to 0
#10
Updated by 開 石切山 over 10 years ago
- Target version changed from OpenPNE 3.8.1 to OpenPNE 3.9.0-old
#11
Updated by 開 石切山 over 10 years ago
- Assignee deleted (
Minoru Takai)
#12
Updated by kaoru n almost 8 years ago
- 3.8 で発生するか set to Unknown (未調査)
#13
Updated by isao sano almost 6 years ago
- Status changed from Pending Fixing(修正待ち) to Won't fix(対応せず)
OpenPNE 3.8.1 にて対応済みであったため、対応せずとします。