Increment takes a number to add and then a default value if there’s no data saved to increment upon.
Instead of that default value, you’ve given the player.
Moreover, the DataStore is for that player anyways.
FYI: Don’t use the second argument of Instance.new, it’s quite inefficient - instead set the .Parent of your Instance. You can read more here: