Modify ↓
Opened 14 years ago
Closed 14 years ago
#8399 closed defect (fixed)
It doesn't work in 0.12.1
Reported by: | Owned by: | Ryan J Ollos | |
---|---|---|---|
Priority: | high | Component: | DuplicateTicketSearchPlugin |
Severity: | critical | Keywords: | regex |
Cc: | Trac Release: | 0.12 |
Description (last modified by )
I have installed trac(0.12.1) and I want to that the plugin work in my trac well. However, It doesn't work, so I see its source, tracDupeSearch.js. In the source, I changed code about regex expression and it is working well. It's below
- var summaryRegex = /#(\d+)<\/span>:\s*([a-z0-9\-\_\ ]+):\s*(.*)\s*\(([a-z0-9\-\_\ ]+)(?:\: ([a-z0-9\-\_\ ]+))?\)$/i; + var summaryRegex = /#(\d+)<\/span>:\s*(.*):\s*(.*)\s*\((.*)(?:\: (.*))?\)$/i;
Attachments (0)
Change History (6)
comment:1 follow-up: 2 Changed 14 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
Owner: | changed from gregmac to Ryan J Ollos |
---|---|
Priority: | normal → high |
Status: | new → assigned |
This might resolve the issue in #7382 as well. I'll apply the patch shortly.
comment:4 Changed 14 years ago by
I don't understand the regex, but it seems to work, so I'm going to go ahead and apply the patch.
comment:5 Changed 14 years ago by
Before and after applying the patch, I created a ticket with summary 한국어/조선말 and I got a subsequent match in both cases. So, I'm not exactly sure why it wasn't working for you and the reporter in #7382, but was working fine for me.
comment:6 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
What language do you use on your Trac installation?