Project

General

Profile

Bug(バグ) #2847

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

Added by Minoru Takai over 7 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
High(高め)
Assignee:
-
Target version:
Start date:
2012-02-27
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

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

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

Associated revisions

Revision 3765b4f9 (diff)
Added by Minoru Takai over 7 years ago

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

This reverts commit 872e491b1ecf5c9606201c91cdc724008a4dcdb6.

Revision 09ee89fd (diff)
Added by Minoru Takai over 7 years ago

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

History

#1 Updated by Minoru Takai over 7 years ago

  • Assignee set to Minoru Takai

#2 Updated by Minoru Takai over 7 years ago

  • Target version set to OpenPNE 3.7.0

3.7.x 向けであることを示すため OpenPNE 3.7.0 を対象バージョンとして指定しておきます。

#3 Updated by Minoru Takai over 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 09ee89fdf1e3de64de93bfc063c9ea14607316b2 で適用されました。

#4 Updated by Kousuke Ebihara over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#5 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#6 Updated by Yuya Watanabe over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8.x

#7 Updated by Yuya Watanabe about 7 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.8.1

まず stable-3.8.x に含まれるかどうかをまず確認する必要がある.

#8 Updated by Yuma Sakata about 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

修正が取り込まれていないので差し戻します。

#9 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Fixing(修正待ち)
  • % Done changed from 50 to 0

#10 Updated by 開 石切山 almost 7 years ago

  • Target version changed from OpenPNE 3.8.1 to OpenPNE 3.9.0-old

#11 Updated by 開 石切山 almost 7 years ago

  • Assignee deleted (Minoru Takai)

#12 Updated by kaoru n over 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

#13 Updated by isao sano over 2 years ago

  • Status changed from Pending Fixing(修正待ち) to Won't fix(対応せず)

OpenPNE 3.8.1 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF