Page MenuHomePhabricator

Fix Phame Post creation time
ClosedPublic

Authored by chad on Dec 3 2015, 9:40 PM.

Details

Summary

I spent way to long to arrive at this solution. Ref T9360

Test Plan

Publish a new post, see time, unpublish post, see draft. Start a new post, wait 10 minutes, publish, see "now".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Fix Phame Post creation time.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/phame/storage/PhamePost.php
37

For consistency, prefer PhabricatorTime::getNow() (which allows unit tests to force the "current time" to some specific known time).

This revision is now accepted and ready to land.Dec 3 2015, 9:42 PM
chad edited edge metadata.
chad added a task: T9360: Unbeta Phame.
This revision was automatically updated to reflect the committed changes.