fixes #1634: Allow users to edit feedback they give out

This commit is contained in:
r888888888
2015-08-02 18:19:44 -07:00
parent 2ceb9a0c4f
commit 5e1dc81ab3
4 changed files with 36 additions and 1 deletions

View File

@@ -33,6 +33,13 @@ class UserFeedbacksController < ApplicationController
respond_with(@user_feedback)
end
def update
@user_feedback = UserFeedback.find(params[:id])
check_privilege(@user_feedback)
@user_feedback.update_attributes(params[:user_feedback])
respond_with(@user_feedback)
end
def destroy
@user_feedback = UserFeedback.find(params[:id])
check_privilege(@user_feedback)

View File

@@ -0,0 +1,23 @@
<div id="c-user-feedbacks">
<div id="a-new">
<h1>Edit User Feedback</h1>
<div id="preview">
</div>
<%= error_messages_for "user_feedback" %>
<%= simple_form_for(@user_feedback) do |f| %>
<%= f.input :category, :collection => ["positive", "neutral", "negative"], :include_blank => false %>
<%= dtext_field "user_feedback", "body" %>
<%= f.button :submit, "Submit" %>
<%= dtext_preview_button "user_feedback", "body" %>
<% end %>
<%= render "secondary_links" %>
</div>
</div>
<% content_for(:page_title) do %>
Edit Feedback - <%= Danbooru.config.app_name %>
<% end %>

View File

@@ -21,7 +21,8 @@
<td><%= format_text(feedback.body) %></td>
<td>
<% if feedback.creator_id == CurrentUser.id || CurrentUser.is_moderator? %>
<%= link_to "delete", user_feedback_path(feedback), :method => :delete, :data => {:confirm => "Are you sure you want to delete this user feedback?"} %>
<%= link_to "edit", edit_user_feedback_path(feedback) %>
| <%= link_to "delete", user_feedback_path(feedback), :method => :delete, :data => {:confirm => "Are you sure you want to delete this user feedback?"} %>
<% end %>
</td>
</tr>

View File

@@ -8,6 +8,10 @@
<li><strong>Category</strong> <%= @user_feedback.category %></li>
<li><strong>Message</strong> <%= format_text @user_feedback.body %></li>
</ul>
<% if @user_feedback.creator_id == CurrentUser.id || CurrentUser.is_moderator? %>
<p><%= link_to "Edit", edit_user_feedback_path(@user_feedback) %></p>
<% end %>
</div>
</div>