Invalid field value with devel_generate

I'm troubleshooting similar behavior at #1851766: Error when bulk deleting content.

Printing $node at the top of node_save() shows that $node->field_og_group_ref[und][0][target_id] is set to 'group' rather than a target ID or being blank.

This seems to be coming from _entityreference_devel_generate():

$referencable_entity = entityreference_get_selection_handler($field, $instance)->getReferencableEntities();

Returns an array such as:

array('group' => array(1 => 'boston', 2 => 'nyc'));

if (is_array($referencable_entity) && !empty($referencable_entity)) {
    // Get a random key.
    $object_field['target_id'] = array_rand($referencable_entity);
  }

Sets the target_id to 'group' rather than an entity ID.

With this patch, I'm able to generate nodes within groups successfully.

This patch works nice. Solved my problem

Worked great for me to! Thank you!

Worked for me too! Thanks!

Patch applied fine, although on using devel create content, error still occurs on the entity reference.

PDOException: SQLSTATE[HY000]: General error: 1366 Incorrect integer value:

note: content types w/ node references can be created, only if there are no nodes of the type referencing yet created.

make sure you clear the cache-- the patch fixed the error in #7 for me.

Committed, thanks.

I marked #1916376: Devel Generate error on target id column as a duplicate.

I prefer the patch in that issue to the one committed in this one. It seems like an odd use of a loop:

    foreach ($referencable_entity as $type => $eids) {
      $object_field['target_id'] = array_rand($eids);
    }

I ran into this while trying to use devel_generate with project and project_issue nodes to get us closer to the D7 port of d.o...

Agreed, the committed code is a bit weird (although it works). Here's a new patch that takes some of the code from #1916376: Devel Generate error on target id column, although I renamed $random_entity to $random_bundle to be more accurately self-documenting.

Thanks,
-Derek

Thanks dww ! It is a lot better for websites having a lot of group types.

#12 rtbc ftw

Patch doesn't appear to apply anymore, but I patched it manually and it works just fine. This is definitely RTBC and would love to see it go in soon.

The latest dev version already got this fix in, so you should use it to avoid this bug.

Right. The patch in #3 was committed: see #9. The issue was closed (#10) and @dww reopened it (#12) with an alternative patch. Either way, the issue should be fixed in the next release. (I have not checked the latest dev release, but @tostinni says it contains the fix.)

Using the current dev release ( drush dl entityreference --dev ) I still get an error trying to generate nodes that reference other nodes using entityreference.

Took patch from 12 and manually applied it. This fixed my issue and I can now generate nodes that reference other nodes. As long as there are existing nodes to reference.

patch attached.

Just tested the patch and it works.

Testing with dev build (downloaded today) and it all works as expected.

Confirming entityreference 7.x-1.0+7-dev (2013-Jun-06) and patch in #18 works :)

+1 works

Also confirming Dev fixes issue.

genarate to generate

+1 RTBC
Also related #1822844: Notice: Undefined index: revision_id in field_collection_field_get_entity() (line 1586 of field_collection/field_collection.modu

Can we get this in?

Merged, thanks.