commiting contoller code