fixed User controller