“Total cash” is your input to the event. And you’re storing to it again?
What is “collected cash?”
If “Collected Cash” is the variable in the player state that stores the total amount of cash, you should store into that.
If the player state variable is called “total cash” then your argument in the event should be called something else, you should not be reading “collected cash.”