[ARVADOS] updated: 5da7bddd63790e524e4b22a6944a92065409241d

git at public.curoverse.com git at public.curoverse.com
Tue Jun 10 16:33:16 EDT 2014


Summary of changes:
 services/api/app/controllers/arvados/v1/schema_controller.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

       via  5da7bddd63790e524e4b22a6944a92065409241d (commit)
      from  0548dd424d6f21f3eb97f3e98d37a604b44b025a (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 5da7bddd63790e524e4b22a6944a92065409241d
Author: Ward Vandewege <ward at curoverse.com>
Date:   Tue Jun 10 16:31:59 2014 -0400

    Bugfix: the discovery doc should not clobber defaults values with the
    word 'string'. Instead, make sure the default values are always exposed
    as a string, which is something the java google apiclient is sensitive
    to.
    
    Refs no issue #

diff --git a/services/api/app/controllers/arvados/v1/schema_controller.rb b/services/api/app/controllers/arvados/v1/schema_controller.rb
index 5d907b8..4470291 100644
--- a/services/api/app/controllers/arvados/v1/schema_controller.rb
+++ b/services/api/app/controllers/arvados/v1/schema_controller.rb
@@ -367,7 +367,8 @@ class Arvados::V1::SchemaController < ApplicationController
                   method[:parameters][k] = {}
                 end
                 if !method[:parameters][k][:default].nil?
-                  method[:parameters][k][:default] = 'string'
+                  # The JAVA SDK is sensitive to all values being strings
+                  method[:parameters][k][:default] = method[:parameters][k][:default].to_s
                 end
                 method[:parameters][k][:type] ||= 'string'
                 method[:parameters][k][:description] ||= ''

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list