Assuming that $entity->uid exists. Causing notice when using inline entity form [#2412799]

This is very much related to #2229739: undefined property entity_metadata_user_access()

Comment	File	Size	Author
#4	entity-2412799-4.patch	849 bytes	Snehal Brahmbhatt
#4	7.x-1.x: PHP 5.3 & MySQL 5.5, D7 42 pass
#1	entity-assuimg-entity-uid-exists-2412799-1.patch	925 bytes	emattias
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

emattias CreditAttribution: emattias commented 23 January 2015 at 09:53

Status:

Active

» Needs review

File	Size
entity-assuimg-entity-uid-exists-2412799-1.patch	925 bytes

This patch avoids the notice.

I am not 100% sure that this is the correct way to fix this issue. The problem might be that Inline Entity Form (see issue referenced in issue description) creates an entity incorrectly (without ->uid property).

Comment #2

emattias CreditAttribution: emattias commented 23 January 2015 at 09:56

Title:

Assuming that $entity->uid exists. Causing notice n using inline entity form

» Assuming that $entity->uid exists. Causing notice when using inline entity form

Comment #3

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented 20 January 2019 at 21:39

Version:	7.x-1.5	» 7.x-1.x-dev
Status:	Needs review	» Needs work
Issue tags:		+Needs reroll

The 4 year old patch to callbacks.inc does not apply to the latest entity 7.x-1.x-dev and (if still relevant) needs a reroll.

Checking patch modules/callbacks.inc...
error: while searching for:
function entity_metadata_user_access($op, $entity = NULL, $account = NULL, $entity_type) {
  $account = isset($account) ? $account : $GLOBALS['user'];
  // Grant access to the users own user account and to the anonymous one.
  if (isset($entity) && $op != 'delete' && (($entity->uid == $account->uid && $entity->uid) || (!$entity->uid && $op == 'view'))) {
    return TRUE;
  }
  if (user_access('administer users', $account) || user_access('access user profiles', $account) && $op == 'view' && $entity->status) {

error: patch failed: modules/callbacks.inc:706
error: modules/callbacks.inc: patch does not apply