ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 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 ebihara)

■現象

PC版のテンプレートにおいて form 要素の開始タグ・終了タグの対応が取れていない箇所がある。

■原因

  • form 要素の終了タグが余分に記述されている箇所がある
  • form 要素の終了タグが不足している（開始タグのみが記述されている）箇所がある

なお、この現象は XHTML+CSS 化がおこなわれていない 2.10.x のみで発生する。

■修正内容

■関連情報

​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 12 years ago by nakasone

Keywords: 再現待ち removed

該当するテンプレートを確認したところ、149行目に余分な　</form>　がありました。

comment:2 Changed 12 years ago by nakasone

Owner: changed from nobody to nakasone
Status: new → assigned

やります。

comment:3 Changed 12 years ago by nakasone

Description: modified (diff)
Keywords: 確認中 added

以下のリビジョンで修正しました。

comment:4 Changed 12 years ago by kiwa

Milestone: → OpenPNE2.10.10

comment:5 Changed 12 years ago by kiwa

Keywords: 差し戻し added; 確認中 removed

2.10系のフォームのタグのみ検証を行ってみたところ、formの閉じタグが余計に存在するページが複数存在したので報告します。こちらの修正もお願いします。

formの余計な閉じタグが存在するページ

補足で、閉じタグが多い以外でフォームタグに関係するおかしな部分を発見したので報告します。

browser:OpenPNE/trunk/webapp/modules/pc/templates/h_com_add_confirm.tpl​
修正フォームの閉じタグがない
browser:OpenPNE/trunk/webapp/modules/ktai/templates/h_diary_edit.tpl​
日記削除フォームの閉じタグがない
browser:OpenPNE/trunk/webapp/modules/admin/templates/list_c_admin_user.tpl​
用途不明の開始フォームタグがある

comment:6 in reply to:  5 Changed 12 years ago by kiwa

comment:5 ソースのリンクを間違えていました。正確には以下です。

formの余計な閉じタグが存在するページ

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:7 Changed 12 years ago by shingo

作業します。

comment:8 Changed 12 years ago by shingo

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:9 Changed 12 years ago by shingo

Description: modified (diff)

以下のリビジョンで追加修正しました ご確認ください。

comment:10 Changed 12 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

管理画面や携帯版の修正については 2.10.x に限らず 2.12.x や trunk に対してもおこなうべきではないでしょうか？　また修正内容と Summary, Description が一致していない気がします。

comment:11 Changed 12 years ago by shingo

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed
Summary: formタグの閉じタグが一つ多い → 不要なformタグがある、または必要なformタグがないところがある

以下のリビジョンで編集前の状態に戻しました。

以下のリビジョンで修正しました。ご確認ください。

comment:12 Changed 12 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • #3007 との違いを明確にしてください
  • Description を最新の状態に変更してください

comment:13 in reply to:  11 Changed 12 years ago by shingo

Replying to shingo:

以下のリビジョンで修正しました。ご確認ください。

上記のように記述しましたが、下記の誤りでした。 申し訳ありません。

comment:14 Changed 12 years ago by shingo

Summary: 不要なformタグがある、または必要なformタグがないところがある → 不要なformタグがある、または必要なformタグがないところがある（2.10.xのみの修正分）

以下のリビジョンで追加修正しました。ご確認ください。

#3007 との違いを明確にしてください

わかりやすいように「2.10.xのみの修正分」とSummaryに補足的に追加

comment:15 Changed 12 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

Description を最新の状態に変更してください

最新の状態になっています。

comment:16 Changed 12 years ago by ebihara

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 ebihara

Keywords: テスト待ち added; 確認中 removed

comment:18 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: → fixed
Status: assigned → closed

確認しました。問題ありません。

Note: See TracTickets for help on using tickets.