Bugfix time display, also bugfix for job state "Completed" -> "Complete". refs ...

diff --git a/apps/workbench/app/helpers/pipeline_instances_helper.rb b/apps/workbench/app/helpers/pipeline_instances_helper.rb
index 599da421cfc91d5c42d9e015fd67b3829f77292e..7e5324be7af92b1709739cc01a667b40ecea4f47 100644 (file)
--- a/apps/workbench/app/helpers/pipeline_instances_helper.rb
+++ b/apps/workbench/app/helpers/pipeline_instances_helper.rb
@@ -256,7 +256,7 @@ module PipelineInstancesHelper
       if minutes > 0 then
         s << "#{minutes} minute#{'s' if minutes != 1}"
       end
-      if seconds > 0 and not round_to_min 
+      if not round_to_min or s.size == 0
         s << "#{seconds} second#{'s' if seconds != 1}"
       end
       s = s * " "

diff --git a/apps/workbench/app/views/application/_job_status_label.html.erb b/apps/workbench/app/views/application/_job_status_label.html.erb
index 86c3415453e06f2d676f37b7636e004b9773d458..ece81676d806bbbb470340581cab71d5a01a24ab 100644 (file)
--- a/apps/workbench/app/views/application/_job_status_label.html.erb
+++ b/apps/workbench/app/views/application/_job_status_label.html.erb
@@ -1,7 +1,7 @@
 <% status = Job.state j %>
 <% to_label = {
      "Cancelled" => "danger",
-     "Completed" => "success",
+     "Complete" => "success",
      "Running" => "info",
      "Failed" => "danger",
      "Queued" => "default",

diff --git a/apps/workbench/app/views/pipeline_instances/_running_component.html.erb b/apps/workbench/app/views/pipeline_instances/_running_component.html.erb
index 8f89916fba84ab64d859ff4423211c517c157b7d..1d52e281f67a390dc4802f7079e69c52f38270ad 100644 (file)
--- a/apps/workbench/app/views/pipeline_instances/_running_component.html.erb
+++ b/apps/workbench/app/views/pipeline_instances/_running_component.html.erb
@@ -31,7 +31,7 @@
               <% end %>
             </div>
 
-            <% if Job::state(current_job).in? ["Completed", "Failed", "Canceled"] %>
+            <% if Job::state(current_job).in? ["Complete", "Failed", "Canceled"] %>
               <div class="col-md-5 text-overflow-ellipsis">
                 <% if pj[:output_uuid] %>
                   <%= link_to_if_arvados_object pj[:output_uuid], friendly_name: true %>

diff --git a/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb b/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb
index 8af8c881d88f0d9b0e8b6990d042db6ba1851565..acaf9d34f47642593c05597ba867a15657cd792a 100644 (file)
--- a/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb
+++ b/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb
@@ -29,9 +29,9 @@
                   end %>
     
     <%= if walltime > runningtime 
-          render_runtime(walltime, true) 
+          render_runtime(walltime, true, false) 
         else 
-          render_runtime(runningtime, true) 
+          render_runtime(runningtime, true, false) 
         end %><% if @object.finished_at %> at <%= render_localized_date(@object.finished_at) %><% end %>.
     <% else %>
       This pipeline is <%= if @object.state.start_with? 'Running' then 'active' else @object.state.downcase end %>.
@@ -61,7 +61,7 @@
     <%= render_runtime(runningtime, true, false) %><% if (walltime - runningtime) > 0 %>
       (<%= render_runtime(walltime - runningtime, true, false) %> queued)<% end %><% if cputime == 0 %>.<% else %>
       and used
-    <%= render_runtime(cputime, true) %>
+    <%= render_runtime(cputime, true, false) %>
     of CPU time (<%= (cputime/runningtime).round(1) %>&Cross; scaling).
     <% end %>
 </p>

diff --git a/apps/workbench/app/views/projects/_show_dashboard.html.erb b/apps/workbench/app/views/projects/_show_dashboard.html.erb
index 47315941a0f74037cedde55b55ba9ae6e95a2ed3..2841637ded00df136128d4e86d8d4409e380c6e4 100644 (file)
--- a/apps/workbench/app/views/projects/_show_dashboard.html.erb
+++ b/apps/workbench/app/views/projects/_show_dashboard.html.erb
@@ -38,7 +38,7 @@
                           <% running << k %>
                         <% elsif Job::state(v[:job]) == "Failed" or Job::state(v[:job]) == "Cancelled" %>
                           <% failed << k %>
-                        <% elsif Job::state(v[:job]) == "Completed" %>
+                        <% elsif Job::state(v[:job]) == "Complete" %>
                           <% completed << k %>
                         <% elsif Job::state(v[:job]) == "Queued" %>
                           <% queued << k %>