Closed
Bug 823634
Opened 12 years ago
Closed 12 years ago
Reason & message of the day strings should have leading and trailing whitespace removed
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: ewong)
References
Details
Attachments
(1 file)
A pretty common pattern is for sheriffs to copy and paste reasons using triple-click on history/wherever and paste, which leads to a trailing space. This then shows up in TBPL due to the appending of the period. We should trim trailing and leading whitespace from both the reason and the message of the day, before saving to the DB.
Whilst we're at it, we could trim off the period (if present) at the end of the reason, so we don't get two appended, once TBPL combines reason and message of the day.
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → ewong
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #732175 -
Flags: review?(catlee)
|
|
||
Updated•12 years ago
|
Attachment #732175 -
Flags: review?(catlee) → review+
|
|
Reporter | |
Comment 2•12 years ago
|
||
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
Updated•12 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
|
|
||
Updated•11 years ago
|
Product: Webtools → Tree Management
|
|
||
Updated•10 years ago
|
Product: Tree Management → Release Engineering
|
|
||
Updated•4 years ago
|
Component: Applications: TreeStatus → General
You need to log in
before you can comment on or make changes to this bug.
Description
•