diff --git a/app/controllers/attachments_controller.rb b/app/controllers/attachments_controller.rb
index 215df8e10..c511b5ace 100644
--- a/app/controllers/attachments_controller.rb
+++ b/app/controllers/attachments_controller.rb
@@ -195,6 +195,8 @@ class AttachmentsController < ApplicationController
         end
       elsif !@attachment.container.nil? && @attachment.container.is_a?(Softapplication)
         format.html { redirect_to_referer_or softapplications_path(@attachment.container) }
+      elsif !@attachment.container.nil? && @attachment.container.is_a?(Bid)
+        format.html { redirect_to_referer_or respond_path(@attachment.container) }
       else
          if @project.nil?
            format.html { redirect_to_referer_or forum_memo_path(@attachment.container.forum, @attachment.container) }
diff --git a/app/views/attachments/_form.html.erb b/app/views/attachments/_form.html.erb
index 5c22ff44c..d6cbb72fe 100644
--- a/app/views/attachments/_form.html.erb
+++ b/app/views/attachments/_form.html.erb
@@ -3,14 +3,17 @@
   <% container.attachments.each_with_index do |attachment, i| %>
     <span id="attachments_p<%= i %>" class="attachment">
       <%= text_field_tag("attachments[p#{i}][filename]", attachment.filename, :class => 'filename readonly', :readonly=>'readonly')%>
-      <%= text_field_tag("attachments[p#{i}][description]", attachment.description, :maxlength => 255, :placeholder => l(:label_optional_description), :class => 'description', :style=>"display: inline-block;") +
-      if attachment.id.nil?
-       else
+      <%= text_field_tag("attachments[p#{i}][description]", attachment.description, :maxlength => 255, :placeholder => l(:label_optional_description), :class => 'description', :style=>"display: inline-block;") %>
+      <span class="ispublic-label"><%= l(:field_is_public)%>:</span>
+      <%= check_box_tag("attachments[p#{i}][is_public_checkbox]", attachment.is_public, :class => 'is_public')%>
+      <%= if attachment.id.nil?
+        #待补充代码
+      else
         link_to('&nbsp;'.html_safe, attachment_path(attachment, :attachment_id => "p#{i}", :format => 'js'), :method => 'delete', :remote => true, :class => 'remove-upload')
       end
       %>
       <%#= render :partial => 'tags/tag', :locals => {:obj => attachment, :object_flag => "6"} %>
-      <%= check_box_tag("attachments[p#{i}][is_public_checkbox]", attachment.is_public, :class => 'is_public')%>
+
       <%= hidden_field_tag "attachments[p#{i}][token]", "#{attachment.token}" %>
     </span>
   <% end %>
diff --git a/app/views/bids/show.html.erb b/app/views/bids/show.html.erb
index 3107d8bd6..b431ef97a 100644
--- a/app/views/bids/show.html.erb
+++ b/app/views/bids/show.html.erb
@@ -16,7 +16,7 @@
 		<div class="bid_description">
 			<%= textilizable(@bid, :description) %>
 			<% if @bid.attachments.any?%>
-			<% options = {:author => true} %>
+			<% options = {:author => true,:deletable => (@bid.author.id == User.current.id || User.current.admin? ? true : false)} %>
 			<%= render :partial => 'attachments/links', :locals => {:attachments => @bid.attachments, :options => options} %>
 		<% end %>
 		</div>