Opened 14 years ago
Last modified 12 years ago
#8795 new defect
ticket number regex fails
| Reported by: | anonymous | Owned by: | Olemis Lang |
|---|---|---|---|
| Priority: | normal | Component: | WhiteboardPlugin |
| Severity: | normal | Keywords: | |
| Cc: | john.tantalo@… | Trac Release: | 0.11 |
Description
The regex to match the ticket number fails for tickets > 9.
Attachments (0)
Change History (4)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
| Cc: | john.tantalo@… added; anonymous removed |
|---|
comment:3 Changed 13 years ago by
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.
comment:4 Changed 12 years ago by
| Owner: | changed from CuriousCurmudgeon to Olemis Lang |
|---|
proposed patch:
- "ticket": /(board_ticket_)(\d)/.exec(ui.item.attr("id"))[2], + "ticket": /(board_ticket_)(\d+)/.exec(ui.item.attr("id"))[2],