From: Julien Lancelot <julien.lancelot@gmail.com>
Date: Thu, 4 Jul 2013 13:22:21 +0000 (+0200)
Subject: SONAR-4418 Fix issue when launching bulk change with a period selected and fix set... 
X-Git-Tag: 3.7~260
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=bce1db9e70ecda6733d68be3db3957aa2d42a897;p=sonarqube.git

SONAR-4418 Fix issue when launching bulk change with a period selected and fix set severity action
---

diff --git a/sonar-server/src/main/java/org/sonar/server/issue/SetSeverityAction.java b/sonar-server/src/main/java/org/sonar/server/issue/SetSeverityAction.java
index 363d963e576..b44fa04ffa5 100644
--- a/sonar-server/src/main/java/org/sonar/server/issue/SetSeverityAction.java
+++ b/sonar-server/src/main/java/org/sonar/server/issue/SetSeverityAction.java
@@ -50,7 +50,7 @@ public class SetSeverityAction extends Action implements ServerComponent {
 
   @Override
   public boolean execute(Map<String, Object> properties, Context context) {
-    return issueUpdater.setSeverity((DefaultIssue) context.issue(), severity(properties), context.issueChangeContext());
+    return issueUpdater.setManualSeverity((DefaultIssue) context.issue(), severity(properties), context.issueChangeContext());
   }
 
   private String severity(Map<String, Object> properties) {
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb
index 1e0b00bfdcc..b645f80b042 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb
@@ -323,7 +323,7 @@ class ResourceController < ApplicationController
     if @period && @period != 0
       date = @snapshot.period_datetime(@period)
       if date
-        options['createdAfter'] = date
+        options['createdAfter'] = Api::Utils.format_datetime(date)
       end
     end
 
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/resource/_tabs.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/resource/_tabs.html.erb
index 5007355a5cc..a81e243c1bb 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/resource/_tabs.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/resource/_tabs.html.erb
@@ -21,18 +21,15 @@
 
   <div class="source_tabs">
     <ul class="tablinks">
-
-      <% if @display_issues %>
-        <li>
+      <% first=true %>
+      <% if logged_in? && @display_issues %>
+        <li class="<%= 'first' if first -%>">
           <a href="<%= url_for @issues_params.merge({:controller => 'issues', :action => 'bulk_change_form', :from => 'drilldown'}) -%>"
              class="bulk-change-link open-modal"><%= message('bulk_change') -%></a>
         </li>
-      <% end %>
-
-      <%
-         first=true
-         if @snapshot.has_source && has_role?(:codeviewer, @snapshot)
-      %>
+      <% first=false
+         end %>
+      <% if @snapshot.has_source && has_role?(:codeviewer, @snapshot) %>
         <li class="<%= 'first' if first -%>">
           <a href="<%= ApplicationController.root_context -%>/api/sources?resource=<%= @resource.key -%>&amp;format=txt"><%= message('raw') -%></a>
         </li>
diff --git a/sonar-server/src/test/java/org/sonar/server/issue/SetSeverityActionTest.java b/sonar-server/src/test/java/org/sonar/server/issue/SetSeverityActionTest.java
index 0148ab2ccc9..487f9bb1389 100644
--- a/sonar-server/src/test/java/org/sonar/server/issue/SetSeverityActionTest.java
+++ b/sonar-server/src/test/java/org/sonar/server/issue/SetSeverityActionTest.java
@@ -57,7 +57,7 @@ public class SetSeverityActionTest {
     when(context.issue()).thenReturn(issue);
 
     action.execute(properties, context);
-    verify(issueUpdater).setSeverity(eq(issue), eq(severity), any(IssueChangeContext.class));
+    verify(issueUpdater).setManualSeverity(eq(issue), eq(severity), any(IssueChangeContext.class));
   }
 
   @Test