Test access to post revisions #275

Closed
wants to merge 5 commits into
from

Conversation

Projects
None yet
2 participants
Member

danielbachhuber commented Jun 2, 2014

And throw an error if user doesn't have access to a post's revisions #251

Member

danielbachhuber commented Jun 2, 2014

Fixed up, and includes #251

@rmccue rmccue commented on the diff Jun 2, 2014

phpunit.xml.dist
@@ -3,6 +3,7 @@
<!-- Default test suite to run all tests -->
<testsuite>
<directory prefix="test_" suffix=".php">tests</directory>
+ <directory prefix="test-" suffix=".php">tests</directory>
@rmccue

rmccue Jun 2, 2014

Owner

Can we keep the file naming consistent please?

@danielbachhuber

danielbachhuber Jun 2, 2014

Member

WordPress core uses dashes, not underscores, for filenames.

@rmccue

rmccue Jun 2, 2014

Owner

Right, but I'd rather change them all at once (in a separate issue) than be inconsistent.

Owner

rmccue commented Jun 2, 2014

Includes changes from #274 mixed in.

rmccue referenced this pull request Jun 2, 2014

Merged

Add revisions tests #279

Owner

rmccue commented Jun 2, 2014

Fixed in #279.

rmccue closed this Jun 2, 2014

@Japh Japh pushed a commit to Japh/WP-API that referenced this pull request Sep 29, 2014

@rmccue rmccue Add better taxonomy support
Fixes #275 for now.

git-svn-id: http://gsoc.svn.wordpress.org/2013/rmccue/trunk@2351 f69abe3c-2ee2-4550-9a6d-17bd458115be
8555bf6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment