hook_user_cancel_methods_alter/7 => #access vs. access

Thanks for filing this! I think you are correct and #access should be access without # there.

It has apparently been fixed in Drupal 8, so this is Drupal 7 only.

Patch to remove it

+++ b/modules/user/user.api.php
@@ -123,7 +123,7 @@ function hook_user_cancel($edit, $account, $method) {
+ *   a method. If access is defined, the method cannot be configured as default
  *   method.

Could we put "the" in here? "... as the default method."

Looks good otherwise!

As you wish :)

Looks good, thanks! Test fails are unrelated.

The infrastructure team is looking into the test bot issue. Please do not hit "retest" yet.

Ok sure, I won't

The opcode problems seem to be fixed. There was a test bot glitch (missing tables in install) in a couple of tests... that seems to be a common test bot glitch, not the new opcode problem... so setting back to RTBC and presumably the auto-testing of RTBC will eventually catch this and test again.

It's a 2-line docs patch so we don't really need the tests beyond "patch applies and doesn't break PHP syntax" anyway.

Committed to 7.x - thanks!