SONAR-4083 add java web service API for metrics

2 files changed, 63 insertions, 45 deletions

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/api/metrics_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/api/metrics_controller.rb
index 9c5e0357794..b2873a28a0c 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/api/metrics_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/api/metrics_controller.rb
@@ -20,7 +20,7 @@
 
 require "json"
 
-class Api::MetricsController < Api::RestController
+class Api::MetricsController < Api::ApiController
 
   # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
   verify :method => :put, :only => [ :update ]
@@ -29,90 +29,98 @@ class Api::MetricsController < Api::RestController
 
   before_filter :admin_required, :only => [ :create, :update, :destroy ]
 
+  # GET /api/metrics
   def index
     metrics = Metric.all
-    rest_render(metrics)
+    respond_to do |format|
+      format.json { render :json => jsonp(metrics_to_json(metrics)) }
+      format.xml { render :xml => metrics_to_xml(metrics) }
+    end
   end
 
+  # GET /api/metrics/foo
   def show
     metric = Metric.by_key(params[:id])
     if !metric
-      rest_status_ko('Metric [' + params[:id] + '] does not exist', 404)
+      render_not_found('Metric [' + params[:id] + '] does not exist')
     else
-      rest_render([metric])
+      respond_to do |format|
+        format.json { render :json => jsonp(metrics_to_json(metric)) }
+        format.xml { render :xml => metrics_to_xml([metric]) }
+      end
     end
   end
 
+  # curl -u admin:admin -v -X POST http://localhost:9000/api/metrics/foo?name=bar&val_type=<type>[&description=<description>&domain=<domain>]
   def create
-    bad_request('Name is required') unless params[:name].present?
-    metric_name = params[:name].downcase.gsub(/\s/, '_')[0..59]
-    metric_id_as_text = params[:id].downcase.gsub(/\s/, '_')[0..59] if params[:id] && params[:id].to_i > 0
-
-    metric_test = Metric.first(:conditions => ['name=? OR id=?', metric_id_as_text, params[:id].to_i])
+    metric_test = Metric.first(:conditions => ['name=?', params[:id]])
 
     exist_and_is_disable = !metric_test.nil? && !metric_test.enabled?
     if exist_and_is_disable
       metric = metric_test
     else
       metric = Metric.new
-    end
-
-    begin
-      metric.attributes = params.merge({:name => metric_id_as_text, :short_name => metric_name})
-      if metric.short_name(false)
-        metric.name = metric.short_name(false) unless metric_id_as_text
-      end
-      metric.origin = Metric::ORIGIN_WS
-      metric.user_managed = true
-      metric.enabled = true
-      metric.save!
-      Metric.clear_cache
-      rest_status_ok
-    rescue
-      rest_status_ko(metric.errors.full_messages.join("."), 400)
+   end
+
+    metric.attributes = params.merge({:name => params[:id], :short_name => params[:name]})
+    metric.origin = Metric::ORIGIN_WS
+    metric.user_managed = true
+    metric.enabled = true
+    metric.save!
+    Metric.clear_cache
+
+    respond_to do |format|
+      format.json { render :json => jsonp(metrics_to_json(metric)) }
+      format.xml { render :xml => metrics_to_xml([metric]) }
     end
   end
 
+  # curl -u admin:admin -v -X PUT http://localhost:9000/api/metrics/foo?name=bar&val_type=<type>[&description=<description>&domain=<domain>]
   def update
-    metric = Metric.first(:conditions => ['(name=? OR id=?) AND enabled=? AND user_managed=?', params[:id], params[:id].to_i, true, true])
+    metric = Metric.first(:conditions => ['name=? AND enabled=? AND user_managed=?', params[:id], true, true])
     if metric
-      begin
-        metric.attributes = params.merge({:name => params[:id], :short_name => params[:name]})
-        metric.save!
-        Metric.clear_cache
-        rest_status_ok
-      rescue
-        rest_status_ko(metric.errors.full_messages.join("."), 400)
+      metric.attributes = params.merge({:name => params[:id], :short_name => params[:name]})
+      metric.save!
+      Metric.clear_cache
+
+      respond_to do |format|
+        format.json { render :json => jsonp(metrics_to_json(metric)) }
+        format.xml { render :xml => metrics_to_xml([metric]) }
       end
     else
-      rest_status_ko('Unable to update manual metric: '+ params[:id], 404)
+      render_not_found('Unable to update manual metric: '+ params[:id])
     end
   end
 
+  # curl -u admin:admin -v -X DELETE http://localhost:9000/api/metrics/foo
   def destroy
     metric = Metric.first(:conditions => ['(name=? OR id=?) AND enabled=? AND user_managed=?', params[:id], params[:id].to_i, true, true])
     if !metric
-      rest_status_ko('Unable to delete manual metric which does not exist: ' + params[:id], 404)
+      render_not_found('Unable to delete manual metric which does not exist: '+ params[:id])
     else
       metric.enabled = false
-      begin
-        metric.save!
-        Metric.clear_cache
-        rest_status_ok
-      rescue
-        rest_status_ko(metric.errors.full_messages.join("."), 400)
-      end
+      metric.save!
+      Metric.clear_cache
+      render_success('metric deleted')
     end
   end
 
 
   protected
 
-  def rest_to_json(metrics)
-    JSON(metrics.collect{|metric| metric.to_hash_json(params)})
+  def metrics_to_json(metrics)
+    json = []
+    metrics.each do |m|
+      json<<rest_to_json(m)
+    end
+    json
+  end
+
+  def metrics_to_json(metric)
+    metric.to_hash_json
   end
 
-  def rest_to_xml(metrics)
+  def metrics_to_xml(metrics)
     xml = Builder::XmlMarkup.new(:indent => 0)
     xml.instruct! 
     xml.metrics do
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/models/metric.rb b/sonar-server/src/main/webapp/WEB-INF/app/models/metric.rb
index 9559cbfdadb..a6b35aa1ad2 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/models/metric.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/models/metric.rb
@@ -44,7 +44,8 @@ class Metric < ActiveRecord::Base
   I18N_DOMAIN_CACHE_KEY='i18n_domains'
   I18N_SHORT_NAME_CACHE_KEY='i18n_metric_short_names'
 
-  validates_length_of       :name, :within => 1..64
+  validates_format_of       :name,   :with => /\A\w+\z/
+  validates_length_of       :name,   :within => 1..64
   validates_uniqueness_of   :name
   validates_length_of       :short_name, :within => 1..64
   validates_inclusion_of    :val_type, :in => [VALUE_TYPE_INT,VALUE_TYPE_BOOLEAN,VALUE_TYPE_FLOAT,VALUE_TYPE_PERCENT,VALUE_TYPE_STRING,VALUE_TYPE_MILLISEC,VALUE_TYPE_LEVEL, VALUE_TYPE_DATA, VALUE_TYPE_DISTRIB], :message => "wrong value type"
@@ -305,6 +306,15 @@ class Metric < ActiveRecord::Base
     origin==ORIGIN_GUI
   end
 
+  HUMANIZED_ATTRIBUTES = {
+      :name => "key",
+      :short_name => "name",
+  }
+
+  def self.human_attribute_name(attr)
+    HUMANIZED_ATTRIBUTES[attr.to_sym] || super
+  end
+
   # METRIC DEFINITIONS
   # WARNING if you edit this file do not forget to also change sonar-commons/src/main/java/ch/hortis/sonar/model/Metrics.java
   LINES = 'lines'