the attached patch adds the $user->access as "last_access" property

Comments

Status: Needs review » Needs work

The last submitted patch, entity_user_last_access.patch, failed testing.

fago’s picture

Status: Needs work » Needs review
+++ b/modules/callbacks.inc
@@ -305,6 +305,9 @@ function entity_metadata_taxonomy_vocabulary_get_properties($vocabulary, array $
+      return empty($account->access) ? NULL : $account->access;

Is the property sometimes FALSE? If so we need that, if it's just NULL or set we could go with the default getter.

fago’s picture

Status: Needs review » Needs work

Tests fails...

@empty. we need it, but let's add a comment like that:

// In case there was no access the value is 0, but we have to return NULL.
dasjo’s picture

Status: Needs work » Needs review
StatusFileSize
new1.67 KB

updated testUserProperties

fago’s picture

Status: Needs review » Fixed

thanks, committed.

Status: Fixed » Closed (fixed)

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