From 0c452f519d8497a93dd52b3aebd45d75c9640a37 Mon Sep 17 00:00:00 2001
From: Oleksii Kriuchykhin <okriuchykhin@biosistemika.com>
Date: Fri, 5 Oct 2018 00:18:42 +0200
Subject: [PATCH] Add CRUD endpoints for inventory cells [SCI-2765]

---
 .../api/v1/inventory_cells_controller.rb      | 87 +++++++++++++++++++
 app/models/repository_cell.rb                 |  1 +
 config/routes.rb                              |  7 +-
 3 files changed, 94 insertions(+), 1 deletion(-)
 create mode 100644 app/controllers/api/v1/inventory_cells_controller.rb

diff --git a/app/controllers/api/v1/inventory_cells_controller.rb b/app/controllers/api/v1/inventory_cells_controller.rb
new file mode 100644
index 000000000..28ab98a94
--- /dev/null
+++ b/app/controllers/api/v1/inventory_cells_controller.rb
@@ -0,0 +1,87 @@
+# frozen_string_literal: true
+
+module Api
+  module V1
+    class InventoryCellsController < BaseController
+      before_action :load_vars
+      before_action :load_inventory_cell, only: %i(show update destroy)
+      before_action :check_manage_permissions, only: %i(create update destroy)
+
+      def index
+        cells = @inventory_item.repository_cells
+                               .includes(Extends::REPOSITORY_SEARCH_INCLUDES)
+                               .page(params.dig(:page, :number))
+                               .per(params.dig(:page, :size))
+        render jsonapi: cells, each_serializer: InventoryCellSerializer
+      end
+
+      def create
+        column = @inventory.repository_columns
+                           .find(inventory_cell_params[:column_id])
+        cell = RepositoryCell.create_with_value!(@inventory_item,
+                                                 column,
+                                                 inventory_cell_params[:value],
+                                                 current_user)
+        render jsonapi: cell,
+               serializer: InventoryCellSerializer,
+               status: :created
+      end
+
+      def show
+        render jsonapi: @inventory_cell, serializer: InventoryCellSerializer
+      end
+
+      def update
+        value = update_inventory_cell_params[:value]
+        if @inventory_cell.value.data_changed?(value)
+          @inventory_cell.value.update_data!(value, current_user)
+          render jsonapi: @inventory_cell, serializer: InventoryCellSerializer
+        else
+          render body: nil, status: :no_content
+        end
+      end
+
+      def destroy
+        @inventory_cell.destroy!
+        render body: nil
+      end
+
+      private
+
+      def load_vars
+        @team = Team.find(params.require(:team_id))
+        render jsonapi: {}, status: :forbidden unless can_read_team?(@team)
+        @inventory = @team.repositories.find(params.require(:inventory_id))
+        @inventory_item = @inventory.repository_rows.find(params[:item_id].to_i)
+      end
+
+      def load_inventory_cell
+        @inventory_cell = @inventory_item.repository_cells
+                                         .find(params[:id].to_i)
+      end
+
+      def check_manage_permissions
+        unless can_manage_repository_rows?(@team)
+          render body: nil, status: :forbidden
+        end
+      end
+
+      def inventory_cell_params
+        unless params.require(:data).require(:type) == 'inventory_cells'
+          raise ActionController::BadRequest,
+                'Wrong object type within parameters'
+        end
+        params.require(:data).require(:attributes).require(%i(value column_id))
+        params.require(:data).require(:attributes).permit(%i(value column_id))
+      end
+
+      def update_inventory_cell_params
+        unless params.require(:data).require(:id).to_i == params[:id].to_i
+          raise ActionController::BadRequest,
+                'Object ID mismatch in URL and request body'
+        end
+        inventory_cell_params
+      end
+    end
+  end
+end
diff --git a/app/models/repository_cell.rb b/app/models/repository_cell.rb
index da502e3fa..8dea363f1 100644
--- a/app/models/repository_cell.rb
+++ b/app/models/repository_cell.rb
@@ -51,6 +51,7 @@ class RepositoryCell < ActiveRecord::Base
       cell.value = value
       value.save!
     end
+    cell
   end
 
   private
diff --git a/config/routes.rb b/config/routes.rb
index a680fa919..1d890b7fb 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -559,7 +559,12 @@ Rails.application.routes.draw do
               resources :inventory_items,
                         only: %i(index create show update destroy),
                         path: 'items',
-                        as: :items
+                        as: :items do
+                resources :inventory_cells,
+                          only: %i(index create show update destroy),
+                          path: 'cells',
+                          as: :cells
+              end
             end
             resources :projects, only: %i(index show) do
               resources :experiments, only: %i(index show) do