3178: Merge bogus _setup_requires_parameters into existing class method.
authorTom Clegg <tom@curoverse.com>
Mon, 11 Aug 2014 23:01:08 +0000 (19:01 -0400)
committerTom Clegg <tom@curoverse.com>
Mon, 11 Aug 2014 23:01:08 +0000 (19:01 -0400)
services/api/app/controllers/arvados/v1/users_controller.rb

index 1b2c6f4b20e6c44fe446674d79ea96db2855a57d..271299b6c97608c68b14d0500e86d779ba610b3d 100644 (file)
@@ -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