Project

General

Profile

Bug(バグ) #4054

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

Added by Youichi Kimura almost 8 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Start date:
2017-01-16
Due date:
% Done:

100%

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

Description

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> を追加する


Subtasks

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


Related issues

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

History

#1 Updated by kaoru n almost 8 years ago

  • Target version set to master

#2 Updated by kaoru n almost 8 years ago

  • Related to Bug(バグ) #2463: 携帯版の communityEvent/templates/showSuccess.php の table 要素のマークアップが不正 added

#3 Updated by kaoru n over 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to kaoru n
  • % Done changed from 0 to 50

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

#4 Updated by kaoru n over 4 years ago

  • Target version changed from master to opCommunityTopicPlugin-1.1.3

#5 Updated by kaoru n over 4 years ago

  • Target version changed from opCommunityTopicPlugin-1.1.3 to opCommunityTopicPlugin-1.1.4

#6 Updated by isao sano over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Merge(マージ待ち)
  • % Done changed from 50 to 80

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

#7 Updated by isao sano over 4 years ago

  • Status changed from Pending Merge(マージ待ち) to Pending Review(レビュー待ち)
  • % Done changed from 80 to 50

#8 Updated by kaoru n over 4 years ago

  • Status changed from Pending Review(レビュー待ち) to Fixed(完了)
  • % Done changed from 50 to 100

マージしました

Also available in: Atom PDF