Commit 01520d5d authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Dont allow edit or remove of system notes

Signed-off-by: default avatarDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
parent 36361f4e
......@@ -30,8 +30,10 @@ class Projects::NotesController < Projects::ApplicationController
end
def update
note.update_attributes(note_params)
note.reset_events_cache
if note.editable?
note.update_attributes(note_params)
note.reset_events_cache
end
respond_to do |format|
format.json { render_note_json(note) }
......@@ -40,8 +42,10 @@ class Projects::NotesController < Projects::ApplicationController
end
def destroy
note.destroy
note.reset_events_cache
if note.editable?
note.destroy
note.reset_events_cache
end
respond_to do |format|
format.js { render nothing: true }
......
......@@ -337,4 +337,8 @@ class Note < ActiveRecord::Base
def set_references
notice_added_references(project, author)
end
def editable?
!system
end
end
......@@ -9,7 +9,7 @@
%i.icon-link
Link here
&nbsp;
- if(note.author_id == current_user.try(:id)) || can?(current_user, :admin_note, @project)
- if can?(current_user, :admin_note, note) && note.editable?
= link_to "#", title: "Edit comment", class: "js-note-edit" do
%i.icon-edit
Edit
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment