fixed player constructor