プロジェクト

全般

プロフィール

Bug(バグ) #4054

完了

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

Youichi Kimura さんが約8年前に追加. 4年以上前に更新.

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

100%

予定工数:
(合計: 0:00時間)
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> を追加する


子チケット 1 (0件未完了1件完了)

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

操作

関連するチケット 1 (1件未完了0件完了)

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

操作

kaoru n さんが約8年前に更新

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

kaoru n さんが約8年前に更新

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

kaoru n さんがほぼ8年前に更新

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

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

kaoru n さんが4年以上前に更新

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

kaoru n さんが4年以上前に更新

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

isao sano さんが4年以上前に更新

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

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

isao sano さんが4年以上前に更新

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

kaoru n さんが4年以上前に更新

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

マージしました

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