Edgewall Software
Modify

Opened 23 months ago

Closed 22 months ago

Last modified 14 months ago

#13515 closed defect (fixed)

Ticket notification failing with 'list' object has no attribute 'update'

Reported by: Jun Omae Owned by: Jun Omae
Priority: normal Milestone: 1.4.4
Component: ticket system Version: 1.4.3
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Fixed notification failure when permission group is used in other than Cc field.

API Changes:
Internal Changes:

Description

Reported at gmessage:trac-users:FM_1L9QFywQ/m/KPtZ2b0iAQAJ.

How to reproduce

  1. trac-admin /path/to/tracenv
    1. initenv 'My Project' sqlite:db/trac.db
    2. permission add anonymous TICKET_CREATE TICKET_MODIFY
    3. permission add user1 group1
  2. tracd -p 3000 /path/to/tracenv
  3. Create new ticket with owner being group1
Trac[web_ui] ERROR: Failure sending notification on creation of ticket #1: AttributeError: 'list' object has no attribute 'update'

Attachments (0)

Change History (5)

comment:1 by Jun Omae, 23 months ago

Owner: set to Jun Omae
Status: newassigned

After the changes of #12808, permission group is able to use in Cc field. However, the permission group in also reporter, owner and updater is expanded to the members (actually, the AttributeError raises).

Proposed changes in [40a35bb4f/jomae.git] (jomae.git@t13515). In the proposal changes, permission group can be used in only Cc field.

comment:2 by Jun Omae, 22 months ago

Release Notes: modified (diff)
Resolution: fixed
Status: assignedclosed

Pushed in [17654] and merged in [17655].

comment:3 by Ryan J Ollos, 14 months ago

r17650 references this ticket and is shown as eligible for merge to trunk. Is it applicable to trunk?

$ svn mergeinfo --show-revs eligible ^/branches/1.4-stable
r17650

in reply to:  3 comment:4 by Jun Omae, 14 months ago

Replying to Ryan J Ollos:

r17650 references this ticket and is shown as eligible for merge to trunk. Is it applicable to trunk?

Ah, I forget to merge it to trunk. We should apply it to trunk (and 1.6-stable).

comment:5 by Ryan J Ollos, 14 months ago

Merged to 1.6-stable in r17725, to trunk in r17726.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jun Omae.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jun Omae to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.