Arvados

Reviewing 23c1bc6

When we're doing small updates, the new update method skips a lot of the encoding work in the ApplicationController's implementation, and then repeats a lot of the rendering code. I think we could help future-proof this by repeating as little as possible, and calling super as early as possible. I think you could do it like this:

 def update
    @updates ||= params[@object.resource_param_name.to_sym]
    if @updates && (@updates.keys - ["name", "description"]).empty?
      # exclude manifest_text since only name or description is being updated
      @object.manifest_text = nil
    end
    super
  end

Let me know what you think of that. Thanks.

Brett, thanks for the suggestion. This is much more efficient and accomplishes the goal more effectively. I made this update.