Contact module doesn't implement AccessControlHandler

Thanks, we might have this in contact_storage already.

contact_storage only has the admin permission.

We're not testing that at the moment, not sure how we want to do that.

Maybe we could check for contact message create access instead of contact_form view access on contact.site_page_form?

Failed as expected.

@mbovan: If you upload the test only patch first, then the issue status will not be set back to needs work.

+++ b/core/modules/contact/src/Tests/MessageEntityTest.php
@@ -62,6 +62,15 @@ public function testMessageMethods() {
+    $access_user = $this->createUser(['uid' => 3], array('access site-wide contact form'));
+    $admin = $this->createUser(['uid' => 4], array('administer contact forms'));

Can you make the array format consistent on these lines? We shouldn't mix old form array() and new form [] in the one line. Personal preference is on [], but let you decide ;)

Minor nit, other than that - looks great - thank you for the patch and the test!

Changed array syntax lines 68 and 69 as per comment #10

changed patch name

@dashaforbes: Remember to provide an interdiff when working on an existing patch, had to download and diff myself to make sure that only the requested changes were made. See https://www.drupal.org/documentation/git/interdiff.

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 4f33bd4 and pushed to 8.0.x. Thanks!