Merge pull request #2501 from aignatov-bio/ai-sci-4474-add-decimals-to-repository-api

Add decimals to repository column API [SCI-4474]
This commit is contained in:
Alex Kriuchykhin 2020-04-03 11:45:55 +02:00 committed by GitHub
commit 3e20efcc8e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 11 deletions

View file

@ -56,9 +56,7 @@ module Api
private
def check_manage_permissions
unless can_manage_repository_column?(@inventory_column)
raise PermissionError.new(RepositoryColumn, :manage)
end
raise PermissionError.new(RepositoryColumn, :manage) unless can_manage_repository_column?(@inventory_column)
end
def check_create_permissions
@ -66,12 +64,11 @@ module Api
end
def inventory_column_params
unless params.require(:data).require(:type) == 'inventory_columns'
raise TypeError
end
raise TypeError unless params.require(:data).require(:type) == 'inventory_columns'
params.require(:data).require(:attributes)
new_params = params
.permit(data: { attributes: %i(name data_type) })[:data]
.permit(data: { attributes: [:name, :data_type, metadata: {}] })[:data]
.merge(created_by: @current_user)
if new_params[:attributes][:data_type].present?
new_params[:attributes][:data_type] =
@ -82,9 +79,8 @@ module Api
end
def update_inventory_column_params
unless params.require(:data).require(:id).to_i == params[:id].to_i
raise IDMismatchError
end
raise IDMismatchError unless params.require(:data).require(:id).to_i == params[:id].to_i
if inventory_column_params[:attributes].include?(:data_type)
raise ActiveRecord::RecordInvalid,
I18n.t('api.core.errors.inventory_column_type.detail')

View file

@ -4,7 +4,7 @@ module Api
module V1
class InventoryColumnSerializer < ActiveModel::Serializer
type :inventory_columns
attributes :name, :data_type
attributes :name, :data_type, :metadata
has_many :repository_list_items,
key: :inventory_list_items,
serializer: InventoryListItemSerializer,