[ARVADOS] updated: e0cf8505654d0e1c20b3d6ad31eefdab1fa342fe

[git at public.curoverse.com]

Summary of changes:
 apps/workbench/app/controllers/application_controller.rb         | 9 +++++++--
 apps/workbench/app/controllers/collections_controller.rb         | 2 +-
 apps/workbench/app/views/application/_choose.js.erb              | 3 ++-
 .../workbench/app/views/pipeline_templates/_choose_rows.html.erb | 5 +----
 .../app/views/pipeline_templates/_show_chooser_preview.html.erb  | 4 ++++
 services/api/app/controllers/arvados/v1/groups_controller.rb     | 3 ---
 6 files changed, 15 insertions(+), 11 deletions(-)
 create mode 100644 apps/workbench/app/views/pipeline_templates/_show_chooser_preview.html.erb

       via  e0cf8505654d0e1c20b3d6ad31eefdab1fa342fe (commit)
      from  a5e1609e6f34ee041a92fa089ccf0ed66448bf15 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e0cf8505654d0e1c20b3d6ad31eefdab1fa342fe
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Tue Jun 17 22:06:42 2014 -0400

    2884: Selecting folders works for run pipeline dialog.  Increased page size
    from 20 items to 40 items.

diff --git a/apps/workbench/app/controllers/application_controller.rb b/apps/workbench/app/controllers/application_controller.rb
index 02b9501..d7ffd9c 100644
--- a/apps/workbench/app/controllers/application_controller.rb
+++ b/apps/workbench/app/controllers/application_controller.rb
@@ -145,8 +145,13 @@ class ApplicationController < ActionController::Base
   end
 
   def choose
-    params[:limit] ||= 20
-    find_objects_for_index if !@objects
+    params[:limit] ||= 40
+    if !@objects
+      if params[:project_uuid] and !params[:project_uuid].empty?
+        @objects = Group.find(params[:project_uuid]).contents({:filters => [['uuid', 'is_a', "arvados\##{ArvadosApiClient.class_kind(model_class)}"]]})
+      end
+      find_objects_for_index if !@objects
+    end
     respond_to do |f|
       if params[:partial]
         f.json {
diff --git a/apps/workbench/app/controllers/collections_controller.rb b/apps/workbench/app/controllers/collections_controller.rb
index 593b3f8..2ca5929 100644
--- a/apps/workbench/app/controllers/collections_controller.rb
+++ b/apps/workbench/app/controllers/collections_controller.rb
@@ -43,7 +43,7 @@ class CollectionsController < ApplicationController
   end
 
   def choose
-    params[:limit] ||= 20
+    params[:limit] ||= 40
 
     filter = [['link_class','=','name'],
               ['head_uuid','is_a','arvados#collection']]
diff --git a/apps/workbench/app/views/application/_choose.js.erb b/apps/workbench/app/views/application/_choose.js.erb
index 14de170..1936bdc 100644
--- a/apps/workbench/app/views/application/_choose.js.erb
+++ b/apps/workbench/app/views/application/_choose.js.erb
@@ -7,10 +7,11 @@ $('body > .modal-container .modal .modal-footer .btn-primary').
     data('action-data', <%= raw params[:action_data] %>);
 $(".chooser-show-project").on("click", function() {
   $("#choose-scroll").html("<%=j image_tag 'ajax-loader.gif' %>");
+  $(".modal-dialog-preview-pane").html('');
   var t = $(this);
   var d = {
       partial: true,
-      multiple: <%= multiple %>
+      multiple: <%= multiple || "false" %>
     };
   if (t.attr("data-project-uuid") != null) {
     d.project_uuid = t.attr("data-project-uuid");
diff --git a/apps/workbench/app/views/pipeline_templates/_choose_rows.html.erb b/apps/workbench/app/views/pipeline_templates/_choose_rows.html.erb
index 5cec682..9aebd69 100644
--- a/apps/workbench/app/views/pipeline_templates/_choose_rows.html.erb
+++ b/apps/workbench/app/views/pipeline_templates/_choose_rows.html.erb
@@ -1,11 +1,8 @@
 <% @objects.each do |object| %>
-  <div class="row filterable selectable <%= 'multiple' if multiple %>" data-object-uuid="<%= object.uuid %>" data-preview-href="<%= url_for object %>?tab_pane=Components">
+  <div class="row filterable selectable <%= 'multiple' if multiple %>" data-object-uuid="<%= object.uuid %>" data-preview-href="<%= url_for object %>?tab_pane=chooser_preview">
     <div class="col-sm-12" style="overflow-x:hidden">
       <i class="fa fa-fw fa-gear"></i>
       <%= object.name %>
     </div>
-    <div class="col-sm-11 col-sm-push-1 arv-description-in-table">
-      <%= object.description %>
-    </div>
   </div>
 <% end %>
diff --git a/apps/workbench/app/views/pipeline_templates/_show_chooser_preview.html.erb b/apps/workbench/app/views/pipeline_templates/_show_chooser_preview.html.erb
new file mode 100644
index 0000000..f8c65ba
--- /dev/null
+++ b/apps/workbench/app/views/pipeline_templates/_show_chooser_preview.html.erb
@@ -0,0 +1,4 @@
+<div class="col-sm-11 col-sm-push-1 arv-description-in-table">
+  <%= @object.description %>
+</div>
+<%= render partial: 'show_components' %>
diff --git a/services/api/app/controllers/arvados/v1/groups_controller.rb b/services/api/app/controllers/arvados/v1/groups_controller.rb
index d7b330d..da82e81 100644
--- a/services/api/app/controllers/arvados/v1/groups_controller.rb
+++ b/services/api/app/controllers/arvados/v1/groups_controller.rb
@@ -24,9 +24,6 @@ class Arvados::V1::GroupsController < ApplicationController
      Job, PipelineInstance, PipelineTemplate,
      Collection,
      Human, Specimen, Trait].each do |klass|
-      if params[:object_kind] and klass.kind != params[:object_kind]
-        continue
-      end
       @objects = klass.readable_by(*@read_users)
       cond_sql = "#{klass.table_name}.owner_uuid = ?"
       cond_params = [@object.uuid]

-----------------------------------------------------------------------


hooks/post-receive
--