Check view policy, not join policy, when raising an error for Phame custom domains
ClosedPublic
Actions

Authored by epriestley on May 5 2014, 3:18 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Sep 10, 1:57 PM

	Unknown Object (File)
	Tue, Sep 10, 1:57 PM

	Unknown Object (File)
	Sat, Sep 7, 6:57 AM

	Unknown Object (File)
	Mon, Sep 2, 9:24 PM

	Unknown Object (File)
	Wed, Aug 21, 11:16 AM

	Unknown Object (File)
	Wed, Aug 21, 4:57 AM

	Unknown Object (File)
	Tue, Aug 20, 8:18 PM

	Unknown Object (File)
	Sun, Aug 18, 8:44 AM

View All Files

Subscribers

epriestley

legien

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rPac9c82fcdf03: Check view policy, not join policy, when raising an error for Phame custom…

Summary

A user in IRC hit this; this looks like a typo.

Test Plan

Created a new blog with a public view policy, a non-public join policy, and a custom domain.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 21305.May 5 2014, 3:18 PM

epriestley retitled this revision from to Check view policy, not join policy, when raising an error for Phame custom domains.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 5 2014, 3:18 PM

legien added a subscriber: legien.May 5 2014, 3:19 PM

Harbormaster completed remote builds in B247: Diff 21305.May 5 2014, 3:20 PM

btrahan accepted this revision.May 5 2014, 4:32 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.May 5 2014, 4:32 PM

Closed by commit rPac9c82fcdf03 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

controller/

blog/

PhameBlogEditController.php

2 lines

Diff 21312

View Options

src/applications/phame/controller/blog/PhameBlogEditController.php

Check view policy, not join policy, when raising an error for Phame custom domainsClosedPublicActions