Box: add other purpose

app/assets/javascripts/components/cdx_select_autocomplete.js.jsx

@@ -1,21 +1,31 @@
 var CdxSelectAutocomplete = React.createClass({
+  getInitialState: function () {
+    return {
+      options: null,
+    };
+  },
+
   render: function () {
     return (<Select
+      ref={this.setSelectRef}
       className={this.props.className}
       name={this.props.name}
       value={this.props.value}
       placeholder={this.props.placeholder}
       searchable={true}
       clearable={true}
-      asyncOptions={this.asyncOptions.bind(this)}
+      asyncOptions={this.asyncOptions}
+      value={this.props.value || ""}
+      onChange={this.onChange}
     />);
   },
 
   asyncOptions: function (query, callback) {
-    if (!query || /^\s*$/.test(query)) {
-      return callback(null, []);
-    }
+    query = query.trim();
+    if (!query) return callback(null, []);
 
+    var autoselect = this.props.autoselect;
+    var prepareOptions = this.props.prepareOptions;
     var url = this.props.url;
     url += (url.includes("?") ? "&query=" : "?query=") + encodeURIComponent(query);
 
@@ -24,15 +34,31 @@ var CdxSelectAutocomplete = React.createClass({
       url: url,
 
       success: function (options) {
+        if (prepareOptions) {
+          options = prepareOptions.call(null, options);
+        }
         callback(null, {
           options: options,
           complete: options.size < 10,
         });
-      },
+        if (autoselect && options.length === 1 && options[0].value === query) {
+          this.selectRef.setValue(query);
+        }
+      }.bind(this),
 
       error: function (_, error) {
         callback(error);
       },
     })
   },
+
+  onChange: function (value, options) {
+    if (this.props.onSelect) {
+      this.props.onSelect.call(null, value, options);
+    }
+  },
+
+  setSelectRef: function (ref) {
+    this.selectRef = ref;
+  }
 });

app/assets/javascripts/components/samples_selector.js.jsx

@@ -0,0 +1,102 @@
+var SamplesSelector = React.createClass({
+  getInitialState: function () {
+    return {
+      samples: this.props.samples,
+    };
+  },
+
+  render: function () {
+    return (<div className="samples-selector">
+      {this.renderTitle()}
+      {this.state.samples.map(this.renderSample)}
+
+      <a className="add-samples" href="#" onClick={this.addSample}>
+        <div className="add-samples">
+          <div className="icon-circle-plus icon-blue icon-margin"></div>
+          <div className="add-sample-link">ADD SAMPLE</div>
+        </div>
+      </a>
+    </div>);
+  },
+
+  renderTitle() {
+    var samples = this.state.samples;
+    if (!samples.length) return;
+
+    var count = samples.reduce(function (a, e) {
+      return e.uuid ? a + 1 : a;
+    }, 0);
+
+    return (<div className="samples-count">
+      <div className="title">{count}&nbsp;{count == 1 ? "sample" : "samples"}</div>
+    </div>);
+  },
+
+  renderSample(sample, index) {
+    if (sample.uuid) {
+      function removeSample(event) {
+        this.removeSample(event, index);
+      }
+      return (<div className="batches-samples" key={"samples-selector-" + index}>
+        <div className="samples-row">
+          <div className="samples-item">{sample.uuid}</div>
+          <div className="samples-row-actions">
+            <input type="hidden" name={this.props.name + "[" + index + "]"} value={sample.uuid}/>
+            <span>{sample.batch_number}</span>
+            <a href="#" onClick={removeSample.bind(this)} title="Remove this sample">
+              <i className="icon-close icon-gray bigger"></i>
+            </a>
+          </div>
+        </div>
+      </div>);
+    } else {
+      function selectSample(_, options) {
+        this.selectSample(index, options && options[0]);
+      }
+      return (<div className="batches-samples" key={"samples-selector-" + index}>
+        <div className="samples-row">
+          <CdxSelectAutocomplete
+            className={this.props.className}
+            url={this.props.url}
+            placeholder={this.props.placeholder}
+            value={sample.uuid}
+            prepareOptions={this.prepareOptions}
+            autoselect={true}
+            onSelect={selectSample.bind(this)}
+          />
+        </div>
+      </div>);
+    }
+  },
+
+  prepareOptions: function (options) {
+    return options.map(function (option) {
+      option.value = option.uuid;
+      option.label = option.uuid + " (" + option.batch_number + ")";
+      return option;
+    });
+  },
+
+  addSample: function (event) {
+    event.preventDefault();
+
+    var samples = this.state.samples;
+    samples.push({ uuid: "" });
+    this.setState({ samples: samples });
+  },
+
+  selectSample: function (index, sample) {
+    var samples = this.state.samples;
+    samples[index] = sample;
+    this.setState({ samples: samples });
+  },
+
+  removeSample: function (event, index) {
+    event.preventDefault();
+
+    var samples = this.state.samples;
+    samples.splice(index, 1);
+
+    this.setState({ samples: samples });
+  },
+});

app/assets/stylesheets/_batches.scss

@@ -69,6 +69,11 @@
   align-items: center;
   min-height: 40px;
 }
+.samples-row-actions {
+  display: flex;
+  align-items: center;
+  gap: 10px;
+}
 
 .transfer-data {
   color: black;

app/assets/stylesheets/_sprites.scss

@@ -67,6 +67,9 @@
     color: inherit;
   }
 
+  &.bigger {
+    font-size: 24px;
+  }
   &.medium {
     font-size: 35px;
   }

app/controllers/boxes_controller.rb

@@ -1,5 +1,6 @@
 class BoxesController < ApplicationController
   before_action :load_box, except: %i[index new create bulk_destroy]
+  helper_method :samples_data
 
   def index
     @can_create = has_access?(@navigation_context.institution, CREATE_INSTITUTION_BOX)
@@ -59,6 +60,7 @@ def create
 
     @box_form = BoxForm.build(@navigation_context, box_params)
     @box_form.batches = check_access(load_batches, READ_BATCH)
+    @box_form.samples = check_access(load_samples, READ_SAMPLE)
 
     if @box_form.valid?
       @box_form.build_samples
@@ -106,13 +108,31 @@ def load_batches
       .where(uuid: @box_form.batch_uuids.values.reject(&:blank?))
   end
 
+  def load_samples
+    Sample
+      .within(@navigation_context.entity, @navigation_context.exclude_subsites)
+      .find_all_by_any_uuid(@box_form.sample_uuids.values.reject(&:blank?))
+  end
+
   def box_params
     if Rails::VERSION::MAJOR == 5 && Rails::VERSION::MINOR == 0
       params.require(:box).permit(:purpose, :media).tap do |allowed|
-        allowed[:batch_uuids] = params[:box][:batch_uuids].permit!
+        allowed[:batch_uuids] = params[:box][:batch_uuids].try(&:permit!)
+        allowed[:sample_uuids] = params[:box][:sample_uuids].try(&:permit!)
       end
     else
-      params.require(:box).permit(:purpose, :media, batch_uuids: {})
+      params.require(:box).permit(:purpose, :media, batch_uuids: {}, sample_uuids: [])
+    end
+  end
+
+  def samples_data(samples)
+    # NOTE: duplicates the samples/autocomplete template (but returns an
+    # Array<Hash> instead of rendering to a JSON String)
+    samples.map do |sample|
+      {
+        uuid: sample.uuid,
+        batch_number: sample.batch_number,
+      }
     end
   end
 end

app/controllers/samples_controller.rb

@@ -25,6 +25,20 @@ def index
       .preload(:batch, :sample_identifiers)
   end
 
+  def autocomplete
+    samples = Sample
+      .where(institution: @navigation_context.institution)
+      .within(@navigation_context.entity, @navigation_context.exclude_subsites)
+
+    samples = samples.without_qc if params[:qc] == "0"
+
+    @samples = check_access(samples, READ_SAMPLE)
+      .joins(:sample_identifiers)
+      .autocomplete(params[:query])
+      .limit(10)
+      .preload(:batch)
+  end
+
   def edit_or_show
     sample = Sample.find(params[:id])
 

app/models/box.rb

@@ -95,7 +95,7 @@ def blind_attributes
       %i[concentration concentration_formula replicate]
     when "Variants"
       %i[batch_number virus_lineage]
-    when "Challenge"
+    when "Challenge", "Other"
       %i[batch_number concentration concentration_formula replicate virus_lineage]
     else
       []

app/models/box_form.rb

@@ -1,5 +1,5 @@
 class BoxForm
-  attr_reader :box, :batch_uuids
+  attr_reader :box, :batch_uuids, :sample_uuids
   attr_accessor :media
 
   delegate :purpose, :purpose=, to: :box
@@ -18,6 +18,7 @@ def initialize(box, params)
     @box = box
     @media = params[:media].presence
     @batch_uuids = params[:batch_uuids].presence.to_h
+    @sample_uuids = params[:sample_uuids].presence.to_h
   end
 
   def batches=(relation)
@@ -28,6 +29,14 @@ def batches=(relation)
     end.compact
   end
 
+  def samples=(relation)
+    records = relation.to_a
+
+    @samples = @sample_uuids.transform_values do |batch_uuid|
+      records.find { |b| b.uuid == batch_uuid }
+    end.compact
+  end
+
   def build_samples
     case @box.purpose
     when "LOD"
@@ -46,13 +55,17 @@ def build_samples
           @box.build_samples(batch, concentration_exponents: [1, 4, 8], replicates: 3, media: media)
         end
       end
+
+    when "Other"
+      @box.samples = @samples.values
     end
   end
 
   def valid?
     @box.valid?
     validate_existence_of_batches
-    validate_batches_for_purpose
+    validate_existence_of_samples
+    validate_batches_or_samples_for_purpose
     @box.errors.empty?
   end
 
@@ -74,21 +87,37 @@ def validate_existence_of_batches
     end
   end
 
-  def validate_batches_for_purpose
-    count = @batches.map { |_, b| b.try(&:uuid) }.uniq.size
+  def validate_existence_of_samples
+    @sample_uuids.each do |key, sample_uuid|
+      unless sample_uuid.blank? || @samples[key]
+        @box.errors.add(key, "Sample doesn't exist")
+      end
+    end
+  end
 
+  def validate_batches_or_samples_for_purpose
     case @box.purpose
     when "LOD"
       @box.errors.add(:lod, "A batch is required") unless @batches["lod"] || @box.errors.include?(:lod)
     when "Variants"
-      @box.errors.add(:base, "You must select at least two batches") unless count >= 2
+      @box.errors.add(:base, "You must select at least two batches") unless unique_batch_count >= 2
     when "Challenge"
       if @batches["virus"]
-        @box.errors.add(:base, "You must select at least one distractor batch") unless count >= 2
+        @box.errors.add(:base, "You must select at least one distractor batch") unless unique_batch_count >= 2
       else
         @box.errors.add(:virus, "A virus batch is required") unless @box.errors.include?(:virus)
-        @box.errors.add(:base, "You must select at least one distractor batch") unless count >= 1
+        @box.errors.add(:base, "You must select at least one distractor batch") unless unique_batch_count >= 1
+      end
+    when "Other"
+      if @samples.empty?
+        @box.errors.add(:base, "You must select at least one sample")
+      elsif @samples.any? { |_, sample| sample.is_quality_control? }
+        @box.errors.add(:base, "You can't select a QC sample")
       end
     end
   end
+
+  def unique_batch_count
+    @batches.map { |_, b| b.try(&:uuid) }.uniq.size
+  end
 end

app/models/sample.rb

@@ -84,6 +84,8 @@ def self.find_all_by_any_uuid(uuids)
     joins(:sample_identifiers).order("sample_identifiers.uuid")
   }
 
+  scope :without_qc, -> { where.not(specimen_role: "q") }
+
   def self.media
     entity_fields.find { |f| f.name == 'media' }.options
   end

app/views/batches/_samples.haml

@@ -20,7 +20,7 @@
         %label.row{for: "sample_ids_#{sample.id}"}
         = check_box_tag 'destroy_sample_ids[]', sample.id, false, { id: "destroy_sample_ids_#{sample.id}", class: "destroy-checkbox" }
         = sample.uuid
-      .sample-row-actions
+      .samples-row-actions
         - if sample.is_quality_control?
           .icon-test.icon-gray{title: 'Q - Control specimen'}
         = link_to edit_sample_path(sample.id) do

app/views/batches/_show_samples.haml

@@ -8,7 +8,7 @@
     .samples-row
       .samples-item
         = sample.uuid
-      .sample-row-actions
+      .samples-row-actions
         - if sample.is_quality_control?
           .icon-test.icon-gray{title: 'Q - Control specimen'}
         = link_to sample_path(sample.id) do