Page MenuHomePhabricator
Differential D18262

Fix an issue when deleting the entire content of an unsubmitted inline comment
ClosedPublic
Actions

Authored by epriestley on Jul 21 2017, 3:10 PM.
Tags
None
Referenced Files
F15466944: D18262.id43918.diff
Thu, Apr 3, 11:04 AM
F15465908: D18262.id43917.diff
Thu, Apr 3, 12:12 AM
F15461382: D18262.id.diff
Tue, Apr 1, 7:55 AM
F15460529: D18262.diff
Mon, Mar 31, 11:37 PM
F15399113: D18262.diff
Mon, Mar 17, 2:56 AM
F15368248: D18262.id43917.diff
Wed, Mar 12, 1:09 AM
Unknown Object (File)
Mar 3 2025, 2:15 AM
Unknown Object (File)
Feb 20 2025, 1:39 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12733: (2017 Week 20) Inline Comments Errata / Feedback
Commits
rP0d8f4170f4dc: Fix an issue when deleting the entire content of an unsubmitted inline comment
Summary

Ref T12733. In this case, the server returns no new row, but we would incorrectly try to bind to one.

Test Plan
  • Created a new comment.
  • Edited it.
  • Deleted all the text.
  • Saved changes.
  • Before: Header continues to show phantom "1 Unsubmitted Comment", browser error log reflects one error.
  • After: Header reflects comment being deleted, error log is quiet.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
resources/
celerity/
12 lines
webroot/
rsrc/
js/
application/
diff/
9 lines

Diff 43918

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/js/application/diff/DiffInline.js

Loading...
Phabricator · Built by Phacility