Node wrapper unable to set() author to anonymous user

Test case:

  // test via setting author directly
  $node = entity_create('node', array('uid'=>1, 'title'=>'test', 'type' => 'page'));
  node_save($node);
  $node_wrapper = entity_metadata_wrapper('node', $node->nid);
  $node_wrapper->author = 0;
  $node_wrapper->save();
  $node = node_load($node->nid, null, true);
  print('Node uid:' . $node->uid); //should print 0 but prints 2

  //test via set()
  $node = entity_create('node', array('uid'=>1, 'title'=>'test', 'type' => 'page'));
  node_save($node);
  $node_wrapper = entity_metadata_wrapper('node', $node->nid);
  $node_wrapper->author->set(0);
  $node_wrapper->save();
  $node = node_load($node->nid, null, true);
  print('Node uid:' . $node->uid); //should print 0 but prints 2

I don't think it's critical as it doesn't render the module unusable, but anyway - good catch, committed thanks.

This patch is causing some problems for me. I'm using the Relation
module with the patch in #1958298: Relation property on entity_metadata_wrapper().

I have a Treatment entity that is related to a Profile through a treatment_has_carer Relation.
The Relation has a field that is set to 0 when the status of the Treatment changes:

      { "LOOP" : {
          "USING" : { "list" : [ "treatment:relation-treatment-has-carer" ] },
          "ITEM" : { "has_carer_relation" : "has_carer relation" },
          "DO" : [
            { "data_set" : { "data" : [ "has-carer-relation:field-actief" ], "value" : 0 } },

With the patch, this no longer works. For example, I have treatment 56 that has relation 95:

% drush ev '$t=treatment_load(56); $w= entity_metadata_wrapper("treatment", $t); $w->relation_treatment_has_carer[0]->field_actief->set(0);'
WD php: EntityMetadataWrapperException: Entity property relation_treatment_has_carer doesn't support writing.

It seems that in EntityDrupalWrapper::set(), the $previous_id is 95,
but $this->id is "95". So it now calls updateParent(), which fails. If
I add a setter callback to the property info in #1958298: Relation property on entity_metadata_wrapper() it
succeeds, but this property should be read-only as it's not a 'real'
property.

The patch should fix it.

reroll.