Ticket #8795 (new defect)

Opened 2 years ago

Last modified 8 months ago

ticket number regex fails

Reported by: anonymous Assigned to: olemis
Priority: normal Component: WhiteboardPlugin
Severity: normal Keywords:
Cc: john.tantalo@gmail.com Trac Release: 0.11

Description

The regex to match the ticket number fails for tickets > 9.

Attachments

Change History

05/13/11 06:39:02 changed by anonymous

proposed patch: 

-            "ticket": /(board_ticket_)(\d)/.exec(ui.item.attr("id"))[2],
+            "ticket": /(board_ticket_)(\d+)/.exec(ui.item.attr("id"))[2],

05/13/11 07:00:22 changed by anonymous

  • cc set to john.tantalo@gmail.com.

07/20/11 23:45:44 changed by Poly

This is a nice plugin. It might even get nicer if the people know how to report bugs effectly (ask Google). Too bad this happens too rarely...

Anyhow, the bug submitter is right with his finding. But the regex does not fail it just matches ticket-numbers from 1 to 9. However, only a few software projects have this small bug count. ;-)

The effect of the glitch in the match of the regexp is that the "save changes" does only work if you move tickets with numbers 1 to 9 around in the whiteboard.

The suggested change which fixes this issue is included in the patch for #7633. I propose to close this issue when the patch is in the mainline of the plugin.

10/05/12 18:27:08 changed by rjollos

  • owner changed from CuriousCurmudgeon to olemis.

Add/Change #8795 (ticket number regex fails)




Change Properties
Action