Firstly, it should be very rare (i.e. virtually unheard of!) to update the
id (primary key) of a record. This is a record's identifier; by changing this, you'll mess up any associations and potentially cause a conflict with another record's
udpate_something [SIC] method doesn't really achieve anything. It's a wrapper method that serves no purpose. Why not just do:
Your controller method doesn't handle error messages, use strong parameters (!!) or perform any render/redirect action - but that's a separate story. There's no generic "right thing to do" across all controller actions; it really depends on what you're trying to achieve!
I'd suggest looking through some basic material such as the rails documentation to see how controllers are typically structured.