David Heinemeier Hansson
139fa7afdc
If you update the name by itself, it is fine. We generally want to be very careful about having txn spans, since all the work done within them will block the sqlite writer.
55 lines
1.2 KiB
Ruby
55 lines
1.2 KiB
Ruby
class BucketsController < ApplicationController
|
|
before_action :set_bucket, except: %i[ index new create ]
|
|
|
|
def index
|
|
@buckets = (Current.user.buckets.all + Current.user.bucket_views.all).sort_by(&:updated_at).reverse!
|
|
end
|
|
|
|
def new
|
|
@bucket = Current.account.buckets.build
|
|
end
|
|
|
|
def create
|
|
@bucket = Current.account.buckets.create! bucket_params
|
|
redirect_to bucket_bubbles_path(@bucket)
|
|
end
|
|
|
|
def edit
|
|
selected_user_ids = @bucket.users.pluck :id
|
|
@selected_users, @unselected_users = User.active.alphabetically.partition { |user| selected_user_ids.include? user.id }
|
|
end
|
|
|
|
def update
|
|
@bucket.update! bucket_params
|
|
@bucket.accesses.revise granted: grantees, revoked: revokees
|
|
|
|
redirect_to bucket_bubbles_path(@bucket)
|
|
end
|
|
|
|
def destroy
|
|
@bucket.destroy
|
|
redirect_to buckets_path
|
|
end
|
|
|
|
private
|
|
def set_bucket
|
|
@bucket = Current.user.buckets.find params[:id]
|
|
end
|
|
|
|
def bucket_params
|
|
params.expect(bucket: [ :name ])
|
|
end
|
|
|
|
def grantees
|
|
Current.account.users.active.where id: grantee_ids
|
|
end
|
|
|
|
def revokees
|
|
@bucket.users.where.not id: grantee_ids
|
|
end
|
|
|
|
def grantee_ids
|
|
params.fetch :user_ids, []
|
|
end
|
|
end
|