Only allow editing text files

Riyad Preukschas
1 parent ad5a6a4a
Showing 1 changed file with 9 additions and 0 deletions Show diff stats
app/controllers/tree_controller.rb
@@ -8,6 +8,7 @@ class TreeController &lt; ProjectResourceController
   before_filter :require_non_empty_project
   before_filter :assign_ref_vars
+  before_filter :edit_requirements, only: [:edit, :update]
   def show
     @hex_path  = Digest::SHA1.hexdigest(@path)
@@ -40,4 +41,12 @@ class TreeController &lt; ProjectResourceController
       render :edit
     end
   end
+
+  private
+
+  def edit_requirements
+    unless @tree.is_blob? && @tree.text?
+      redirect_to project_tree_path(@project, @id), notice: "You can only edit text files"
+    end
+  end
 end
	@@ -8,6 +8,7 @@ class TreeController < ProjectResourceController		@@ -8,6 +8,7 @@ class TreeController < ProjectResourceController
8	before_filter :require_non_empty_project	8	before_filter :require_non_empty_project
9		9
10	before_filter :assign_ref_vars	10	before_filter :assign_ref_vars
		11	+ before_filter :edit_requirements, only: [:edit, :update]
11		12
12	def show	13	def show
13	@hex_path = Digest::SHA1.hexdigest(@path)	14	@hex_path = Digest::SHA1.hexdigest(@path)
	@@ -40,4 +41,12 @@ class TreeController < ProjectResourceController		@@ -40,4 +41,12 @@ class TreeController < ProjectResourceController
40	render :edit	41	render :edit
41	end	42	end
42	end	43	end
		44	+
		45	+ private
		46	+
		47	+ def edit_requirements
		48	+ unless @tree.is_blob? && @tree.text?
		49	+ redirect_to project_tree_path(@project, @id), notice: "You can only edit text files"
		50	+ end
		51	+ end
43	end	52	end