[Mezuro] Refactoring project content, views and controller (profile)

Paulo Meireles
1 parent 1f34a5b4
Showing 11 changed files with 312 additions and 233 deletions Show diff stats
plugins/mezuro/controllers/mezuro_plugin_profile_controller.rb
plugins/mezuro/lib/mezuro_plugin/project_content.rb
plugins/mezuro/public/javascripts/project_content.js
plugins/mezuro/test/functional/mezuro_plugin_profile_controller_test.rb
plugins/mezuro/views/cms/mezuro_plugin/_project_content.html.erb
plugins/mezuro/views/content_viewer/_module_result.rhtml
plugins/mezuro/views/content_viewer/_project_error.rhtml
plugins/mezuro/views/content_viewer/_project_result.rhtml
plugins/mezuro/views/content_viewer/_score_history.rhtml
plugins/mezuro/views/content_viewer/_source_tree.rhtml
plugins/mezuro/views/content_viewer/show_project.rhtml
 class MezuroPluginProfileController < ProfileController
   append_view_path File.join(File.dirname(__FILE__) + '/../views')
-
+  
   def project_state
-    content = profile.articles.find(params[:id])
-    project = content.project
+    @content = profile.articles.find(params[:id])
+    project = @content.project
     state = project.error.nil? ? project.state : "ERROR"
     render :text => state
   end
   def project_error
-    content = profile.articles.find(params[:id])
-    project = content.project
-    render :partial => 'content_viewer/project_error', :locals => { :project => project }
+    @content = profile.articles.find(params[:id])
+    @project = @content.project
+    render :partial => 'content_viewer/project_error'
   end
   def project_result
-    content = profile.articles.find(params[:id])
+    @content = profile.articles.find(params[:id])
     date = params[:date]
-    project_result = date.nil? ? content.project_result : content.get_date_result(date)
-    project = content.project
-    render :partial => 'content_viewer/project_result', :locals => { :project_result => project_result}
+    @project_result = date.nil? ? @content.project_result : @content.get_date_result(date)
+    render :partial => 'content_viewer/project_result'
   end 	
   def module_result
-    content = profile.articles.find(params[:id])
+    @content = profile.articles.find(params[:id])
     date = params[:date]
-    date.nil? ? content.project_result : content.get_date_result(date)
-    module_result = content.module_result(params[:module_name])
-   render :partial => 'content_viewer/module_result', :locals => { :module_result =>  module_result}
+    date.nil? ? @content.project_result : @content.get_date_result(date)
+    @module_result = @content.module_result(params[:module_name])
+    render :partial => 'content_viewer/module_result'
   end
   def project_tree
-    content = profile.articles.find(params[:id])
+    @content = profile.articles.find(params[:id])
     date = params[:date]
-    project_result = date.nil? ? content.project_result : content.get_date_result(date)
-    source_tree = project_result.node_of(params[:module_name])
-    render :partial =>'content_viewer/source_tree', :locals => { :source_tree => source_tree, :project_name => content.project.name}
+    project_result = date.nil? ? @content.project_result : @content.get_date_result(date)
+    @project_name = @content.project.name
+    @source_tree = project_result.node_of(params[:module_name])
+    render :partial =>'content_viewer/source_tree'
   end
   def module_metrics_history
     metric_name = params[:metric_name]
-    content = profile.articles.find(params[:id])
-    module_history = content.result_history(params[:module_name])
-    score_history = (module_history.collect { |module_result| (module_result.metric_results.select { |metric_result| metric_result.metric.name.delete("() ") == metric_name })[0] }).collect { |metric_result| metric_result.value }
-    render :partial => 'content_viewer/score_history', :locals => {:score_history => score_history}
+    @content = profile.articles.find(params[:id])
+    module_history = @content.result_history(params[:module_name])
+    @score_history = filtering_metric_history(metric_name, module_history)
+    render :partial => 'content_viewer/score_history'
   end
   def module_grade_history
-    content = profile.articles.find(params[:id])
-    modules_results = content.result_history(params[:module_name])
-    score_history = modules_results.collect { |module_result| module_result.grade }
-    render :partial => 'content_viewer/score_history', :locals => {:score_history => score_history}
+    @content = profile.articles.find(params[:id])
+    modules_results = @content.result_history(params[:module_name])
+    @score_history = modules_results.collect { |module_result| module_result.grade }
+    render :partial => 'content_viewer/score_history'
+  end
+  
+  private
+  
+  def filtering_metric_history(metric_name, module_history)
+    metric_history = module_history.collect do |module_result| 
+      module_result.metric_results.select do |metric_result| 
+        metric_result.metric.name.delete("() ") == metric_name
+      end
+    end
+    metric_history[0].collect{ |metric_result| metric_result.value }
   end
 end
 class MezuroPlugin::ProjectContent < Article 
-  validate :validate_kalibro_project_name
-
+  validate_on_create :validate_kalibro_project_name 
+  validate_on_create :validate_repository_url
   def self.short_description
     'Kalibro project'
   end
@@ -20,17 +20,29 @@ class MezuroPlugin::ProjectContent &lt; Article
   def project
-    @project ||= Kalibro::Client::ProjectClient.project(name)
+    begin
+      @project ||= Kalibro::Client::ProjectClient.project(name)
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
   end
   def project_result
-    @project_result ||= Kalibro::Client::ProjectResultClient.last_result(name)
+    begin
+      @project_result ||= Kalibro::Client::ProjectResultClient.last_result(name)
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
   end
   def get_date_result(date)
-    client =  Kalibro::Client::ProjectResultClient.new
-    @project_result ||= client.has_results_before(name, date) ? client.last_result_before(name, date) : 
+    begin
+      client =  Kalibro::Client::ProjectResultClient.new
+      @project_result ||= client.has_results_before(name, date) ? client.last_result_before(name, date) : 
 client.first_result_after(name, date)
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
   end
   def module_result(module_name)
@@ -48,24 +60,47 @@ client.first_result_after(name, date)
   private
   def validate_kalibro_project_name
-    existing = Kalibro::Client::ProjectClient.new.project_names
+    begin
+      existing = Kalibro::Client::ProjectClient.new.project_names
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
-    if existing.include?(name)
+    if existing.any?{|existing_name| existing_name.casecmp(name)==0} # existing.include?(name) + case insensitive
       errors.add_to_base("Project name already exists in Kalibro")
     end
   end
-
+  
+  def validate_repository_url
+    if(repository_url.nil? || repository_url == "")
+      errors.add_to_base("Repository URL is mandatory")
+    end
+  end
+  
   def send_project_to_service
-    Kalibro::Client::ProjectClient.save(self)
-    Kalibro::Client::KalibroClient.process_project(name, periodicity_in_days)
+    begin
+      Kalibro::Client::ProjectClient.save(self)
+      Kalibro::Client::KalibroClient.process_project(name, periodicity_in_days)
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
+
   end
   def remove_project_from_service
-    Kalibro::Client::ProjectClient.remove(name)
+    begin
+      Kalibro::Client::ProjectClient.remove(name)
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
+
   end
   def module_result_client
-    @module_result_client ||= Kalibro::Client::ModuleResultClient.new
+    begin
+      @module_result_client ||= Kalibro::Client::ModuleResultClient.new
+    rescue Exception => error
+      errors.add_to_base(error.message)
+    end
   end
 end
-
@@ -5,6 +5,7 @@ jQuery(function (){
   jQuery('[data-show]').live("click", toggle_mezuro);
   jQuery('[show-metric-history]').live("click", display_metric_history);
   jQuery('[show-grade-history]').live("click", display_grade_history);
+  jQuery('#project_date_submit').live("click", reloadProjectWithDate);
   showLoadingProcess(true);
   showProjectContent();
 });
@@ -55,8 +56,17 @@ function reloadProjectWithDate(){
 	var month = jQuery("#project_date_month").val();
 	var year = jQuery("#project_date_year").val();
-	var date = year + "-" + month + "-" + day + "T00:00:00+00:00";
+  if(day.length == 1) 
+    day = "0" + day;
+  if(month.length == 1) 
+    month = "0" + month; 
+	var date = new Date(year + "-" + month + "-" + day + "T00:00:00+00:00");
+	
+	if(isNaN(date)){
+	  alert("Invalid date! " + date);
+	  return false;
+	}
 	reloadProject(date);
   return false;
 }
@@ -6,6 +6,7 @@ require &quot;#{RAILS_ROOT}/plugins/mezuro/test/fixtures/error_fixtures&quot;
 require "#{RAILS_ROOT}/plugins/mezuro/test/fixtures/base_tool_fixtures"
 require "#{RAILS_ROOT}/plugins/mezuro/test/fixtures/native_metric_fixtures"
 require "#{RAILS_ROOT}/plugins/mezuro/test/fixtures/metric_configuration_fixtures"
+require "#{RAILS_ROOT}/plugins/mezuro/test/fixtures/repository_fixtures"
 class MezuroPluginProfileControllerTest < ActionController::TestCase
@@ -17,6 +18,7 @@ class MezuroPluginProfileControllerTest &lt; ActionController::TestCase
     @project_result = ProjectResultFixtures.qt_calculator
     @module_result = ModuleResultFixtures.create
+    @repository_url = RepositoryFixtures.qt_calculator.address
     @project = @project_result.project
     @name = @project.name
@@ -66,7 +68,6 @@ class MezuroPluginProfileControllerTest &lt; ActionController::TestCase
   should 'get project results without date' do
     create_project_content
     Kalibro::Client::ProjectResultClient.expects(:last_result).with(@name).returns(@project_result)
-    Kalibro::Client::ProjectClient.expects(:project).with(@name).returns(@project)
     get :project_result, :profile => @profile.identifier, :id => @content.id
     assert_response 200
     assert_select('h4', 'Last Result')
@@ -75,7 +76,6 @@ class MezuroPluginProfileControllerTest &lt; ActionController::TestCase
   should 'get project results from a specific date' do
     create_project_content
     mock_project_result
-    Kalibro::Client::ProjectClient.expects(:project).with(@name).returns(@project)
     get :project_result, :profile => @profile.identifier, :id => @content.id, :date => @project_result.date
     assert_response 200
   end
@@ -90,7 +90,6 @@ class MezuroPluginProfileControllerTest &lt; ActionController::TestCase
     mock_module_result
     Kalibro::Client::ProjectResultClient.expects(:last_result).with(@name).returns(@project_result)
     Kalibro::Client::ProjectClient.expects(:project).with(@name).returns(@project)
-
     get :module_result, :profile => @profile.identifier, :id => @content.id, :module_name => @name
     assert_response 200
     assert_select('h5', 'Metric results for: Qt-Calculator (APPLICATION)')
@@ -157,6 +156,7 @@ class MezuroPluginProfileControllerTest &lt; ActionController::TestCase
     @content = MezuroPlugin::ProjectContent.new(:profile => @profile, :name => @name)
     @content.expects(:send_project_to_service).returns(nil)
     Kalibro::Client::ProjectClient.expects(:new).returns(client)
+    @content.repository_url = @repository_url
     client.expects(:project_names).returns([])
     @content.save
   end
@@ -15,8 +15,11 @@
 <%= hidden_field_tag 'id', @article.id %>
 <%= required_fields_message %>
-
-<%= required f.text_field(:name) %>
+<% if !@project.nil? && !@article.id.nil? %>
+  <%= required f.text_field(:name, :disabled => 'true') %>
+<% else %>
+  <%= required f.text_field(:name) %>
+<% end %>
 <%= f.text_field :license %><br/>
@@ -31,9 +34,14 @@
 <% @configuration_names = Kalibro::Client::ConfigurationClient.new.configuration_names.sort %>
 <% @selected = (@project.nil? ? @configuration_names[0] : @project.configuration_name) %>
-<%= required labelled_form_field _('Configuration'),
-  f.select(:configuration_name, @configuration_names, {:selected => @selected}) %><br/>
+<% if !@project.nil? && !@article.id.nil? %>
+  <%= required labelled_form_field _('Configuration'),
+    f.select(:configuration_name, @configuration_names, {:selected => @selected}, :disabled => 'true') %>
+<% else %>
+  <%= required labelled_form_field _('Configuration'),
+    f.select(:configuration_name, @configuration_names, {:selected => @selected}) %><br/>
+<% end %>
 <% selected = (@project.nil? ? 0 : @kalibro_client.process_period(@article.title).to_i) %>
 <%= required labelled_form_field _('Periodic Avaliation'),
-<% the_module = module_result.module %>
-<% module_label = "#{the_module.name} (#{the_module.granularity})" %>
+<% unless @content.errors[:base].nil? %>
+  <%= @content.errors[:base] %>
+<% else %>
+  <% the_module = @module_result.module %>
+  <% module_label = "#{the_module.name} (#{the_module.granularity})" %>
-<h5><%= _('Metric results for: ') + module_label %> </h5>
+  <h5><%= _('Metric results for: ') + module_label %> </h5>
-<hr/>
-<div class="zoomable-image">
-<table>
-  <thead>
-    <tr>
-      <th>Metric</th>
-      <th>Value</th>
-      <th>Weight</th>
-      <th>Threshold</th>
-    </tr>
-  </thead>
-  <tbody>
-    <% module_result.metric_results.each do |metric_result|  %>
-      <% range = metric_result.range %>
-      <% if !range.nil? %>
-        <tr>
-          <td><a href="#" data-show=".<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"><%= metric_result.metric.name %></a></td>
-          <td><%= MezuroPlugin::Helpers::ContentViewerHelper.format_grade(metric_result.value) %></td>
-          <td><%= metric_result.weight %></td>
-          <td style="background-color: #<%= range.color[2..-1] %>"><%= range.label %></td>
-        </tr>	  
-        <tr class="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" style="display: none;">
-          <td colspan="3">
-            <div id='historical-<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>'>
-              <a href="#" show-metric-history="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" data-module-name="<%= the_module.name %>" data-metric-name="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"> <p style="text-indent: 3em;"> Get Historical Values </p> </a>
-            </div>
-          </td>
-          <td align="right">
-            <%= range.comments.nil? ? '' : range.comments %>
-          </td> 
-        </tr>
+  <hr/>
+  <div class="zoomable-image">
+  <table>
+    <thead>
+      <tr>
+        <th>Metric</th>
+        <th>Value</th>
+        <th>Weight</th>
+        <th>Threshold</th>
+      </tr>
+    </thead>
+    <tbody>
+      <% @module_result.metric_results.each do |metric_result|  %>
+        <% range = metric_result.range %>
+        <% if !range.nil? %>
+          <tr>
+            <td><a href="#" data-show=".<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"><%= metric_result.metric.name %></a></td>
+            <td><%= MezuroPlugin::Helpers::ContentViewerHelper.format_grade(metric_result.value) %></td>
+            <td><%= metric_result.weight %></td>
+            <td style="background-color: #<%= range.color[2..-1] %>"><%= range.label %></td>
+          </tr>	  
+          <tr class="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" style="display: none;">
+            <td colspan="3">
+              <div id='historical-<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>'>
+                <a href="#" show-metric-history="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" data-module-name="<%= the_module.name %>" data-metric-name="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"> <p style="text-indent: 3em;"> Get Historical Values </p> </a>
+              </div>
+            </td>
+            <td align="right">
+              <%= range.comments.nil? ? '' : range.comments %>
+            </td> 
+          </tr>
+        <% end %>
       <% end %>
-    <% end %>
-  </tbody>
-  <tfoot>
-    <tr>
-      <td colspan = "3">
-        <div id='historical-grade'></div>
-      </td>
-      <td align = "right">
-        <a href="#" show-grade-history="<%= module_result.module.name %>" data-module-name="<%= the_module.name%>" >
-          <strong>
-            <%= _('Grade:') %>
-            <%= "%.02f" % module_result.grade %>
-          </strong>
-        </a>
-      </td>
-    </tr>
-  </tfoot>
-</table>
-</div>
+    </tbody>
+    <tfoot>
+      <tr>
+        <td colspan = "3">
+          <div id='historical-grade'></div>
+        </td>
+        <td align = "right">
+          <a href="#" show-grade-history="<%= @module_result.module.name %>" data-module-name="<%= the_module.name%>" >
+            <strong>
+              <%= _('Grade:') %>
+              <%= "%.02f" % @module_result.grade %>
+            </strong>
+          </a>
+        </td>
+      </tr>
+    </tfoot>
+  </table>
+  </div>
+<% end %>
 <h3><%= _('ERROR') %></h3>
 <p>
-  <%= "State when error ocurred: #{project.state}" %>
+  <%= "State when error ocurred: #{@project.state}" %>
   <br/>
-  <% error = project.error %>
+  <% error = @project.error %>
   <%= error.message %>
 <ul>
   <% error.stack_trace.each do |trace| %>
-
-<% form_for :project_date, :html=>{:id=>"project_history_date"} do |f| %>
-	<%= f.label :day, "Choose project date:" %>
+<% unless @content.errors[:base].nil? %>
+  <%= @content.errors[:base] %>
+<% else %>
+  <% form_for :project_date, :html=>{:id=>"project_history_date"} do |f| %>
+	  <%= f.label :day, "Choose project date:" %>
-<table>
-	<tr>
-		<td>
-			Day
-		</td>
-		<td>
-			Month
-		</td>
-		<td>
-			Year
-		</td>
-	</tr>
-	<tr>
-		<td>
-			<%= f.text_field :day, :size => 1, :maxlength => 2, :placeholder =>"dd" %>
-			</td>
-		<td>
-			<%= f.text_field :month, :size => 1, :maxlength => 2, :placeholder =>"mm"  %>
-		</td>
-		<td>
-			<%= f.text_field :year, :size => 1, :maxlength => 4, :placeholder =>"yyyy"   %>
-		</td>
-	</tr>
-</table>
-  <%= f.submit "Refresh" %>
-<% end %>
+  <table>
+	  <tr>
+		  <td>
+			  Day
+		  </td>
+		  <td>
+			  Month
+		  </td>
+		  <td>
+			  Year
+		  </td>
+	  </tr>
+	  <tr>
+		  <td>
+			  <%= f.text_field :day, :size => 1, :maxlength => 2, :placeholder =>"dd" %>
+			  </td>
+		  <td>
+			  <%= f.text_field :month, :size => 1, :maxlength => 2, :placeholder =>"mm"  %>
+		  </td>
+		  <td>
+			  <%= f.text_field :year, :size => 1, :maxlength => 4, :placeholder =>"yyyy"   %>
+		  </td>
+	  </tr>
+  </table>
+    <%= f.submit "Refresh" %>
+  <% end %>
- 
-<h4><%= _('Last Result') %></h4>
+   
+  <h4><%= _('Last Result') %></h4>
-<table>
-  <tr>
-    <td><%= _('Date') %></td>
-    <td><%= project_result.date %></td>
-  </tr>
-  <tr>
-    <td><%= _('Load time') %></td>
-    <td><%= project_result.formatted_load_time %></td>
-  </tr>
-  <tr>
-    <td><%= _('Analysis time') %></td>
-    <td><%= project_result.formatted_analysis_time %></td>
-  </tr>
-</table>
+  <table>
+    <tr>
+      <td><%= _('Date') %></td>
+      <td><%= @project_result.date %></td>
+    </tr>
+    <tr>
+      <td><%= _('Load time') %></td>
+      <td><%= @project_result.formatted_load_time %></td>
+    </tr>
+    <tr>
+      <td><%= _('Analysis time') %></td>
+      <td><%= @project_result.formatted_analysis_time %></td>
+    </tr>
+  </table>
+<% end %>
-<%= image_tag(MezuroPlugin::Helpers::ContentViewerHelper.generate_chart(score_history)) %>
+<%= image_tag(MezuroPlugin::Helpers::ContentViewerHelper.generate_chart(@score_history)) %>
-<h4><%= _('Source tree') %></h4>
-<% module_name = source_tree.module.name %>
-<% module_label = "#{module_name} (#{source_tree.module.granularity})" %>
+<% unless @content.errors[:base].nil? %>
+  <%= @content.errors[:base] %>
+<% else %>
+  <h4><%= _('Source tree') %></h4>
+  <% module_name = @source_tree.module.name %>
+  <% module_label = "#{module_name} (#{@source_tree.module.granularity})" %>
-<p><h2 class="path">
-  <% if module_name != project_name %>
-    <a href="#" class="source-tree-link" data-module-name="<%= project_name %>">
-      <%= project_name %>
-    </a>
-  <% end %>
+  <p><h2 class="path">
+    <% if module_name != @project_name %>
+      <a href="#" class="source-tree-link" data-module-name="<%= @project_name %>">
+        <%= @project_name %>
+      </a>
+    <% end %>
-  <% split_link = source_tree.module.ancestor_names %>
-  <% split_link.each do |link| %>
-    /<a href="#" class="source-tree-link" data-module-name="<%= link %>">
-      <%= link.split(".").last %>
-    </a>
+    <% split_link = @source_tree.module.ancestor_names %>
+    <% split_link.each do |link| %>
+      /<a href="#" class="source-tree-link" data-module-name="<%= link %>">
+        <%= link.split(".").last %>
+      </a>
+    <% end %>
+  </h2></p>
+    
+  <% if @source_tree.children %>
+    <table border="0" class="source-tree">
+      <% @source_tree.children.each do |child| %>
+        <% if child.module.granularity=='PACKAGE' %>
+          <tr>
+	          <td class="icon"><%= image_tag('/plugins/mezuro/images/folder.png')%></td>
+	          <td class="source-tree-text"><a href='#' class="source-tree-link" data-module-name="<%= child.module.name %>"><%= child.module.name %></a></td>
+          </tr>
+        <% else %>
+          <tr>
+	          <td class="icon"><%= image_tag('/plugins/mezuro/images/file.png') %></td>
+	          <td class="source-tree-text">
+                    <a href='#' class="source-tree-link" data-module-name="<%= child.module.name %>">
+                      <%= child.module.name %>
+                    </a>
+                  </td>
+          </tr>
+        <% end %>
+      <% end %>  
+    </table>
   <% end %>
-</h2></p>
-  
-<% if source_tree.children %>
-  <table border="0" class="source-tree">
-    <% source_tree.children.each do |child| %>
-      <% if child.module.granularity=='PACKAGE' %>
-        <tr>
-	        <td class="icon"><%= image_tag('/plugins/mezuro/images/folder.png')%></td>
-	        <td class="source-tree-text"><a href='#' class="source-tree-link" data-module-name="<%= child.module.name %>"><%= child.module.name %></a></td>
-        </tr>
-      <% else %>
-        <tr>
-	        <td class="icon"><%= image_tag('/plugins/mezuro/images/file.png') %></td>
-	        <td class="source-tree-text">
-                  <a href='#' class="source-tree-link" data-module-name="<%= child.module.name %>">
-                    <%= child.module.name %>
-                  </a>
-                </td>
-        </tr>
-      <% end %>
-    <% end %>  
-  </table>
 <% end %>
 <script src="/plugins/mezuro/javascripts/project_content.js" type="text/javascript"></script>
 <% @project = @page.project %>
-<% if (@project==nil) %>
-  <h3>Warning:</h3>
- <p>This project doesn't exist on the Web Service. Do you want to <a href="/myprofile/<%= @page.profile.name %>/cms/destroy/<%= @page.id%>">delete</a> or <a href="/myprofile/<%= @page.profile.name %>/cms/edit/<%= @page.id%>">save it again</a>?</p>
+<% unless @page.errors[:base].nil? %>
+  <%= @page.errors[:base] %>
 <% else %>
+  <% if (@project==nil) %>
+    <h3>Warning:</h3>
+    <p>This project doesn't exist on the Web Service. Do you want to <a href="/myprofile/<%= @page.profile.name %>/cms/destroy/<%= @page.id%>">delete</a> or <a href="/myprofile/<%= @page.profile.name %>/cms/edit/<%= @page.id%>">save it again</a>?</p>
+  <% else %>
-<table>
-  <tr>
-    <td><%= _('Name') %></td>
-    <td><%= @project.name %></td>
-  </tr>
-  <tr>
-    <td><%= _('License') %></td>
-    <td><%= @project.license %></td>
-  </tr>
-  <tr>
-    <td><%= _('Description') %></td>
-    <td><%= @project.description %></td>
-  </tr>
-  <tr>
-    <td><%= _('Repository type') %></td>
-    <td><%= @project.repository.type %></td>
-  </tr>
-  <tr>
-    <td><%= _('Repository address') %></td>
-    <td><%= @project.repository.address %></td>
-  </tr>
-  <tr>
-    <td><%= _('Configuration') %></td>
-    <td><%= @project.configuration_name %></td>
-  </tr>
-  <tr> 
-    <td><%= _('Periodicity') %></td>
-    <td><%= MezuroPlugin::Helpers::ContentViewerHelper.get_periodicity_option(@page.periodicity_in_days) %></td>
-  </tr>
-  <tr>
-    <td><%= _('Status')%></td>
-    <td>
-      <div id="project-state"><%= @project.state %></div>
-      <div id="msg-time"></div> 
-    </td>
-  </tr>
-</table>
-<br />
+    <table>
+      <tr>
+        <td><%= _('Name') %></td>
+        <td><%= @project.name %></td>
+      </tr>
+      <tr>
+        <td><%= _('License') %></td>
+        <td><%= @project.license %></td>
+      </tr>
+      <tr>
+        <td><%= _('Description') %></td>
+        <td><%= @project.description %></td>
+      </tr>
+      <tr>
+        <td><%= _('Repository type') %></td>
+        <td><%= @project.repository.type %></td>
+      </tr>
+      <tr>
+        <td><%= _('Repository address') %></td>
+        <td><%= @project.repository.address %></td>
+      </tr>
+      <tr>
+        <td><%= _('Configuration') %></td>
+        <td><%= @project.configuration_name %></td>
+      </tr>
+      <tr> 
+        <td><%= _('Periodicity') %></td>
+        <td><%= MezuroPlugin::Helpers::ContentViewerHelper.get_periodicity_option(@page.periodicity_in_days) %></td>
+      </tr>
+      <tr>
+        <td><%= _('Status')%></td>
+        <td>
+          <div id="project-state"><%= @project.state %></div>
+          <div id="msg-time"></div> 
+        </td>
+      </tr>
+    </table>
-<div id="project-result" data-profile="<%= @page.profile.identifier %>" data-content="<%= @page.id %>"
-     data-project-name="<%= @project.name %>">
-</div>
-<div id="project-tree"></div>
-<div id="module-result">
-</div>
-<% end %>
+    <br />
+    <div id="project-result" data-profile="<%= @page.profile.identifier %>" data-content="<%= @page.id %>"
+         data-project-name="<%= @project.name %>">
+    </div>
+    <div id="project-tree"></div>
+    <div id="module-result">
+    </div>
+  <% end %>
+<% end %>
1	-<% the_module = module_result.module %>
2	-<% module_label = "#{the_module.name} (#{the_module.granularity})" %>	1	+<% unless @content.errors[:base].nil? %>
		2	+ <%= @content.errors[:base] %>
		3	+<% else %>
		4	+ <% the_module = @module_result.module %>
		5	+ <% module_label = "#{the_module.name} (#{the_module.granularity})" %>
3		6
4	-<h5><%= _('Metric results for: ') + module_label %> </h5>	7	+ <h5><%= _('Metric results for: ') + module_label %> </h5>
5		8
6	-<hr/>
7	-<div class="zoomable-image">
8	-<table>
9	- <thead>
10	- <tr>
11	- <th>Metric</th>
12	- <th>Value</th>
13	- <th>Weight</th>
14	- <th>Threshold</th>
15	- </tr>
16	- </thead>
17	- <tbody>
18	- <% module_result.metric_results.each do \|metric_result\| %>
19	- <% range = metric_result.range %>
20	- <% if !range.nil? %>
21	- <tr>
22	- <td><a href="#" data-show=".<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"><%= metric_result.metric.name %></a></td>
23	- <td><%= MezuroPlugin::Helpers::ContentViewerHelper.format_grade(metric_result.value) %></td>
24	- <td><%= metric_result.weight %></td>
25	- <td style="background-color: #<%= range.color[2..-1] %>"><%= range.label %></td>
26	- </tr>
27	- <tr class="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" style="display: none;">
28	- <td colspan="3">
29	- <div id='historical-<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>'>
30	- <a href="#" show-metric-history="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" data-module-name="<%= the_module.name %>" data-metric-name="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"> <p style="text-indent: 3em;"> Get Historical Values </p> </a>
31	- </div>
32	- </td>
33	- <td align="right">
34	- <%= range.comments.nil? ? '' : range.comments %>
35	- </td>
36	- </tr>	9	+ <hr/>
		10	+ <div class="zoomable-image">
		11	+ <table>
		12	+ <thead>
		13	+ <tr>
		14	+ <th>Metric</th>
		15	+ <th>Value</th>
		16	+ <th>Weight</th>
		17	+ <th>Threshold</th>
		18	+ </tr>
		19	+ </thead>
		20	+ <tbody>
		21	+ <% @module_result.metric_results.each do \|metric_result\| %>
		22	+ <% range = metric_result.range %>
		23	+ <% if !range.nil? %>
		24	+ <tr>
		25	+ <td><a href="#" data-show=".<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"><%= metric_result.metric.name %></a></td>
		26	+ <td><%= MezuroPlugin::Helpers::ContentViewerHelper.format_grade(metric_result.value) %></td>
		27	+ <td><%= metric_result.weight %></td>
		28	+ <td style="background-color: #<%= range.color[2..-1] %>"><%= range.label %></td>
		29	+ </tr>
		30	+ <tr class="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" style="display: none;">
		31	+ <td colspan="3">
		32	+ <div id='historical-<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>'>
		33	+ <a href="#" show-metric-history="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>" data-module-name="<%= the_module.name %>" data-metric-name="<%= MezuroPlugin::Helpers::ContentViewerHelper.format_name(metric_result) %>"> <p style="text-indent: 3em;"> Get Historical Values </p> </a>
		34	+ </div>
		35	+ </td>
		36	+ <td align="right">
		37	+ <%= range.comments.nil? ? '' : range.comments %>
		38	+ </td>
		39	+ </tr>
		40	+ <% end %>
37	<% end %>	41	<% end %>
38	- <% end %>
39	- </tbody>
40	- <tfoot>
41	- <tr>
42	- <td colspan = "3">
43	- <div id='historical-grade'></div>
44	- </td>
45	- <td align = "right">
46	- <a href="#" show-grade-history="<%= module_result.module.name %>" data-module-name="<%= the_module.name%>" >
47	- <strong>
48	- <%= _('Grade:') %>
49	- <%= "%.02f" % module_result.grade %>
50	- </strong>
51	- </a>
52	- </td>
53	- </tr>
54	- </tfoot>
55	-</table>
56	-</div>	42	+ </tbody>
		43	+ <tfoot>
		44	+ <tr>
		45	+ <td colspan = "3">
		46	+ <div id='historical-grade'></div>
		47	+ </td>
		48	+ <td align = "right">
		49	+ <a href="#" show-grade-history="<%= @module_result.module.name %>" data-module-name="<%= the_module.name%>" >
		50	+ <strong>
		51	+ <%= _('Grade:') %>
		52	+ <%= "%.02f" % @module_result.grade %>
		53	+ </strong>
		54	+ </a>
		55	+ </td>
		56	+ </tr>
		57	+ </tfoot>
		58	+ </table>
		59	+ </div>
		60	+<% end %>
1	<h3><%= _('ERROR') %></h3>	1	<h3><%= _('ERROR') %></h3>
2	<p>	2	<p>
3	- <%= "State when error ocurred: #{project.state}" %>	3	+ <%= "State when error ocurred: #{@project.state}" %>
4	<br/>	4	<br/>
5	- <% error = project.error %>	5	+ <% error = @project.error %>
6	<%= error.message %>	6	<%= error.message %>
7	<ul>	7	<ul>
8	<% error.stack_trace.each do \|trace\| %>	8	<% error.stack_trace.each do \|trace\| %>
1	-
2	-<% form_for :project_date, :html=>{:id=>"project_history_date"} do \|f\| %>
3	- <%= f.label :day, "Choose project date:" %>	1	+<% unless @content.errors[:base].nil? %>
		2	+ <%= @content.errors[:base] %>
		3	+<% else %>
		4	+ <% form_for :project_date, :html=>{:id=>"project_history_date"} do \|f\| %>
		5	+ <%= f.label :day, "Choose project date:" %>
4		6
5	-<table>
6	- <tr>
7	- <td>
8	- Day
9	- </td>
10	- <td>
11	- Month
12	- </td>
13	- <td>
14	- Year
15	- </td>
16	- </tr>
17	- <tr>
18	- <td>
19	- <%= f.text_field :day, :size => 1, :maxlength => 2, :placeholder =>"dd" %>
20	- </td>
21	- <td>
22	- <%= f.text_field :month, :size => 1, :maxlength => 2, :placeholder =>"mm" %>
23	- </td>
24	- <td>
25	- <%= f.text_field :year, :size => 1, :maxlength => 4, :placeholder =>"yyyy" %>
26	- </td>
27	- </tr>
28	-</table>
29	- <%= f.submit "Refresh" %>
30	-<% end %>	7	+ <table>
		8	+ <tr>
		9	+ <td>
		10	+ Day
		11	+ </td>
		12	+ <td>
		13	+ Month
		14	+ </td>
		15	+ <td>
		16	+ Year
		17	+ </td>
		18	+ </tr>
		19	+ <tr>
		20	+ <td>
		21	+ <%= f.text_field :day, :size => 1, :maxlength => 2, :placeholder =>"dd" %>
		22	+ </td>
		23	+ <td>
		24	+ <%= f.text_field :month, :size => 1, :maxlength => 2, :placeholder =>"mm" %>
		25	+ </td>
		26	+ <td>
		27	+ <%= f.text_field :year, :size => 1, :maxlength => 4, :placeholder =>"yyyy" %>
		28	+ </td>
		29	+ </tr>
		30	+ </table>
		31	+ <%= f.submit "Refresh" %>
		32	+ <% end %>
31		33
32	-
33	-<h4><%= _('Last Result') %></h4>	34	+
		35	+ <h4><%= _('Last Result') %></h4>
34		36
35	-<table>
36	- <tr>
37	- <td><%= _('Date') %></td>
38	- <td><%= project_result.date %></td>
39	- </tr>
40	- <tr>
41	- <td><%= _('Load time') %></td>
42	- <td><%= project_result.formatted_load_time %></td>
43	- </tr>
44	- <tr>
45	- <td><%= _('Analysis time') %></td>
46	- <td><%= project_result.formatted_analysis_time %></td>
47	- </tr>
48	-</table>	37	+ <table>
		38	+ <tr>
		39	+ <td><%= _('Date') %></td>
		40	+ <td><%= @project_result.date %></td>
		41	+ </tr>
		42	+ <tr>
		43	+ <td><%= _('Load time') %></td>
		44	+ <td><%= @project_result.formatted_load_time %></td>
		45	+ </tr>
		46	+ <tr>
		47	+ <td><%= _('Analysis time') %></td>
		48	+ <td><%= @project_result.formatted_analysis_time %></td>
		49	+ </tr>
		50	+ </table>
		51	+<% end %>
1	-<%= image_tag(MezuroPlugin::Helpers::ContentViewerHelper.generate_chart(score_history)) %>	1	+<%= image_tag(MezuroPlugin::Helpers::ContentViewerHelper.generate_chart(@score_history)) %>
2		2