Is your variable "onground" true when you press "uiright"?
If "on_ground" is true, I don't see anything wrong, maybe your node is already colliding with another one, and that keeps him from moving?
If you have nodes in your scene that can collide with your character, try disabling them. Just keep the ground and test.