Internal function _getInheritedRoleNames() should return role object

[SimonSimCity]

meteor-roles/roles/roles_common.js

Lines 543 to 557 in 32c230d

	_getInheritedRoleNames: function (role) {
	const inheritedRoles = new Set()
	const nestedRoles = new Set([role])
	nestedRoles.forEach(r => {
	const roles = Meteor.roles.find({ _id: { $in: r.children.map(r => r._id) } }, { fields: { children: 1 } }).fetch()
	roles.forEach(r2 => {
	inheritedRoles.add(r2._id)
	nestedRoles.add(r2)
	})
	})
	return [...inheritedRoles]
	},

The function _getInheritedRoleNames() is currently used to construct the array inheritedRoles on a role-assignment, where it only takes over the parameter _id. If you have data stored on the role, wouldn't it be better to have it copied along?

If one of these parameter changes, the programmer extending this package would then be responsible for also keeping this data updated there. If he doesn't need it there, he can extend this package wrapping this function to remove the additional properties.