SONAR-3692 Boolean type are now always displayed as select box

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/helpers/properties_helper.rb b/sonar-server/src/main/webapp/WEB-INF/app/helpers/properties_helper.rb
index 6744ae13f219880d5c6245ef4712c288ebaf2fdb..a63afaf7b96a4d44e7e556d4a0c64968ebbdc9a6 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/helpers/properties_helper.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/helpers/properties_helper.rb
@@ -41,7 +41,7 @@ module PropertiesHelper
     case type
 
       when PropertyType::TYPE_STRING
-        text_field_tag name, value, {:size => options[:size] || 25}.update(html_options)
+        text_field_tag name, value, {:size => options[:size] || 50}.update(html_options)
 
       when PropertyType::TYPE_TEXT
         cols = options[:size] || nil
@@ -49,18 +49,13 @@ module PropertiesHelper
         text_area_tag name, value, {:class => html_class, :rows => '5', :cols => cols}.update(html_options)
 
       when PropertyType::TYPE_PASSWORD
-        password_field_tag name, value, {:size => options[:size] || 25}.update(html_options)
+        password_field_tag name, value, {:size => options[:size] || 50}.update(html_options)
 
       when PropertyType::TYPE_BOOLEAN
-        if !options[:default].blank? || screen == SCREEN_SETTINGS
-          default_value = options[:default].blank? ? '' : message('default')
-          select_options = "<option value='' #{ 'selected' if value.blank? }>#{ default_value }</option>"
-          select_options += "<option value='true' #{ 'selected' if value=='true' }>#{ message('true') }</option>"
-          select_options += "<option value='false' #{ 'selected' if value=='false' }>#{ message('false') }</option>"
-          select_tag name, select_options, html_options
-        else
-          (hidden_field_tag name, 'false', html_options) + (check_box_tag name, 'true', value=='true', html_options)
-        end
+        select_options = "<option value='' #{ 'selected' if value.blank? }>#{ message('default') }</option>"
+        select_options += "<option value='true' #{ 'selected' if value=='true' }>#{ message('true') }</option>"
+        select_options += "<option value='false' #{ 'selected' if value=='false' }>#{ message('false') }</option>"
+        select_tag name, select_options, html_options
 
       when PropertyType::TYPE_INTEGER
         size = options[:size] || 10
@@ -75,7 +70,7 @@ module PropertiesHelper
                                                                         :placeholder => !options[:default].blank? ?  message('default') : nil}
 
       when PropertyType::TYPE_REGULAR_EXPRESSION
-        size = options[:size] || 25
+        size = options[:size] || 50
         text_field_tag name, value, {:size => size}.update(html_options)
 
       when PropertyType::TYPE_FILTER

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/helpers/widget_properties_helper.rb b/sonar-server/src/main/webapp/WEB-INF/app/helpers/widget_properties_helper.rb
index 465f7ba0c3b41c6fbf9d4d964baa50bad22d3c47..ccada016ab8e8377c90b370b1fe31c34f315b814 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/helpers/widget_properties_helper.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/helpers/widget_properties_helper.rb
@@ -27,4 +27,12 @@ module WidgetPropertiesHelper
     property_input_field definition.key, definition.type.name, value, 'WIDGET', options
   end
 
+  def default_value(property_def)
+    defaultValue = property_def.defaultValue
+    # Boolean type should always have a default value, if no one is provided it's force to false
+    defaultValue = property_def.type.name == PropertyType::TYPE_BOOLEAN ? 'false' : property_def.defaultValue if defaultValue.blank?
+    defaultValue = '********' if property_def.type.name == PropertyType::TYPE_PASSWORD
+    defaultValue
+  end
+
 end

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
index df14ac28ff4f9d6df3106df6064dbb08182c4af8..ae05758e1cacbecff76c12b36b59fb75e00e2d48 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
@@ -37,8 +37,8 @@
             <% unless property_description.blank? -%>
               <div><%= property_description -%></div>
             <% end %>
-            <% unless property_def.defaultValue.blank? -%>
-              <div><%= message('default') %>:: <%= property_def.type.name == PropertyType::TYPE_PASSWORD ? '********' : h(property_def.defaultValue) -%></div>
+            <% if !property_def.defaultValue.blank? || property_def.type.name == PropertyType::TYPE_BOOLEAN -%>
+              <div><%= message('default') %>: <%= h(default_value property_def) -%></div>
             <% end -%>
           </div>
         </td>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_FLOAT.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_FLOAT.html.erb
index d9e9399908dcc0001b9d643b9d5a8f571400701b..721480a62aaeeb76b0020f5e04fc507e9bc90b95 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_FLOAT.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_FLOAT.html.erb
@@ -1,5 +1,5 @@
 <%
    options = {:id => id}
-   options[:size] = (defined? size) ? size : 50
+   options[:size] = (defined? size) ? size : nil
 %>
 <%= property_input_field(name, PropertyType::TYPE_FLOAT, value, PropertiesHelper::SCREEN_SETTINGS, options) %>
\ No newline at end of file

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_INTEGER.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_INTEGER.html.erb
index 79f4a1f6efa0c33faac280d4d8e1becab3fa478d..5f7c39e8c7b0618492f3b6ccbeaf57e9a5b7c6cf 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_INTEGER.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_INTEGER.html.erb
@@ -1,5 +1,5 @@
 <%
    options = {:id => id}
-   options[:size] = (defined? size) ? size : 50
+   options[:size] = (defined? size) ? size : nil
 %>
 <%= property_input_field(name, PropertyType::TYPE_INTEGER, value, PropertiesHelper::SCREEN_SETTINGS, options) %>
\ No newline at end of file

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_PASSWORD.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_PASSWORD.html.erb
index ee7ca50b73ac1dd4c27e593123c01a46e05fda49..68bc7edb71a1bebe299bdbeb43fe48f0517e8a36 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_PASSWORD.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_PASSWORD.html.erb
@@ -1,5 +1,5 @@
 <%
    options = {:id => id}
-   options[:size] = (defined? size) ? size : 50
+   options[:size] = (defined? size) ? size : nil
 %>
 <%= property_input_field(name, PropertyType::TYPE_PASSWORD, value, PropertiesHelper::SCREEN_SETTINGS, options) %>
\ No newline at end of file

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_REGULAR_EXPRESSION.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_REGULAR_EXPRESSION.html.erb
index 7dececf6629c33208d9a8937b8be98388aa57e79..8274fd8ef8f8912a390555ce9a8439264a871c91 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_REGULAR_EXPRESSION.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_REGULAR_EXPRESSION.html.erb
@@ -1,5 +1,5 @@
 <%
    options = {:id => id}
-   options[:size] = (defined? size) ? size : 50
+   options[:size] = (defined? size) ? size : nil
 %>
 <%= property_input_field(name, PropertyType::TYPE_REGULAR_EXPRESSION, value, PropertiesHelper::SCREEN_SETTINGS, options) %>
\ No newline at end of file

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_STRING.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_STRING.html.erb
index 3df9028bebe7e6eb0a50dbe86b964fabe585464e..b28c75604a737beb82c2a339c157af8034372ddc 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_STRING.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/settings/_type_STRING.html.erb
@@ -1,5 +1,5 @@
 <%
    options = {:id => id}
-   options[:size] = (defined? size) ? size : 50
+   options[:size] = (defined? size) ? size : nil
 %>
 <%= property_input_field(name, PropertyType::TYPE_STRING, value, PropertiesHelper::SCREEN_SETTINGS, options) %>
\ No newline at end of file