Page MenuHomePhabricator
Differential D18475

Fix fatal on logged out Phame Post
ClosedPublic
Actions

Authored by chad on Aug 25 2017, 3:39 AM.
Tags
None
Referenced Files
F17711917: D18475.diff
Thu, Jul 17, 9:19 AM
Unknown Object (File)
May 8 2025, 4:34 AM
Unknown Object (File)
Apr 30 2025, 4:57 PM
Unknown Object (File)
Apr 29 2025, 11:04 AM
Unknown Object (File)
Apr 28 2025, 5:49 PM
Unknown Object (File)
Apr 27 2025, 8:07 AM
Unknown Object (File)
Apr 22 2025, 5:40 AM
Unknown Object (File)
Mar 14 2025, 10:09 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
rP79c6b5004914: Fix fatal on logged out Phame Post
Summary

Just deletes the view code until I have time to better plan this out, or just not ship.

Test Plan

Visit Phame post on public logged out page, view count doesnt cause transaction fatal.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Aug 25 2017, 3:39 AM
Herald added a subscriber: Korvin. · View Herald TranscriptAug 25 2017, 3:39 AM
Harbormaster completed remote builds in B18170: Diff 44383.Aug 25 2017, 3:40 AM
epriestley accepted this revision.Aug 25 2017, 3:50 AM
This revision is now accepted and ready to land.Aug 25 2017, 3:50 AM
chad planned changes to this revision.Aug 25 2017, 4:33 AM

I'll just remove the transaction in the morning.

chad updated this revision to Diff 44386.Aug 25 2017, 3:31 PM
  • remove phame views
This revision is now accepted and ready to land.Aug 25 2017, 3:31 PM
Harbormaster completed remote builds in B18173: Diff 44386.Aug 25 2017, 3:33 PM
chad edited the summary of this revision. (Show Details)Aug 25 2017, 3:37 PM
chad edited the test plan for this revision. (Show Details)
epriestley accepted this revision.Aug 25 2017, 3:40 PM
Closed by commit rP79c6b5004914: Fix fatal on logged out Phame Post (authored by Chad Little <chad@phacility.com>). · Explain WhyAug 25 2017, 3:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
src/
2 lines
applications/
phame/
controller/
post/
24 lines
editor/
12 lines
storage/
5 lines
xaction/

Diff 44387

View Options

resources/sql/autopatches/20170825.phame.01.post.views.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/phame/controller/post/PhamePostViewController.php

Loading...
View Options

src/applications/phame/editor/PhamePostEditor.php

Loading...
View Options

src/applications/phame/storage/PhamePost.php

Loading...
View Options

src/applications/phame/xaction/PhamePostViewsTransaction.php

Loading...
Phabricator · Built by Phacility