User Import

[jadamsbit]

I have been using the user import and found out that the role is being set in the database with the role label instead of the machine name. I looked at the code and in the FeedUserProcessor.inc file on line 625 and noticed the entity role was being set with role label. I changed that line to set with the role name and the issue was resolved.

[herbdool]

Thank you! Merged

[argiepiano]

Just a side comment about this PR.

This PR correctly assigns name (instead of the label) as the value to the user's roles array. However, it does so using an associative array with a key that is also the role's name. This key actually is ignored when the user object is saved, so, this PR doesn't do any harm. But I did want to point out that the key is not needed in the User object's roles array. It's an indexed array, not an associative array.

You can actually see how UserStorageController:preSave() converts it to an indexed array:

    // Prepare user roles.
    if (isset($entity->roles)) {
      $entity->roles = array_values(array_filter($entity->roles));
    }

[herbdool]

Good catch @argiepiano. I'm happy to just remove the key in a new commit. Just to have it cleaned up.