Modify

Opened 10 years ago

Closed 10 years ago

Last modified 7 years 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 10 years ago.

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by Tetsuya Morimoto

comment:1 Changed 10 years ago by Tetsuya Morimoto

comment:2 Changed 10 years ago by Ryan J Ollos

It seems 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.

Version 0, edited 10 years ago by Ryan J Ollos (next)

comment:3 Changed 10 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 10 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 10 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.