プロジェクト

全般

プロフィール

Backport(バックポート) #2853

#1193 の修正により op_include_form() を使用している箇所で renderHelp() の内容が出力されていない

Minoru Takai約12年前に追加. 約12年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2012-02-27
期日:
進捗率:

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 テキストを出力できなくなってしまっているため、この不具合は速やかに修正されることが好ましい。


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2847: #1193 の修正により op_include_form() を使用している箇所で renderHelp() の内容が出力されていない Won't fix(対応せず) 2012-02-27
関連している OpenPNE 3 - Backport(バックポート) #2724: 確認キーワードを誤入力または未入力の場合にエラーメッセージが表示されない Fixed(完了) 2010-06-22

関係しているリビジョン

リビジョン 64e6c7cc (差分)
Minoru Takai約12年前に追加

Revert "(refs #2853, commit of #2724) use renderError() and render() instead of renderRow() to output error messages in global/partsForm"

This reverts commit 7460cb18955af672d36185b243d5608b197f9ef2.

リビジョン 03e64ed4 (差分)
Minoru Takai約12年前に追加

(fixes #2853, BP from #2847) use renderError() and render() instead of renderRow() ONLY captcha field.

履歴

#1 Minoru Takai約12年前に更新

  • 対象バージョンOpenPNE 3.4.x から OpenPNE 3.4.21 に変更

3.4.21 対象とします。

#2 Minoru Takai約12年前に更新

  • 担当者Minoru Takai にセット

#3 Minoru Takai約12年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 03e64ed4339135493bdfa323df6f8f593f560f56 で適用されました。

#4 Kousuke Ebihara約12年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#5 Yuma Sakata約12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

他の形式にエクスポート: Atom PDF