Modify

Opened 3 years ago

Closed 3 years ago

Last modified 10 days ago

#11423 closed defect (fixed)

The components are duplicated in admin page

Reported by: Tetsuya Morimoto Owned by: Ryan J Ollos
Priority: normal Component: DefaultCcPlugin
Severity: normal Keywords:
Cc: Trac Release: 1.0

Description

The components are duplicated on Components page in admin panel when DefaultCCAdmin is enabled.

My environment are:

  • DefaultCc-0.3dev_r13333
  • Trac-1.0.2dev_r12281

It seems that the save process is fine, this bug is only for display in admin page.

141                     filter = filter.append(tag.tr(tag.td(tag.input(type='checkbox', name='sel', value=comp.name), class_='sel'),
142                                                   tag.td(tag.a(comp.name, href=req.href.admin('ticket', 'components') + '/' + comp.name), class_='name'),
143                                                   tag.td(comp.owner, class_='owner'),
144                                                   tag.td(default_tag, class_='default'),
145                                                   tag.td(default_cc, class_='defaultcc')))

Attachments (1)

defaultcc_admin_components1.png (55.6 KB) - added by Tetsuya Morimoto 3 years ago.

Download all attachments as: .zip

Change History (6)

Changed 3 years ago by Tetsuya Morimoto

comment:1 Changed 3 years ago by Tetsuya Morimoto

comment:2 Changed 3 years ago by Ryan J Ollos

It appears this might somehow be due to [trac 11919]. Previously, it seems that exhausting the iterator would cause the table to not be rendered, therefore the original author recreated the entire table in Genshi. After 1.0.2dev-r11919, the original table and recreated table are rendered.

Last edited 3 years ago by Ryan J Ollos (previous) (diff)

comment:3 Changed 3 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

In 13445:

0.3dev: After Trac 1.0.2dev-r11919, duplicate entries would appear in the table. Fixes #11423,

With components having been cast to a list in the template data dictionary, it has been possible to simplify the filter_stream implementation. It is no longer necessary to recreate the entire table, rather we can just append the "defaultcc" data to the table rows.

comment:4 Changed 3 years ago by Ryan J Ollos

I fixed some other issues in [13446:13449]. Please let me know if you spot any problems.

comment:5 Changed 3 years ago by Tetsuya Morimoto

I updated from trunk and confirmed it works well. Thank you for quick response!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.