Add multi-reply feature

app/controllers/replies_controller.rb

@@ -3,6 +3,7 @@
 
 class RepliesController < WritableController
   before_action :login_required, except: [:search, :show, :history]
+  before_action :get_multi_replies, only: [:create, :update]
   before_action :find_model, only: [:show, :history, :edit, :update, :destroy]
   before_action :editor_setup, only: [:edit]
   before_action :require_create_permission, only: [:create]
@@ -116,7 +117,19 @@ def create
       redirect_to post_path(post_id, page: :unread, anchor: :unread) and return
     elsif params[:button_preview]
       draft = make_draft
-      preview(ReplyDraft.reply_from_draft(draft)) and return
+      preview_reply(ReplyDraft.reply_from_draft(draft)) and return
+    elsif params[:button_submit_previewed_multi_reply]
+      post_replies
+      return
+    elsif params[:button_discard_multi_reply]
+      flash[:success] = "Replies discarded."
+      if @multi_replies_json.present? && (editing_reply_id = @multi_replies_json.first["id"]).present?
+        # Editing multi reply, going to redirect back to the reply I'm editing
+        redirect_to reply_path(editing_reply_id, anchor: "reply-#{editing_reply_id}") and return
+      else
+        # Posting a new multi reply, go back to unread
+        redirect_to post_path(params[:reply][:post_id], page: :unread, anchor: :unread) and return
+      end
     end
 
     reply = Reply.new(permitted_params)
@@ -139,10 +152,10 @@ def create
           flash.now[:error] = "This looks like a duplicate. Did you attempt to post this twice? Please resubmit if this was intentional."
           @allow_dupe = true
           if most_recent_unseen_reply.nil? || (most_recent_unseen_reply.id == last_by_user.id && @unseen_replies.count == 1)
-            preview(reply)
+            preview_reply(reply)
           else
             draft = make_draft(false)
-            preview(ReplyDraft.reply_from_draft(draft))
+            preview_reply(ReplyDraft.reply_from_draft(draft))
           end
           return
         end
@@ -154,21 +167,17 @@ def create
         pluraled = num > 1 ? "have been #{num} new replies" : "has been 1 new reply"
         flash.now[:error] = "There #{pluraled} since you last viewed this post."
         draft = make_draft
-        preview(ReplyDraft.reply_from_draft(draft)) and return
+        preview_reply(ReplyDraft.reply_from_draft(draft)) and return
       end
     end
 
-    begin
-      reply.save!
-    rescue ActiveRecord::RecordInvalid => e
-      render_errors(reply, action: 'created', now: true, err: e)
-
-      redirect_to posts_path and return unless reply.post
-      redirect_to post_path(reply.post)
-    else
-      flash[:success] = "Reply posted."
-      redirect_to reply_path(reply, anchor: "reply-#{reply.id}")
+    if params[:button_add_more]
+      # If they click "Add More", fetch the existing array of multi replies if present and add the current permitted_params to that list
+      add_to_multi_reply(reply, permitted_params)
+      return
     end
+
+    post_replies(new_reply: reply)
   end
 
   def show
@@ -187,26 +196,20 @@ def edit
   def update
     @reply.assign_attributes(permitted_params)
     process_npc(@reply, permitted_character_params)
-    preview(@reply) and return if params[:button_preview]
+    if params[:button_preview]
+      preview_reply(@reply) and return
+    elsif params[:button_add_more]
+      # This will take us to reply create instead, but this reply's ID will be saved
+      add_to_multi_reply(@reply, permitted_params) and return
+    end
 
     if current_user.id != @reply.user_id && @reply.audit_comment.blank?
       flash[:error] = "You must provide a reason for your moderator edit."
       editor_setup
       render :edit and return
     end
 
-    begin
-      @reply.save!
-    rescue ActiveRecord::RecordInvalid => e
-      render_errors(@reply, action: 'updated', now: true, err: e)
-
-      @audits = { @reply.id => @post.audits.count }
-      editor_setup
-      render :edit
-    else
-      flash[:success] = "Reply updated."
-      redirect_to reply_path(@reply, anchor: "reply-#{@reply.id}")
-    end
+    edit_reply
   end
 
   def destroy
@@ -297,19 +300,147 @@ def require_edit_permission
     redirect_to post_path(@reply.post)
   end
 
-  def preview(written)
-    @written = written
-    @post = @written.post
-    @written.user = current_user unless @written.user
-    @audits = @written.id.present? ? { @written.id => @written.audits.count } : {}
+  def get_multi_replies
+    @multi_replies_json = JSON.parse(params.fetch(:multi_replies_json, "[]"))
+    @multi_replies = @multi_replies_json.map do |reply_json|
+      Reply.new(permitted_params(ActionController::Parameters.new({ reply: reply_json })))
+        .tap { |r| r.user = current_user }
+    end
+  end
+
+  def preview_reply(reply)
+    preview_replies(reply_to_preview: reply)
+  end
+
+  def add_to_multi_reply(reply, reply_params)
+    post_id = params[:reply][:post_id]
+    ReplyDraft.draft_for(post_id, current_user.id).try(:destroy)
+
+    if reply.character&.new_record?
+      if reply.character.save
+        flash[:success] = "Your new NPC has been persisted!"
+        params[:reply][:character_id] = reply.character.id
+        reply_params[:character_id] = reply.character.id
+        reply.character_id = reply.character.id
+      else
+        flash[:error] = "There was a problem persisting your new NPC."
+      end
+    end
+
+    preview_replies(multi_reply_to_add: reply, multi_reply_params: reply_params)
+  end
+
+  def preview_replies(reply_to_preview: nil, multi_reply_to_add: nil, multi_reply_params: nil)
+    if reply_to_preview
+      # Previewing a specific reply
+      @post = reply_to_preview.post
+      @written = reply_to_preview
+      @written.user = current_user unless @written.user
+      @audits = @written.id.present? ? { @written.id => @written.audits.count } : {}
+      @reply = @written # So that editor_setup shows the correct characters
+    else
+      # Adding a new reply to a multi reply
+      @post = multi_reply_to_add.post
+      empty_reply_hash = permitted_params.permit(:character_id, :icon_id, :character_alias_id)
+      @empty_written = @post.build_new_reply_for(current_user, empty_reply_hash)
+      @empty_written.editor_mode ||= params[:editor_mode] || current_user.default_editor
+      @reply = @empty_written # So that editor_setup shows the correct characters
+      @audits = {}
+    end
+
+    if multi_reply_to_add
+      # Adding a new reply to the multi replies
+      multi_reply_to_add.user = current_user unless multi_reply_to_add.user
+      @multi_replies << multi_reply_to_add
+
+      multi_reply_params[:id] = multi_reply_to_add.id if multi_reply_to_add.id.present?
+      @multi_replies_json << multi_reply_params
+    end
 
     @page_title = @post.subject
 
-    @reply = @written # So that editor_setup shows the correct characters
     editor_setup
     render :preview
   end
 
+  def post_replies(new_reply: nil)
+    @multi_replies << new_reply if new_reply.present?
+
+    if @multi_replies_json.present? && (@reply = Reply.find_by_id(@multi_replies_json.first["id"]))
+      # The first reply of the multi replies has an ID, that means I'm editing rather than posting a new one
+      original_reply_params = permitted_params(ActionController::Parameters.new({ reply: @reply.attributes }))
+      @reply.assign_attributes(permitted_params(ActionController::Parameters.new({ reply: @multi_replies_json.shift })))
+      @multi_replies.shift
+      edit_reply(original_reply_params: original_reply_params)
+      return
+    end
+
+    first_reply = @multi_replies.first
+    begin
+      Reply.transaction { @multi_replies.each(&:save!) }
+    rescue ActiveRecord::RecordInvalid => e
+      errored_reply = @multi_replies.detect { |r| r.errors.present? } || first_reply
+      render_errors(errored_reply, action: 'created', now: true, err: e)
+
+      redirect_to posts_path and return unless errored_reply.post
+      redirect_to post_path(errored_reply.post)
+      return
+    end
+
+    if @multi_replies.length == 1
+      flash[:success] = "Reply posted."
+    else
+      flash[:success] = "Replies posted."
+    end
+    redirect_to reply_path(first_reply, anchor: "reply-#{first_reply.id}")
+  end
+
+  def edit_reply(original_reply_params: nil)
+    begin
+      Reply.transaction do
+        if @multi_replies.present?
+          # Add new replies after the current one and before the next
+
+          # Reorder the replies after this one
+          original_reply_order = @reply.order
+          num_new_replies = @multi_replies.length
+          following_replies = @reply.post.replies.where("reply_order > ?", original_reply_order)
+          following_replies.update_all("reply_order = reply_order + #{num_new_replies}") # rubocop:disable Rails/SkipsModelValidations
+
+          # Create the new replies
+          @multi_replies.each_with_index do |r, idx|
+            # Create a fake temporary reply with the contents of the original one to be in history
+            new_reply_params = permitted_params(ActionController::Parameters.new({ reply: r.attributes }))
+
+            r.assign_attributes(original_reply_params)
+            r.user = current_user
+            r.created_at = @reply.created_at
+            r.order = original_reply_order + idx + 1
+            r.save!
+
+            r.update!(new_reply_params)
+          end
+        end
+
+        @reply.save!
+      end
+    rescue ActiveRecord::RecordInvalid => e
+      if @multi_replies.blank?
+        render_errors(@reply, action: 'updated', now: true, err: e)
+      else
+        errored_reply = @multi_replies.detect { |r| r.errors.present? } || @reply
+        render_errors(errored_reply, action: 'updated', now: true, err: e)
+      end
+
+      @audits = { @reply.id => @post.audits.count }
+      editor_setup
+      render :edit
+    else
+      flash[:success] = "Reply updated."
+      redirect_to reply_path(@reply, anchor: "reply-#{@reply.id}")
+    end
+  end
+
   def make_draft(show_message=true)
     if (draft = ReplyDraft.draft_for(params[:reply][:post_id], current_user.id))
       draft.assign_attributes(permitted_params)

app/views/posts/_write.haml

@@ -29,7 +29,7 @@
         = fields_for :character, post.character do |cf|
           = cf.hidden_field :name, id: 'character_name'
           = cf.hidden_field :npc, id: 'character_npc'
-      = render 'posts/editor_help', editor_location: 'post'
+      = render 'writable/editor_help', editor_location: 'post'
   - content_for :buttons do
     - if post.id.present? && !post.editable_by?(current_user)
       = submit_tag 'Save', class: 'button', id: 'submit_button', data: { disable_with: 'Saving...' }
@@ -90,4 +90,4 @@
     #post-editor.padding-10= yield :form
     .subber.centered= yield :buttons
   - else
-    = render 'editor', item: post
+    = render 'writable/editor', writable: post

app/views/replies/_write.haml

@@ -1,4 +1,4 @@
--# locals: ( reply: )
+-# locals: ( reply:, multi_replies_json: nil )
 
 - content_for :form do
   = form_for reply, html: { id: 'post_form' } do |f|
@@ -10,6 +10,8 @@
     %br
     = unread_warning.try(:html_safe)
     %br
+    - unless multi_replies_json.nil?
+      = hidden_field_tag :multi_replies_json, multi_replies_json.to_json
     = f.hidden_field :post_id
     = f.hidden_field :character_id
     = f.hidden_field :icon_id
@@ -28,12 +30,19 @@
         %br
         = f.text_area :audit_comment, placeholder: 'Explain reason for moderation here', class: 'mod'
         %br
-      = submit_tag (reply.new_record? ? 'Post' : 'Save'), class: 'button', id: 'submit_button', data: { disable_with: 'Saving...' }
-      = submit_tag "Preview", class: 'button', id: 'preview_button', name: 'button_preview', data: { disable_with: 'Previewing...' }
-      - if reply.new_record?
-        = submit_tag "Save Draft", class: 'button', id: 'draft_button', data: { disable_with: 'Drafting...' }, name: 'button_draft'
-      - if @draft.present?
-        = submit_tag "Delete Draft", class: 'button', id: 'delete_draft_button', data: { disable_with: 'Deleting...', confirm: "Delete draft?" }, name: 'button_delete_draft'
+      - if multi_replies_json.blank?
+        = submit_tag (reply.new_record? ? 'Post' : 'Save'), class: 'button', id: 'submit_button', data: { disable_with: 'Saving...' }
+        = submit_tag "Preview", class: 'button', id: 'preview_button', name: 'button_preview', data: { disable_with: 'Previewing...' }
+        - if reply.new_record?
+          = submit_tag "Save Draft", class: 'button', id: 'draft_button', name: 'button_draft', data: { disable_with: 'Drafting...' }
+        - if @draft.present?
+          = submit_tag "Delete Draft", class: 'button', id: 'delete_draft_button', name: 'button_delete_draft', data: { disable_with: 'Deleting...', confirm: "Delete draft?" }
+      - else
+        = submit_tag (multi_replies_json.first["id"].blank? ? 'Post All' : 'Save All'), class: 'button', id: 'submit_button', data: { disable_with: 'Saving...' }
+        = submit_tag "Preview Current", class: 'button', id: 'preview_button', name: 'button_preview', data: { disable_with: 'Previewing...' }
+      = submit_tag "Add More Replies", class: 'button', id: 'add_more_button', name: 'button_add_more', data: { disable_with: 'Adding...' }
+      - if multi_replies_json.present?
+        = submit_tag "Discard Replies", class: 'button', id: 'discard_multi_reply_button', name: 'button_discard_multi_reply', data: { disable_with: 'Discarding...', confirm: "Discard all replies added?"}
     - if reply.post.author_for(current_user).present?
       %hr.clear
       .post-note-editor
@@ -61,6 +70,6 @@
           %p What would you like to do with your unsaved changes?
     - else
       %br
-    = render 'posts/editor_help', editor_location: 'reply'
+    = render 'writable/editor_help', editor_location: 'reply'
 
-= render 'posts/editor', item: reply
+= render 'writable/editor', writable: reply

app/views/replies/preview.haml

@@ -10,8 +10,22 @@
     .post-ender ... and #{@unseen_replies.total_entries - @unseen_replies.per_page} more ...
   %br
 
-.content-header= @post.subject
-- if @post.description.present?
-  .post-subheader= sanitize_simple_link_text(@post.description)
-= render 'replies/single', reply: @written, hide_footer: true
-= render 'write', reply: @written
+- if @multi_replies.present?
+  .content-header Multi reply
+  - @multi_replies.each_with_index do |reply, index|
+    = render 'replies/single', reply: reply, hide_footer: true
+  = form_for @written || @empty_written, html: { id: 'post_form' } do |f|
+    = hidden_field_tag :multi_replies_json, @multi_replies_json.to_json
+    = submit_tag 'Post Previewed', class: 'button', id: 'submit_previewed_multi_reply_button', name: 'button_submit_previewed_multi_reply', data: { disable_with: 'Saving...', confirm: 'The reply currently in the editor will not be posted. Continue?' }
+    %br
+    %br
+
+- if @written.present?
+  - if @multi_replies.present?
+    .content-header Previewing
+  - else
+    .content-header= @post.subject
+    - if @post.description.present?
+      .post-subheader= sanitize_simple_link_text(@post.description)
+  = render 'replies/single', reply: @written, hide_footer: true
+= render 'write', reply: @written || @empty_written, multi_replies_json: @multi_replies_json

app/views/users/profile_edit.haml

@@ -23,7 +23,7 @@
               = f.hidden_field :profile_editor_mode, id: 'profile_editor_mode'
               #post-form-wrapper.no-margin-left
                 = f.text_area :profile, class: 'tinymce'
-              = render 'posts/editor_help', editor_location: 'profile'
+              = render 'writable/editor_help', editor_location: 'profile'
           %tr
             %th.sub.vtop= f.label :moiety, "Moiety hex"
             %td{class: cycle('even', 'odd')}