fixed role assigning