From a8a489d5da78514bb3c6bcb7ea170786a8da27cf Mon Sep 17 00:00:00 2001
From: Jean-Baptiste Vilain <jean-baptiste.vilain@sonarsource.com>
Date: Wed, 17 Jul 2013 09:54:36 +0200
Subject: SONAR-4453 Adjusted labels and presentation for bulk permission
 changes

---
 .../WEB-INF/app/controllers/roles_controller.rb    |  1 -
 .../app/views/roles/_apply_template_form.html.erb  | 26 ++++++++++++++++------
 .../WEB-INF/app/views/roles/projects.html.erb      |  2 +-
 3 files changed, 20 insertions(+), 9 deletions(-)

(limited to 'sonar-server')

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/roles_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/roles_controller.rb
index b0eb5c091d4..5fb870f2d37 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/roles_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/roles_controller.rb
@@ -77,7 +77,6 @@ class RolesController < ApplicationController
 
   # GET /roles/apply_template_form?projects=<projects>&qualifier=<qualifier>
   def apply_template_form
-    bad_request('There are currently no results to apply the permission template to') if params[:projects].blank?
     @permission_templates = Internal.permission_templates.selectAllPermissionTemplates().collect {|pt| [pt.name, pt.key]}
     render :partial => 'apply_template_form', :locals => {:components => params[:projects], :qualifier => params[:qualifier] || 'TRK'}
   end
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/_apply_template_form.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/_apply_template_form.html.erb
index 4f76e4f9519..659ff4d47ef 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/_apply_template_form.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/_apply_template_form.html.erb
@@ -1,19 +1,31 @@
 <form id="apply-template-form" method="post" action="<%= ApplicationController.root_context -%>/roles/apply_template">
-  <input type="hidden" name="components" id="apply-template-components" value="<%= components.join(',') %>"/>
+  <% unless components.nil? %>
+    <input type="hidden" name="components" id="apply-template-components" value="<%= components.join(',') %>"/>
+  <% end %>
   <fieldset>
     <div class="modal-head">
       <h2><%= message('projects_role.apply_template.form.title', {:params => ''}) -%></h2>
     </div>
     <div class="modal-body">
       <div class="apply-template errors error" style="display:none;"/>
-      <div class="modal-field">
-        <span><%= "Select the template to apply to #{components.size} #{message('qualifier.' + qualifier)}(s):" %></span><br/>
-        <%= select_tag('template_key', options_for_select(@permission_templates, -1)) %>
-      </div>
+      <% if components.nil? %>
+        <div class="info">
+          <img src="<%= ApplicationController.root_context -%>/images/information.png" style="vertical-align: text-bottom"/>
+          <%= message 'projects_role.no_projects' -%>
+        </div>
+      <% else %>
+        <div class="modal-field">
+          <%= label_tag 'template_key', 'Permission template:' %>
+          <%= select_tag('template_key', options_for_select(@permission_templates, -1), :style => 'width: 250px;') %>
+          <span style="float:right" class="note">(<%= "#{components.size} #{message('qualifier.' + qualifier)}(s)" %>)</span><br/>
+        </div>
+      <% end %>
     </div>
     <div class="modal-foot">
-      <span id="apply-template-loading-image" class="hidden"><%= image_tag 'loading.gif' %></span>
-      <input type="submit" value="<%= message('apply') -%>" id="apply-template-submit" class="apply-template" onclick="return displayLoadingImage()" />
+      <% unless components.nil? %>
+        <span id="apply-template-loading-image" class="hidden"><%= image_tag 'loading.gif' %></span>
+        <input type="submit" value="<%= message('apply') -%>" id="apply-template-submit" class="apply-template" onclick="return displayLoadingImage()" />
+      <% end %>
       <a href="#" onclick="return closeModalWindow()" id="apply-template-cancel"><%= message('cancel') -%></a>
     </div>
   </fieldset>
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb
index 8fc851b830b..5eab3af0fb9 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/roles/projects.html.erb
@@ -31,7 +31,7 @@
 <div id="project-roles-operations" style="float: right;">
   <ul class="operations">
     <li class="last">
-      <%= link_to message('projects_role.apply_template'), {:action => :apply_template_form, :projects => @projects.collect {|proj| proj.id}, :qualifier => @qualifier},
+      <%= link_to message('projects_role.bulk_change'), {:action => :apply_template_form, :projects => @projects.collect {|proj| proj.id}, :qualifier => @qualifier},
                    :id => 'apply-template-modal', :class => 'open-modal link-action' %>
     </li>
   </ul>
-- 
cgit v1.2.3