上傳和下載文件（Rails 4）

Question

我無法下載我上傳的文件，當我單擊該文件時，出現錯誤“找不到ID的新聞通訊”。 這是我的代碼

controller.rb

     class NewslettersController < ApplicationController
   before_action :set_newsletter, only: [:show, :edit, :update, :destroy]

   # GET /newsletters
    # GET /newsletters.json
   def index
     @newsletters = Newsletter.all
   end

   # GET /newsletters/1
    # GET /newsletters/1.json
   def show
   end

   # GET /newsletters/new
  def new
     @newsletter = Newsletter.new
     end

    # GET /newsletters/1/edit
    def edit
    end

    # POST /newsletters
    # POST /newsletters.json
     def create
      @newsletter = Newsletter.new(newsletter_params)

      respond_to do |format|
        if @newsletter.save
       format.html { redirect_to @newsletter, notice: 'Newsletter was successfully    created.' }
      format.json { render action: 'show', status: :created, location: @newsletter }
    else
    format.html { render action: 'new' }
    format.json { render json: @newsletter.errors, status: :unprocessable_entity }
     end
   end
 end
  #file download
 def document_download
  @newsletter = Newsletter.find(params[:id])
   file_path = @newsletter.document_download
    if !file_path.nil?
send_file "#{Rails.root}/public/system/documents#{params[:id]}/original/#{file_path}",  :x_sendfile => true
  else
     redirect_to newsletters_url
   end
  end
 # PATCH/PUT /newsletters/1
  # PATCH/PUT /newsletters/1.json
  def update
  respond_to do |format|
    if @newsletter.update(newsletter_params)
     format.html { redirect_to @newsletter, notice: 'Newsletter was successfully       updated.' }
      format.json { head :no_content }
       else
    format.html { render action: 'edit' }
    format.json { render json: @newsletter.errors, status: :unprocessable_entity }
  end
   end
 end

# DELETE /newsletters/1
  # DELETE /newsletters/1.json
  def destroy
  @newsletter.destroy
   respond_to do |format|
      format.html { redirect_to newsletters_url }
    format.json { head :no_content }
   end
  end

  private
  # Use callbacks to share common setup or constraints between actions.
   def set_newsletter
  @newsletter = Newsletter.find(params[:newsletter_id])
   end

  # Never trust parameters from the scary internet, only allow the white list through.
   def newsletter_params
    params.require(:newsletter).permit(:document, :year)
  end
end

index.html

   <tbody>
   <% @newsletters.each do |newsletter| %>
     <tr>

    <td><a href="/public/newsletters/document_download/", style="text-decoration:none"> <%=newsletter.document_file_name %></a></td>
      <td><%= newsletter.year %></td>

    <td><%= link_to 'Show', newsletter %></td>
    <td><%= link_to 'Edit', edit_newsletter_path(newsletter) %></td>
     <td><%= link_to 'Destroy', newsletter, method: :delete, data: { confirm: 'Are you    sure?' } %></td>
  </tr>
  <% end %>
 </tbody>

show.html

     <p id="notice"><%= notice %></p>

  <p>
    <strong>Document:</strong>
  <%= document_download_newsletter_path(@id_or_model_itself) %>

  </p>


    <p>
   <strong>Year:</strong>
   <%= @newsletter.year %>
   </p>

 <%= link_to 'Edit', edit_newsletter_path(@newsletter) %> |
 <%= link_to 'Back', newsletters_path %>

form.html

    <%= form_for(@newsletter) do |f| %>
     <% if @newsletter.errors.any? %>
        <div id="error_explanation">
     <h2><%= pluralize(@newsletter.errors.count, "error") %> prohibited this newsletter from being saved:</h2>

  <ul>
  <% @newsletter.errors.full_messages.each do |msg| %>
    <li><%= msg %></li>
  <% end %>
  </ul>
</div>
  <% end %>

   <div class="field">
      <%= f.label :document %><br>
        <%= f.text_field :document %>
    <%= f.file_field :document %>
     </div>
    <div class="field">
      <%= f.label :year %><br>
      <%= f.text_field :year %>
    </div>
    <div class="actions">
       <%= f.submit %>
    </div>
    <% end %>

routes.rb

      resources :newsletters do
    get :document_download, on: :member 

end

型號（newsletters.rb）

   class Newsletter < ActiveRecord::Base

    has_attached_file :document
    validates_attachment_presence :document
    validates_attachment_content_type :document, :content_type => [ 'application/pdf','text/plain']


    end

Answer 1

沒有閱讀完整的代碼，但是基於您提到的錯誤，問題出在set_newsletter方法中。 由於這是NewslettersController並且假設您沒有為Newsletter模型手動設置其他主鍵，因此應使用params[:id]而不是params[:newsletter_id] ，如下所示：

# app/controllers/newsletters_controller.rb

def set_newsletter
  @newsletter = Newsletter.find(params[:id]) 
end

Answer 2

更新的回復（2013年12月1日）：

根據我們的評論，我再次查看您的代碼，並在index.html（我認為是index.html.erb）中遇到了這一行：

<td><a href="/public/newsletters/document_download/", style="text-decoration:none"> <%=newsletter.document_file_name %></a></td>
  <td><%= newsletter.year %></td>

看來您指的是新聞通訊的文檔名稱，但沒有在href中傳遞新聞通訊ID。 您的show.html（show.html.erb）似乎是正確的，但是我看不到您在哪里初始化“ @id_or_model_itself”的值。

您可以修改這些部分並驗證其是否有效嗎？

原始回應

在“ set_newsletter”方法中，您嘗試使用“ params [：newsletter_id]”獲取新聞簡報的ID，但是對於“ document_download”操作，您將使用“ params [：id]”。 我懷疑這是問題背后的原因。 嘗試在document_download中也使用“ params [：newsletter_id]”。