add ability to use Grape::Entity documentation in the params block

[Nerian]

You can use entity documentation in the params block

class User::Entity < Grape::Entity
  expose :id,          documentation: { type: Integer}
  expose :name,        documentation: { type: String, desc: "Name" }
  expose :email,       documentation: { type: String, desc: "Email address" }
end

class Users < Grape::API
  desc 'Create user'
  params do
    requires :all, except: [:name], using: User::Entity.documentation.except(:id)
  end
  post 'users' do
    # implement create user
  end
end

[dblock]

This is getting a bit dense. Maybe try to refactor this into a function or two? Same for the else block.

[dblock]

This looks very good. Also need a CHANGELOG update, please. Thx.

[Nerian]

Cool, changes done :)

[Nerian]

I created a new pull request (can't change repo/branch a pull request):

#562

[reynardmh]

Sorry for the delay finishing the test. It should be ready to merge now.
Note: To avoid dependency with Grape::Entity, I wrote the test using hash to define the entity doc. This should provide more flexibility for people who don't use Grape::Entity.

[dblock]

Thanks! Merged via f5d7306. Can you please double-check it, I rewrote the README section to be a bit simpler.

[reynardmh]

Thanks!
Did you change Array(opts[:except]) back to opts[:except].is_a?(Array) ? opts[:except] : [opts[:except]].compact, or my refactoring reynardmh@3805c79 didn't make it to the pull request?
It's better to use Array(opts[:except])right?

[dblock]

Check master. If anything is missing, sorry about that, and please make another PR.

[dblock]

Def Array() is better.

[kuraga]

Is using a bad name for documentation (i.e.) option, isn't it?

[dblock]

Yes.