Fix missing transaction constants in Phame
ClosedPublic
Actions

Authored by chad on May 12 2017, 4:32 AM.

Tags

None

Referenced Files

	F18733592: D17871.diff
	Tue, Sep 30, 10:37 PM

	F18679192: D17871.id.diff
	Fri, Sep 26, 12:33 AM

	F18618457: D17871.diff
	Mon, Sep 15, 12:11 AM

	F18603992: D17871.diff
	Sat, Sep 13, 5:49 PM

	F18599728: D17871.id.diff
	Sat, Sep 13, 8:05 AM

	F18591643: D17871.id.diff
	Fri, Sep 12, 9:45 AM

	F18591024: D17871.id42984.diff
	Fri, Sep 12, 8:00 AM

	F18587301: D17871.diff
	Thu, Sep 11, 8:58 PM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley
amckinley

Commits

rP7aeb498565a9: Fix missing transaction constants in Phame

Summary

Trailing logs for errors here, picked up some unset constants in Phame.

Test Plan

New Blog, New Post, tail daemon log for errors. See feed stories.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.May 12 2017, 4:32 AM

Harbormaster completed remote builds in B16908: Diff 42984.May 12 2017, 4:32 AM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 12 2017, 4:32 AM

chad retitled this revision from Fix missing transaction contants in Phame to Fix missing transaction constants in Phame.May 12 2017, 4:33 AM

Harbormaster completed remote builds in B16908: Diff 42984.May 12 2017, 4:33 AM

I feel like a linter should catch these.

epriestley accepted this revision.May 12 2017, 6:32 PM

This revision is now accepted and ready to land.May 12 2017, 6:32 PM

Closed by commit rP7aeb498565a9: Fix missing transaction constants in Phame (authored by chad, committed by chad). · Explain WhyMay 12 2017, 7:04 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

editor/

PhamePostEditor.php

2 lines

storage/

PhameBlogTransaction.php

16 lines

Diff 42986

View Options

src/applications/phame/editor/PhamePostEditor.php