fixed Player constructor