Differential D17609

Provide a missing feed transaction string for space creation
ClosedPublic
Actions

Authored by epriestley on Apr 4 2017, 12:56 PM.

Details

Reviewers

chad

Maniphest Tasks

T12502: Creating a task in the non-default space causes exception in feed renderer

Commits

rP5e447112184e: Provide a missing feed transaction string for space creation

Summary

Fixes T12502. This transaction probably should not be getting picked for feed rendering, but it currently does get selected in some cases.

This should probably be revisited eventually (e.g., when Maniphest moves to ModularTransactions) but just fix the brokenness for now.

Test Plan

Created a task in a space.
Viewed feed.
Saw the story render with readable text.

Screen Shot 2017-04-04 at 5.54.57 AM.png (163×630 px, 29 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 4 2017, 12:56 PM

Harbormaster completed remote builds in B16280: Diff 42353.Apr 4 2017, 12:57 PM

chad accepted this revision.Apr 4 2017, 5:03 PM

This revision is now accepted and ready to land.Apr 4 2017, 5:03 PM

Closed by commit rP5e447112184e: Provide a missing feed transaction string for space creation (authored by epriestley, committed by epriestley). · Explain WhyApr 4 2017, 5:24 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

transactions/

storage/

PhabricatorApplicationTransaction.php

20 lines

Diff 42361

View Options

src/applications/transactions/storage/PhabricatorApplicationTransaction.php

Provide a missing feed transaction string for space creationClosedPublicActions