diff --git a/app/controllers/bubbles_controller.rb b/app/controllers/bubbles_controller.rb index c89f697ca..ce863eb1e 100644 --- a/app/controllers/bubbles_controller.rb +++ b/app/controllers/bubbles_controller.rb @@ -16,35 +16,31 @@ class BubblesController < ApplicationController @bubble = Bubble.new end - def edit - end - - def update - @bubble.update(bubble_params) - remove_image if params[:remove_image] == "true" - - redirect_to bubble_path(@bubble) - end - def create - @bubble = Bubble.create! bubble_params - - redirect_to bubble_path(@bubble) + @bubble = Bubble.create!(bubble_params) + redirect_to @bubble end def show end + def edit + end + + def update + @bubble.update!(bubble_params) + @bubble.image.purge_later if params.key?(:remove_image) + + redirect_to @bubble + end + + private def set_bubble @bubble = Bubble.find(params[:id]) end def bubble_params - params.require(:bubble).permit(:title, :body, :color, :due_on, :image, :remove_image, tag_ids: []) - end - - def remove_image - @bubble.image.purge + params.require(:bubble).permit(:title, :body, :color, :due_on, :image, tag_ids: []) end end diff --git a/app/views/bubbles/_image.html.erb b/app/views/bubbles/_image.html.erb index 86d36b370..70de42639 100644 --- a/app/views/bubbles/_image.html.erb +++ b/app/views/bubbles/_image.html.erb @@ -7,9 +7,8 @@ <% if bubble.image.attached? %> <%= tag.label class: "btn btn--plain", style: "--btn-icon-size: 3em; margin-block-start: -0.5em; margin-inline-end: -0.2em", data: { action: "click->upload-preview#clear click->form#submit", upload_preview_target: "button" } do %> - <%= form.hidden_field :bubble, value: bubble.id %> - <%= check_box_tag "remove_image", "true" %> - Remove imagee + <%= check_box_tag :remove_image, true %> + Remove image <% end %> <% else %>