Opened 13 years ago
Closed 12 years ago
#2833 closed defect (fixed)
PC版のテンプレートでform要素の開始タグ・終了タグが対応していない箇所がある
Reported by: | imamura623 | Owned by: | nakasone |
---|---|---|---|
Priority: | minor | Milestone: | OpenPNE2.10.10 |
Component: | 指定しない | Version: | 2.10.x |
Keywords: | Cc: |
Description (last modified by )
■現象
PC版のテンプレートにおいて form 要素の開始タグ・終了タグの対応が取れていない箇所がある。
■原因
- form 要素の終了タグが余分に記述されている箇所がある
- form 要素の終了タグが不足している(開始タグのみが記述されている)箇所がある
なお、この現象は XHTML+CSS 化がおこなわれていない 2.10.x のみで発生する。
■修正内容
- 以下のファイルに存在した不要な form 要素の終了タグを削除した。
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/h_review_add_write_confirm.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_event_invite_end.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_event_write_delete_confirm.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_join_commu.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_join_err_wait.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/fh_comment_list.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/fh_delete_comment.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/o_public_invite_end.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp_biz/modules/biz/templates/fh_biz_schedule_calendar.tpl
- 以下のファイルで不足していた form 要素の終了タグを補った。
■関連情報
http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=17861 より転記
・現象 特に何も起きない ・原因 formの閉じるタグが多い ・当該箇所 OpenPNE-2.10.7にて確認 webapp/modules/pc/templates/h_review_add_write_confirm.tpl 行149目の、</form>は必要ないと思われる
Change History (18)
comment:1 Changed 13 years ago by
Keywords: | 再現待ち removed |
---|
comment:2 Changed 13 years ago by
Owner: | changed from nobody to nakasone |
---|---|
Status: | new → assigned |
やります。
comment:3 Changed 13 years ago by
Description: | modified (diff) |
---|---|
Keywords: | 確認中 added |
以下のリビジョンで修正しました。
- 2.10.x - r8098
comment:4 Changed 13 years ago by
Milestone: | → OpenPNE2.10.10 |
---|
comment:5 follow-up: 6 Changed 12 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
2.10系のフォームのタグのみ検証を行ってみたところ、formの閉じタグが余計に存在するページが複数存在したので報告します。こちらの修正もお願いします。
formの余計な閉じタグが存在するページ
- browser:OpenPNE/trunk/webapp/modules/pc/templates/c_event_invite_end.tpl
- browser:OpenPNE/trunk/webapp/modules/pc/templates/c_event_write_delete_confirm.tpl
- browser:OpenPNE/trunk/webapp/modules/pc/templates/c_join_commu.tpl
- browser:OpenPNE/trunk/webapp/modules/pc/templates/c_join_err_wait.tpl
- browser:OpenPNE/trunk/webapp/modules/pc/templates/fh_comment_list.tpl
- browser:OpenPNE/trunk/webapp/modules/pc/templates/fh_delete_comment.tpl
- browser:OpenPNE/trunk/webapp/modules/pc/templates/o_public_invite_end.tpl
- browser:OpenPNE/trunk/webapp/modules/admin/templates/delete_topic.tpl
- browser:OpenPNE/trunk/webapp/modules/admin/templates/delete_topic_comment.tpl
- browser:OpenPNE/trunk/webapp/modules/admin/templates/edit_c_profile.tpl
- browser:OpenPNE/trunk/webapp_biz/modules/biz/templates/fh_biz_schedule_calendar.tpl
補足で、閉じタグが多い以外でフォームタグに関係するおかしな部分を発見したので報告します。
comment:6 Changed 12 years ago by
comment:5 ソースのリンクを間違えていました。正確には以下です。
formの余計な閉じタグが存在するページ
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_event_invite_end.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_event_write_delete_confirm.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_join_commu.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/c_join_err_wait.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/fh_comment_list.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/fh_delete_comment.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/o_public_invite_end.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/admin/templates/delete_topic.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/admin/templates/delete_topic_comment.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/admin/templates/edit_c_profile.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp_biz/modules/biz/templates/fh_biz_schedule_calendar.tpl
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/pc/templates/h_com_add_confirm.tpl
- 修正フォームの閉じタグがない
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/ktai/templates/h_diary_edit.tpl
- 日記削除フォームの閉じタグがない
- browser:OpenPNE/branches/stable-2.10.x/webapp/modules/admin/templates/list_c_admin_user.tpl
- 用途不明の開始フォームタグがある
comment:8 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Keywords: | 確認中 added; 差し戻し removed |
以下のリビジョンで修正しました。ご確認ください。
- 2.10.x - r8340
comment:9 Changed 12 years ago by
Description: | modified (diff) |
---|
以下のリビジョンで追加修正しました ご確認ください。
- 2.10.x - r8341
comment:10 Changed 12 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
管理画面や携帯版の修正については 2.10.x に限らず 2.12.x や trunk に対してもおこなうべきではないでしょうか? また修正内容と Summary, Description が一致していない気がします。
comment:11 follow-up: 13 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Keywords: | 確認中 added; 差し戻し removed |
Summary: | formタグの閉じタグが一つ多い → 不要なformタグがある、または必要なformタグがないところがある |
comment:12 Changed 12 years ago by
Keywords: | 差し戻し added; 確認中 removed |
---|
- #3007 との違いを明確にしてください
- Description を最新の状態に変更してください
comment:13 Changed 12 years ago by
comment:14 Changed 12 years ago by
Summary: | 不要なformタグがある、または必要なformタグがないところがある → 不要なformタグがある、または必要なformタグがないところがある(2.10.xのみの修正分) |
---|
comment:15 Changed 12 years ago by
Keywords: | 確認中 added; 差し戻し removed |
---|
Description を最新の状態に変更してください
最新の状態になっています。
comment:16 Changed 12 years ago by
Description: | modified (diff) |
---|---|
Summary: | 不要なformタグがある、または必要なformタグがないところがある(2.10.xのみの修正分) → PC版のテンプレートでform要素の開始タグ・終了タグが対応していない箇所がある |
- Summary を ChangeLog に掲載しても不自然でないように変更しました
- 「原因」が以下のように報告時点のままだった(内容が古かった)のを修正しました
=== ■原因 === formの閉じるタグが多い。
- 存在しないファイルパスを示していたので修正しました
=== ■修正内容 === 以下のファイルの不要な</form>を削除 * browser:webapp/modules/pc/templates/h_review_add_write_confirm.tpl
- 2.10.x に対しての修正であるにもかかわらず、問題の確認できない trunk のファイルパスを記述したままなので修正しました
=== ■修正内容 === 以下のファイルの不要な</form>を削除 (中略) * browser:OpenPNE/trunk/webapp/modules/pc/templates/c_event_invite_end.tpl * browser:OpenPNE/trunk/webapp/modules/pc/templates/c_event_write_delete_confirm.tpl (後略)
comment:17 Changed 12 years ago by
Keywords: | テスト待ち added; 確認中 removed |
---|
comment:18 Changed 12 years ago by
Keywords: | テスト待ち removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
確認しました。問題ありません。
Note: See
TracTickets for help on using
tickets.
該当するテンプレートを確認したところ、149行目に余分な </form> がありました。