fixed User constructor