Closed Bug 916633 Opened 11 years ago Closed 11 years ago

join_activity_entries doesn't reconstitute text with commas correctly.

Categories

(Bugzilla :: Bugzilla-General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: mail, Assigned: mail)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch v1 patch (obsolete) — Splinter Review
If you update a text field with the following "Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo, consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum."

in the history and e-mail notification, there will be spaced before the commas. This is because of the way that find_wrap_point works. This patch should fix that problem.

  -- simon
Assignee: general → sgreen
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 4.4
Attachment #805105 - Flags: review?(gerv)
Comment on attachment 805105 [details] [diff] [review]
v1 patch

>     # All other fields get a space
>-    if (substr($new_change, 0, 1) eq ' ') {
>+    if (substr($new_change, 0, 1) eq ',' || substr($new_change, 0, 1) eq ' ') {

The comment explicitly says that *all* other fields get a space. So either the comment is wrong or the patch is incorrect. In both cases must something be fixed.
Attachment #805105 - Flags: review?(gerv) → review-
Attached patch v2 patchSplinter Review
Fixed the comment.
Attachment #805105 - Attachment is obsolete: true
Attachment #805111 - Flags: review?(gerv)
Attachment #805111 - Flags: review?(gerv) → review?(glob)
Comment on attachment 805111 [details] [diff] [review]
v2 patch

r=glob
Attachment #805111 - Flags: review?(glob) → review+
Flags: approval?
Flags: approval4.4?
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/4.4/                         
modified Bugzilla/Util.pm
Committed revision 8634.                                                       

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Util.pm
Committed revision 8808.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: