Modify

Opened 11 years ago

Closed 11 years ago

#166 closed enhancement (fixed)

Propose using '&' instead of comma as field separator on subject line

Reported by: Dennis McRitchie <dmcr@…> Owned by: bas
Priority: minor Milestone: Release 1.1.0
Component: email2trac Version: 1.0.0
Keywords: Cc:

Description

Currently, fields on subject lines are separated by commas. e.g.,

Subject: This is a test ticket#?priority=minor,cc=knight,component=component1

This poses a problem when a keyword's value is itself a comma-separated list such as the 'cc' field. e.g.,

Subject: This is a test ticket#?priority=blocker,cc=knight,dmcr,component=component2

However, using '&' instead of comma as a field separator allows such specifications, and is consistent with standard web usage. e.g.,

This is a test ticket#?priority=blocker&cc=knight,dmcr&component=component2

The only change needed is on email2trac line 524:

fields = string.split(str, '&')

Dennis

Attachments (0)

Change History (2)

comment:1 Changed 11 years ago by bas

  • Milestone set to Release 1.1.0
  • Status changed from new to assigned

I adjusted the source and introduced a new parameter:

  • subject_field_separator

The default value is & as you suggested and i completely agree with it. But for backwards compatibility i made it an option so people can change it to the old value.

comment:2 Changed 11 years ago by bas

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [297]) added a new parameter subject_field_separator default value is &, closes #166

Add Comment

Modify Ticket

Change Properties
Action
as closed The owner will remain bas.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.