Remove duplicated api update shipments controller method

In case no one have noticed or updated it yet. I realized this while looking into #2668. I believe the duplicated method was introduced accidentally in https://github.com/spree/spree/commit/1594fc9