Page MenuHomePhabricator

Fix a couple of other missing getApplicationTransactionCommentObject() implementations
ClosedPublic

Authored by epriestley on Oct 24 2017, 12:59 PM.
Tags
None
Referenced Files
F14406483: D18729.id44958.diff
Mon, Dec 23, 2:39 PM
Unknown Object (File)
Thu, Dec 19, 12:47 AM
Unknown Object (File)
Tue, Dec 17, 9:27 PM
Unknown Object (File)
Nov 17 2024, 9:45 PM
Unknown Object (File)
Nov 13 2024, 4:40 PM
Unknown Object (File)
Nov 13 2024, 3:30 AM
Unknown Object (File)
Nov 9 2024, 11:52 PM
Unknown Object (File)
Nov 5 2024, 9:59 PM
Subscribers
None

Details

Summary

See PHI165. See D18715. These objects (projects, blogs) also need implementations now.

(I thought about making this method abstract or doing try/catch to maybe make this more robust, but I think this should be the end of it, and those changes have mild complexity/compatibility/risk issues.)

Test Plan

Changed bin/search index to index only one document of each type, ran bin/search index --all --force, saw no more comment-related errors.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster completed remote builds in B18738: Diff 44958.
This revision is now accepted and ready to land.Oct 24 2017, 3:44 PM
This revision was automatically updated to reflect the committed changes.