[ARVADOS] updated: 2bf927358c5956c5296009948c45403c54787021

[git at public.curoverse.com]

Summary of changes:
 apps/workbench/app/controllers/application_controller.rb          | 6 +++---
 apps/workbench/app/controllers/groups_controller.rb               | 4 ++--
 apps/workbench/app/models/group.rb                                | 2 +-
 apps/workbench/app/views/application/_content_layout.html.erb     | 2 +-
 services/api/app/controllers/arvados/v1/groups_controller.rb      | 2 +-
 services/api/db/structure.sql                                     | 1 +
 services/api/test/functional/arvados/v1/groups_controller_test.rb | 4 ++--
 7 files changed, 11 insertions(+), 10 deletions(-)

       via  2bf927358c5956c5296009948c45403c54787021 (commit)
      from  4eafb2b779b5048c5b5ecb04711c853a974e5796 (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 2bf927358c5956c5296009948c45403c54787021
Author: Tom Clegg <tom at curoverse.com>
Date:   Wed Aug 6 10:40:26 2014 -0400

    3444: Remove support for obsolete group_class=folder

diff --git a/apps/workbench/app/controllers/application_controller.rb b/apps/workbench/app/controllers/application_controller.rb
index df57e9e..df95335 100644
--- a/apps/workbench/app/controllers/application_controller.rb
+++ b/apps/workbench/app/controllers/application_controller.rb
@@ -577,7 +577,7 @@ class ApplicationController < ActionController::Base
   helper_method :all_projects
   def all_projects
     @all_projects ||= Group.
-      filter([['group_class','in',['project','folder']]]).order('name')
+      filter([['group_class','=','project']]).order('name')
   end
 
   helper_method :my_projects
@@ -688,7 +688,7 @@ class ApplicationController < ActionController::Base
     crumbs = []
     current = @name_link || @object
     while current
-      if current.is_a?(Group) and current.group_class.in?(['project','folder'])
+      if current.is_a?(Group) and current.group_class == 'project'
         crumbs.prepend current
       end
       if current.is_a? Link
@@ -702,7 +702,7 @@ class ApplicationController < ActionController::Base
 
   helper_method :current_project_uuid
   def current_project_uuid
-    if @object.is_a? Group and @object.group_class.in?(['project','folder'])
+    if @object.is_a? Group and @object.group_class == 'project'
       @object.uuid
     elsif @name_link.andand.tail_uuid
       @name_link.tail_uuid
diff --git a/apps/workbench/app/controllers/groups_controller.rb b/apps/workbench/app/controllers/groups_controller.rb
index 7698fdb..080386e 100644
--- a/apps/workbench/app/controllers/groups_controller.rb
+++ b/apps/workbench/app/controllers/groups_controller.rb
@@ -1,6 +1,6 @@
 class GroupsController < ApplicationController
   def index
-    @groups = Group.filter [['group_class', 'not in', ['folder', 'project']]]
+    @groups = Group.filter [['group_class', '!=', 'project']]
     @group_uuids = @groups.collect &:uuid
     @links_from = Link.where link_class: 'permission', tail_uuid: @group_uuids
     @links_to = Link.where link_class: 'permission', head_uuid: @group_uuids
@@ -8,7 +8,7 @@ class GroupsController < ApplicationController
   end
 
   def show
-    if @object.group_class.in?(['project','folder'])
+    if @object.group_class == 'project'
       redirect_to(project_path(@object))
     else
       super
diff --git a/apps/workbench/app/models/group.rb b/apps/workbench/app/models/group.rb
index 30488c3..558c587 100644
--- a/apps/workbench/app/models/group.rb
+++ b/apps/workbench/app/models/group.rb
@@ -22,7 +22,7 @@ class Group < ArvadosBase
   end
 
   def class_for_display
-    group_class.in?(['folder', 'project']) ? 'Project' : super
+    group_class == 'project' ? 'Project' : super
   end
 
   def editable?
diff --git a/apps/workbench/app/views/application/_content_layout.html.erb b/apps/workbench/app/views/application/_content_layout.html.erb
index c8e3827..519e885 100644
--- a/apps/workbench/app/views/application/_content_layout.html.erb
+++ b/apps/workbench/app/views/application/_content_layout.html.erb
@@ -1,5 +1,5 @@
 <%= content_for :content_top %>
-  <% if @object and @object.is_a?(Group) and @object.group_class.in?(['project','folder']) %>
+  <% if @object and @object.is_a?(Group) and @object.group_class == 'project' %>
   <div class="pull-right">
     <%= content_for :tab_line_buttons %>
   </div>
diff --git a/services/api/app/controllers/arvados/v1/groups_controller.rb b/services/api/app/controllers/arvados/v1/groups_controller.rb
index e7d5530..f51f37a 100644
--- a/services/api/app/controllers/arvados/v1/groups_controller.rb
+++ b/services/api/app/controllers/arvados/v1/groups_controller.rb
@@ -64,7 +64,7 @@ class Arvados::V1::GroupsController < ApplicationController
      Human, Specimen, Trait].each do |klass|
       @objects = klass.readable_by(*@read_users)
       if klass == Group
-        @objects = @objects.where('group_class in (?)', ['project', 'folder'])
+        @objects = @objects.where(group_class: 'project')
       end
       if opts[:owner_uuid]
         conds = []
diff --git a/services/api/db/structure.sql b/services/api/db/structure.sql
index 2d0c76f..aa05031 100644
--- a/services/api/db/structure.sql
+++ b/services/api/db/structure.sql
@@ -3,6 +3,7 @@
 --
 
 SET statement_timeout = 0;
+SET lock_timeout = 0;
 SET client_encoding = 'UTF8';
 SET standard_conforming_strings = on;
 SET check_function_bodies = false;
diff --git a/services/api/test/functional/arvados/v1/groups_controller_test.rb b/services/api/test/functional/arvados/v1/groups_controller_test.rb
index 5eeca07..c02da12 100644
--- a/services/api/test/functional/arvados/v1/groups_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/groups_controller_test.rb
@@ -16,11 +16,11 @@ class Arvados::V1::GroupsControllerTest < ActionController::TestCase
 
   test "get list of projects" do
     authorize_with :active
-    get :index, filters: [['group_class', 'in', ['project', 'folder']]], format: :json
+    get :index, filters: [['group_class', '=', 'project']], format: :json
     assert_response :success
     group_uuids = []
     json_response['items'].each do |group|
-      assert_includes ['folder', 'project'], group['group_class']
+      assert_equal 'project', group['group_class']
       group_uuids << group['uuid']
     end
     assert_includes group_uuids, groups(:aproject).uuid

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


hooks/post-receive
--