Environment:

 Package           Name                                           Version
 Chaos tool suite  Chaos tools (ctools)                           7.x-1.3
 Fields            Entity Reference (entityreference)             7.x-1.1
 Organic groups    Organic groups (og)                            7.x-2.4
 Organic groups    Organic groups UI (og_ui)                      7.x-2.4
 Other             Entity API (entity)                            7.x-1.2
 Views             Views (views)                                  7.x-3.7
 Views             Views Bulk Operations (views_bulk_operations)  7.x-3.1

Message:

Notice: Undefined index: entity keys in entity_extract_ids() (line 7691 of /var/aegir/platforms/d201312161012/includes/common.inc).
Notice: Undefined index: entity keys in entity_extract_ids() (line 7692 of /var/aegir/platforms/d201312161012/includes/common.inc).

Steps to reproduce:

  1. Create a fresh og installation with Standard D7 profile
  2. Make the basic page content type Group Content
  3. Edit og_group_ref
  4. Save field (no changes necessary)

The problem seems to be the validate function that was expanded in OG 7.x-2.4, specifically the following line:

list(,, $bundle) = entity_extract_ids($entity_type, $entity);

However, $entity_type and $entity are both null. The $bundle variable is not actually used in the validate function so maybe the call to entity_extract_ids can be removed or, if a value for $bundle is required, it can be retrieved from the $instance array.

Comments

eft’s picture

Status: Active » Needs review
StatusFileSize
new625 bytes

Here is a patch the removes the line described above.

Status: Needs review » Needs work

The last submitted patch, 1: og-behaviour-handler-validate-2157837-1.patch, failed testing.

eft’s picture

Version: 7.x-2.4 » 7.x-2.x-dev
Status: Needs work » Needs review
StatusFileSize
new625 bytes

Changing version to test patch against.

shushu’s picture

Status: Needs review » Reviewed & tested by the community
amitaibu’s picture

Status: Reviewed & tested by the community » Fixed

Merged, thanks.

Status: Fixed » Closed (fixed)

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

ergophobe’s picture