Refactor memberships query in users controller

[sauloperez]

Now that the user's query also fetches their membership and account, as shown below:

timeoverflow/app/controllers/users_controller.rb

Lines 8 to 14 in 6660a79

	@users = @search
	.result(distinct: false)
	.joins(members: :account)
	.eager_load(members: :account)
	.where(members: { organization: current_organization.id })
	.page(params[:page])
	.per(25)

There's no point in fetching the memberships and building a Hash associating the user and his membership, as shown below:

timeoverflow/app/controllers/users_controller.rb

Lines 16 to 20 in 6660a79

	@memberships = current_organization.members.
	where(user_id: @users.map(&:id)).
	includes(:account).each_with_object({}) do |mem, ob|
	ob[mem.user_id] = mem
	end

We already have that information. We simply need to access it directly from each of the user records returned by the first query.

[markets]

@sauloperez I'm going to close this one, as it has been refactored ✅ in #351 (now it relies on the decorated member instead of user + membership).