Fix missing transaction constants in Phame
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15500713: D17871.id.diff
	Sun, Apr 13, 7:00 PM

	F15472213: D17871.diff
	Sat, Apr 5, 11:24 AM

	F15456311: D17871.id.diff
	Sun, Mar 30, 8:34 AM

	F15414094: D17871.diff
	Wed, Mar 19, 11:09 PM

	F15396499: D17871.id42984.diff
	Mar 16 2025, 2:48 PM

	F15384449: D17871.id42986.diff
	Mar 14 2025, 7:56 PM

	Unknown Object (File)
	Jan 28 2025, 6:32 PM

	Unknown Object (File)
	Jan 28 2025, 12:01 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