[ARVADOS] updated: 2bec2ba1ce3b37fb8ff9a94f20d6c8a694d87183

git at public.curoverse.com git at public.curoverse.com
Mon Aug 11 19:02:48 EDT 2014


Summary of changes:
 .../app/controllers/arvados/v1/users_controller.rb | 36 +++++++++-------------
 1 file changed, 15 insertions(+), 21 deletions(-)

       via  2bec2ba1ce3b37fb8ff9a94f20d6c8a694d87183 (commit)
      from  fd38dcafdb0b89aff5f722946c7326291ad69290 (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 2bec2ba1ce3b37fb8ff9a94f20d6c8a694d87183
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon Aug 11 19:01:08 2014 -0400

    3178: Merge bogus _setup_requires_parameters into existing class method.

diff --git a/services/api/app/controllers/arvados/v1/users_controller.rb b/services/api/app/controllers/arvados/v1/users_controller.rb
index 1b2c6f4..271299b 100644
--- a/services/api/app/controllers/arvados/v1/users_controller.rb
+++ b/services/api/app/controllers/arvados/v1/users_controller.rb
@@ -57,26 +57,6 @@ class Arvados::V1::UsersController < ApplicationController
     show
   end
 
-  def _setup_requires_parameters
-    {
-      user: {
-        type: 'object', required: false
-      },
-      openid_prefix: {
-        type: 'string', required: false
-      },
-      repo_name: {
-        type: 'string', required: false
-      },
-      vm_uuid: {
-        type: 'string', required: false
-      },
-      send_notification_email: {
-        type: 'boolean', required: false, default: false
-      },
-    }
-  end
-
   # create user object and all the needed links
   def setup
     @object = nil
@@ -138,7 +118,21 @@ class Arvados::V1::UsersController < ApplicationController
 
   def self._setup_requires_parameters
     {
-      send_notification_email: { type: 'boolean', required: true },
+      user: {
+        type: 'object', required: false
+      },
+      openid_prefix: {
+        type: 'string', required: false
+      },
+      repo_name: {
+        type: 'string', required: false
+      },
+      vm_uuid: {
+        type: 'string', required: false
+      },
+      send_notification_email: {
+        type: 'boolean', required: false, default: false
+      },
     }
   end
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list