Ticket #8399 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

It doesn't work in 0.12.1

Reported by: zwei22@gmail.com Assigned to: rjollos
Priority: high Component: DuplicateTicketSearchPlugin
Severity: critical Keywords: regex
Cc: Trac Release: 0.12

Description (Last modified by rjollos)

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

Change History

(follow-up: ↓ 2 ) 01/20/11 06:03:31 changed by rjollos

  • description changed.

What language do you use on your Trac installation?

(in reply to: ↑ 1 ) 01/20/11 06:12:36 changed by zwei22@gmail.com

I use korean in my trac.

Replying to rjollos:

What language do you use on your Trac installation?

01/20/11 07:11:45 changed by rjollos

  • owner changed from gregmac to rjollos.
  • priority changed from normal to high.
  • status changed from new to assigned.

This might resolve the issue in #7382 as well. I'll apply the patch shortly.

01/20/11 23:52:16 changed by rjollos

I don't understand the regex, but it seems to work, so I'm going to go ahead and apply the patch.

01/21/11 00:00:35 changed by rjollos

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.

01/21/11 00:01:22 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [9784]) Modified regex to support unicode characters in match. Fixes #8399.

Add/Change #8399 (It doesn't work in 0.12.1)




Change Properties
Action