OpenPNE 3

参考程度にコメントをしておきます。以下の考察に基づいて修正が可能か、あるいは以下の考察が間違っているようであれば対応・指摘等、どなたかお願いします。

問題を解消できているか¶

テキストボックス（テキストフィールド：HTMLにおける input[type="input"] 要素）とテキストエリア（HTMLにおける textarea 要素）に対してサイズの属性を与える修正としては、修正に誤りなどはないと判断しています。そういう意味ではコードチェックはOKかと思います（が、懸念事項を考慮する必要があります）。

属性値がそれぞれ size=40, cols=50, rows=10 でよいかについても、2系に合わせるという観点からは問題はないと思います。

この修正による副作用がないか¶

この修正の懸念事項として、widgetで指定していた幾つかの attributes (size, col, rows) を無効化してしまう可能性がある。

と（チケットの概要で）示されていますが、属性値を上書きしてしまうのであれば、この懸念は考慮しなければなりません。

widget で属性が指定されていない場合に限り、デフォルトとなる属性値（size=40 および cols=50, rows=10）を与えるようなことができれば、この問題に対する修正としては適切ではないかと思っています。

「widget で属性が指定されていない場合に限り、デフォルトとなる属性値を与えることができるか」についてはまだ調査していないため、コメントのみを残しておきます。

「widget で属性が指定されていない場合に限り、デフォルトとなる属性値を与えることができるか」について、実際にテストしてみました。

opBlogPluginのPC版「BlogのURL設定（member/config?category=blogUrl）」にて、
MemberConfigBlogUrlFormでは「size=57」と属性指定されていますが、上記修正を取り込んだ状態では「size=40」となっています。

普通にインストールした状態で属性指定しているのは、上記とopIntroFriendPluginのPluginIntroFriendFormのみのようですが
IntroFriendの方は、_partsForm.phpを利用していないため、指定した属性（'rows' => 8, 'cols' => 88）が反映しています。

サイズ指定が上書きされる問題の回避措置についての検討¶

当初考えていた実装方針¶

• サイズの指定の有無をテンプレート側で判別し、サイズの指定がない場合はデフォルト値を設定する。

上記実装方針の問題点¶

sfWidgetFormTextareaのconfigureでrows, colsのデフォルトを指定しているため、OpenPNEの実装者が指定した値なのかどうかの区別がつかない

「OpenPNEの実装者が指定した値なのかどうかの区別がつかない」問題の回避策¶

• デフォルト値を設定しているクラスのサブクラスを作り対応

補足事項¶

• partsForm を使用していない箇所の対応についての検討も必要

• 優先度が「high」である理由が不明確
• 対応するにしてもマイナーバージョンアップでも可能なないようであることが予想される