Set Entity Title to Place Name

At the moment, the module has this line which works for Nodes:
$entity->title = (trimed/cleaned-up version of $place['name']);

I understand that your patch is meant to work for Entities which tend to use $entity->label.

Do you think it would cause any problems to just add $entity->label after $entity->title, or should I do a more extensive check for the correct label property as demonstrated in entity_label() (i.e. ['label callback'] or ['entity keys']['label'])?

label won't work across entity types, i.e. for nodes it is title, for taxonomy terms its name, for comments it's subject, etc

A dynamic approach to finding the correct label machine name is the right method.

Greg, I'm curious, are you using ECK to create custom entity types, or rolling your own via a custom module? I ask because AFAIR the ECK module uses the title field to store entity titles. If you're using a custom module, I'm guessing you're able to specify the machine name of the label...?

I just committed a fix to the dev branch that should detect an entity's "label" property. It seems to work correctly for nodes, taxonomy terms, and custom entities. Would y'all be available to test it?

I made a checkbox on the field instance settings form for "Set title/label" to be optional. I think it's ready for prime-time.