fix warnings

skv
1 parent c797493e
Showing 2 changed files with 7 additions and 3 deletions Show diff stats
app/controllers/projects/protected_branches_controller.rb
app/views/projects/edit.html.haml
@@ -6,7 +6,7 @@ class Projects::ProtectedBranchesController &lt; Projects::ApplicationController
   before_filter :authorize_admin_project!, only: [:destroy, :create]
   def index
-    @branches = @project.protected_branches.all
+    @branches = @project.protected_branches.to_a
     @protected_branch = @project.protected_branches.new
   end
@@ -113,7 +113,9 @@
                 The project can be committed to.
                 %br
                 %strong Once active this project shows up in the search and on the dashboard.
-              = link_to 'Unarchive', unarchive_project_path(@project), confirm: "Are you sure that you want to unarchive this project?\nWhen this project is unarchived it is active and can be comitted to again.", method: :post, class: "btn btn-remove"
+              = link_to 'Unarchive', unarchive_project_path(@project),
+                  data: { confirm: "Are you sure that you want to unarchive this project?\nWhen this project is unarchived it is active and can be comitted to again." },
+                  method: :post, class: "btn btn-remove"
             - else
               %p
                 Archiving the project will mark its repository as read-only.
@@ -121,7 +123,9 @@
                 It is hidden from the dashboard and doesn't show up in searches.
                 %br
                 %strong Archived projects cannot be committed to!
-              = link_to 'Archive', archive_project_path(@project), confirm: "Are you sure that you want to archive this project?\nAn archived project cannot be committed to.", method: :post, class: "btn btn-remove"
+              = link_to 'Archive', archive_project_path(@project),
+                  data: { confirm: "Are you sure that you want to archive this project?\nAn archived project cannot be committed to." },
+                  method: :post, class: "btn btn-remove"
       - else
         %p.nothing_here_message Only the project owner can archive a project
	@@ -6,7 +6,7 @@ class Projects::ProtectedBranchesController < Projects::ApplicationController		@@ -6,7 +6,7 @@ class Projects::ProtectedBranchesController < Projects::ApplicationController
6	before_filter :authorize_admin_project!, only: [:destroy, :create]	6	before_filter :authorize_admin_project!, only: [:destroy, :create]
7		7
8	def index	8	def index
9	- @branches = @project.protected_branches.all	9	+ @branches = @project.protected_branches.to_a
10	@protected_branch = @project.protected_branches.new	10	@protected_branch = @project.protected_branches.new
11	end	11	end
12		12