Missing docblocks from these three functions.

function user_is_anonymous() {
  // Menu administrators can see items for anonymous when administering.
  return !$GLOBALS['user']->uid || !empty($GLOBALS['menu_admin']);
}

function user_is_logged_in() {
  return (bool) $GLOBALS['user']->uid;
}

function user_register_access() {
  return user_is_anonymous() && (config('user.settings')->get('register') != USER_REGISTER_ADMINISTRATORS_ONLY);
}
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

larowlan’s picture

Status: Active » Needs review
FileSize
1.13 KB

straight forward

Status: Needs review » Needs work

The last submitted patch, user-module-docblocks-1864216.1.patch, failed testing.

larowlan’s picture

Status: Needs work » Needs review
tim-e’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me.

jhodgdon’s picture

Status: Reviewed & tested by the community » Needs work

This patch does not follow our docs standards.
http://drupal.org/node/1354#functions

larowlan’s picture

Assigned: Unassigned » jhodgdon

hi @jhodgdon, I can't see what I've missed, unless it needs to be Determines instead of Determine.
Please advise?

jhodgdon’s picture

Assigned: jhodgdon » Unassigned

Correct. all the verb tenses are wrong.

larowlan’s picture

Status: Needs work » Needs review
FileSize
1.13 KB
1000 bytes

Thanks @jhodgdon, fixed in attached.

Status: Needs review » Needs work

The last submitted patch, user-module-docblocks-1864216.8.patch, failed testing.

jhodgdon’s picture

Status: Needs work » Reviewed & tested by the community

Thanks! That looks fine. The test failure is obviously unrelated (and it's one that is already reported as a critical bug, the "deadlock" error on merge queries). I'll hit retest just in case.

jhodgdon’s picture

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

jhodgdon’s picture

Version: 8.x-dev » 7.x-dev
Status: Fixed » Patch (to be ported)

Looks like we should backport this to 7.x, which is also missing the docs on at least some of the same functions.

marvin_B8’s picture

Status: Patch (to be ported) » Needs review
FileSize
1.17 KB
larowlan’s picture

Status: Needs review » Reviewed & tested by the community

straight forward backport

David_Rothstein’s picture

David_Rothstein’s picture

(pardon the noise - just checking something with the testbot)

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.