before_filter :login_required, :except => :render_not_found
before_filter :catch_redirect_hint
+ before_filter :load_where_param, :only => :index
before_filter :find_objects_for_index, :only => :index
before_filter :find_object_by_uuid, :except => [:index, :create]
attr_accessor :resource_attrs
+ def index
+ @objects.uniq!(&:id)
+ if params[:eager] and params[:eager] != '0' and params[:eager] != 0 and params[:eager] != ''
+ @objects.each(&:eager_load_associations)
+ end
+ render_list
+ end
+
+ def show
+ if @object
+ render json: @object.as_api_response(:superuser)
+ else
+ render_not_found("object not found")
+ end
+ end
+
+ def create
+ @object = model_class.new resource_attrs
+ @object.save
+ show
+ end
+
+ def update
+ if @object.update_attributes resource_attrs
+ show
+ else
+ render json: { errors: @object.errors.full_messages }, status: 422
+ end
+ end
+
+ def destroy
+ @object.destroy
+ show
+ end
+
def catch_redirect_hint
if !current_user
if params.has_key?('redirect_to') then
render json: { errors: ["Path not found"] }, status: 404
end
+ protected
+
+ def load_where_param
+ @where = params[:where] || {}
+ @where = Oj.load(@where) if @where.is_a?(String)
+ end
+
def find_objects_for_index
uuid_list = [current_user.uuid, *current_user.groups_i_can(:read)]
sanitized_uuid_list = uuid_list.
true, current_user.is_admin,
uuid_list,
current_user.uuid)
- @where = params[:where] || {}
- @where = Oj.load(@where) if @where.is_a?(String)
- if params[:where]
+ if !@where.empty?
conditions = ['1=1']
@where.each do |attr,value|
- if (!value.nil? and
- attr.to_s.match(/^[a-z][_a-z0-9]+$/) and
- model_class.columns.collect(&:name).index(attr))
- if value.is_a? Array
+ if attr == 'any'
+ if value.is_a?(Array) and
+ value[0] == 'contains' and
+ model_class.columns.collect(&:name).index('name') then
+ conditions[0] << " and #{table_name}.name ilike ?"
+ conditions << "%#{value[1]}%"
+ end
+ elsif attr.to_s.match(/^[a-z][_a-z0-9]+$/) and
+ model_class.columns.collect(&:name).index(attr)
+ if value.nil?
+ conditions[0] << " and #{table_name}.#{attr} is ?"
+ conditions << nil
+ elsif value.is_a? Array
conditions[0] << " and #{table_name}.#{attr} in (?)"
conditions << value
elsif value.is_a? String or value.is_a? Fixnum or value == true or value == false
end
end
end
- elsif (!value.nil? and attr == 'any' and
- value.is_a?(Array) and value[0] == 'contains' and
- model_class.columns.collect(&:name).index('name')) then
- conditions[0] << " and #{table_name}.name ilike ?"
- conditions << "%#{value[1]}%"
end
end
if conditions.length > 1
begin
@objects = @objects.limit(params[:limit].to_i)
rescue
- raise "invalid argument (limit)"
+ raise ArgumentError.new("Invalid value for limit parameter")
end
else
@objects = @objects.limit(100)
end
- @objects = @objects.order("#{table_name}.modified_at desc")
- end
-
- def index
- @objects.uniq!(&:id)
- if params[:eager] and params[:eager] != '0' and params[:eager] != 0 and params[:eager] != ''
- @objects.each(&:eager_load_associations)
+ orders = []
+ if params[:order]
+ params[:order].split(',').each do |order|
+ attr, direction = order.strip.split " "
+ direction ||= 'asc'
+ if attr.match /^[a-z][_a-z0-9]+$/ and
+ model_class.columns.collect(&:name).index(attr) and
+ ['asc','desc'].index direction.downcase
+ orders << "#{table_name}.#{attr} #{direction.downcase}"
+ end
+ end
end
- render_list
- end
-
- def show
- if @object
- render json: @object.as_api_response(:superuser)
- else
- render_not_found("object not found")
+ if orders.empty?
+ orders << "#{table_name}.modified_at desc"
end
+ @objects = @objects.order(orders.join ", ")
end
- def create
- @object = model_class.new resource_attrs
- @object.save
- show
- end
-
- def update
- @object.update_attributes resource_attrs
- show
- end
-
- def destroy
- @object.destroy
- show
- end
-
- protected
-
def resource_attrs
return @attrs if @attrs
@attrs = params[resource_name]
@attrs = uncamelcase_hash_keys(Oj.load @attrs)
end
unless @attrs.is_a? Hash
- raise "no #{resource_name} (or #{resource_name.camelcase(:lower)}) hash provided with request #{params.inspect}"
+ message = "No #{resource_name}"
+ if resource_name.index('_')
+ message << " (or #{resource_name.camelcase(:lower)})"
+ end
+ message << " hash provided with request"
+ raise ArgumentError.new(message)
end
%w(created_at modified_by_client modified_by_user modified_at).each do |x|
@attrs.delete x
user = nil
api_client = nil
api_client_auth = nil
- supplied_token = params[:api_token] || params[:oauth_token]
+ supplied_token =
+ params[:api_token] ||
+ params[:oauth_token] ||
+ request.headers["Authorization"].andand.match(/OAuth2 ([a-z0-9]+)/).andand[1]
if supplied_token
api_client_auth = ApiClientAuthorization.
includes(:api_client, :user).
def uncamelcase_params_hash_keys
self.params = uncamelcase_hash_keys(params)
end
-
def uncamelcase_hash_keys(h, max_depth=-1)
if h.is_a? Hash and max_depth != 0
nh = Hash.new
def render_list
@object_list = {
- :kind => "orvos##{resource_name}List",
+ :kind => "arvados##{resource_name}List",
:etag => "",
:self_link => "",
:next_page_token => "",