プロジェクト

全般

プロフィール

Bug(バグ) #4054

管理画面のトピックリストなどの画面でtr要素の閉じタグが不足している

Youichi Kimura7年以上前に追加. 3年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2017-01-16
期日:
進捗率:

100%

3.6 で発生するか:
Unknown (未調査)
[QA]バグ通知済:
いいえ
3.8 で発生するか:
Unknown (未調査)
opCommunityTopicPlugin 絞込条件:

説明

Overview (現象)

管理画面のトピックリスト (/pc_backend.php/communityTopic/topicList) の検索フォームは下記のようにマークアップされている:

<form method="get" action="/master/pc_backend.php/communityTopic/topicList"><table>
<tr>
  <th><label for="communityTopic_id">トピックID</label></th>
  <td><input type="text" id="communityTopic_id" name="communityTopic[id][text]" value="" /><br /> </td>
</tr>
<tr>
  <th><label for="communityTopic_name">トピックタイトル</label></th>
  <td><input type="text" id="communityTopic_name" name="communityTopic[name][text]" value="" /><br /> </td>
</tr>
<tr>
  <th><label for="communityTopic_body">本文</label></th>
  <td><input type="text" id="communityTopic_body" name="communityTopic[body][text]" value="" /><br /> </td>
</tr>
<tr>
<td colspan="2"><input type="submit" value="検索" /></td>
</table>
</form>

この中で、検索ボタンが含まれている最後の列について tr 要素の閉じタグが存在せず、XHTML の構文として正しくない状態になっている。

トピックコメントリストやイベントリストなどにも同様の不具合がある。

Way to fix (修正内容)

</table> の手前の行に </tr> を追加する


子チケット

Backport(バックポート) #4126: 管理画面のトピックリストなどの画面でtr要素の閉じタグが不足しているWon't fix(対応せず)kaoru n


関連するチケット

関連している opCommunityTopicPlugin - Bug(バグ) #2463: 携帯版の communityEvent/templates/showSuccess.php の table 要素のマークアップが不正 New(新規) 2011-10-03

履歴

#1 kaoru n約7年前に更新

  • 対象バージョンmaster にセット

#2 kaoru n約7年前に更新

  • 関連している Bug(バグ) #2463: 携帯版の communityEvent/templates/showSuccess.php の table 要素のマークアップが不正 を追加

#3 kaoru n約7年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者kaoru n にセット
  • 進捗率0 から 50 に変更

https://github.com/tejimaya/opCommunityTopicPlugin/pull/33
にてプルリクエストを行いました。
レビューをお願いします。

#4 kaoru nほぼ4年前に更新

  • 対象バージョンmaster から opCommunityTopicPlugin-1.1.3 に変更

#5 kaoru nほぼ4年前に更新

  • 対象バージョンopCommunityTopicPlugin-1.1.3 から opCommunityTopicPlugin-1.1.4 に変更

#6 isao sano3年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率50 から 80 に変更

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

#7 isao sano3年以上前に更新

  • ステータスPending Merge(マージ待ち) から Pending Review(レビュー待ち) に変更
  • 進捗率80 から 50 に変更

#8 kaoru n3年以上前に更新

  • ステータスPending Review(レビュー待ち) から Fixed(完了) に変更
  • 進捗率50 から 100 に変更

マージしました

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