Entity class delete function does not delete entities

Entity:identifier() and entity_id() are used throughout the entity module to gather the ID representing the entity. However, these functions actually return the name of the entity if it is defined. What really needs to be used in these scenarios is Entity:internalIdentifier(). An example of this is the Entity:delete() function:

  public function delete() {
    $id = $this->identifier(); // << This is not an id, it is a name property
    if (isset($id)) {
      entity_get_controller($this->entityType)->delete(array($id));
    }
  }

This should be using the internalIdentifier() as follows:

  public function delete() {
    $id = $this->internalIdentifier();
    if (isset($id)) {
      entity_get_controller($this->entityType)->delete(array($id));
    }
  }

Anywhere entity_id(), Entity:identifier() and Entity:internalIdentifier() are called in the module need attention. A patch has been uploaded with recommended changes.