his_emr_api_lab 1.1.15 → 1.1.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a0345aafa5a183730006a41e0451a129eea8b201622af42c9ae9b13fb23e410d
-  data.tar.gz: d2003b6e1872c4d3c16b1bc80edea36d4ac353d62b37aad71932119085c5aecb
+  metadata.gz: 1417f8496820741d5eb37f1d2ae95da6ffdb27cc3b434242d079f6780d7eaeec
+  data.tar.gz: 17bf60bd54605de176c26f87f7da8653789d1df101920814b354512d8cca3c68
 SHA512:
-  metadata.gz: afb1c33968cb197c30b9f702b04188e93c886982a2304dc21aefcfd46add7aeb62df2307d53c6f2794dba8895a049f131b82835046280de920996d70eb6f1b84
-  data.tar.gz: ad3514949b17d623983acd6c8f7c4af305da52e4eda81e2345d94d7390e2d35aa7796d91397d819f0f64efa73803a011b4e9bd46baa574b265b04346da30b1d2
+  metadata.gz: 6c5722a094de688a6803de102553bdc928e29b28b208bf12f999a98e5d631cbc5b817c5982e5a1bc349a366263dcd3bd66ab2a1618a70b8d4ecbd8f2c86c0644
+  data.tar.gz: 8ed510173d514efcdf307cbb38d70fb4e25b19752bb95f1f20a0f4a0cdc6efd5b2d3cf5761648b8a5ac92d17072fb4a35bdbca0de929937ecccbe6525e30cfaf

data/app/controllers/lab/orders_controller.rb

@@ -8,14 +8,15 @@ module Lab
         OrdersService.order_test(order_params)
       end
 
+      orders.each { |order| Lab::PushOrderJob.perform_later(order.fetch(:order_id)) }
+
       render json: orders, status: :created
     end
 
     def update
       specimen = params.require(:specimen).permit(:concept_id)
-
-      order = OrdersService.update_order(params[:id], specimen: specimen,
-                                                      force_update: params[:force_update])
+      order = OrdersService.update_order(params[:id], specimen: specimen, force_update: params[:force_update])
+      Lab::PushOrderJob.perform_later(order.fetch(:order_id))
 
       render json: order
     end
@@ -23,11 +24,13 @@ module Lab
     def index
       filters = params.permit(%i[patient_id accession_number date status])
 
+      Lab::UpdatePatientOrdersJob.perform_later(filters[:patient_id]) if filters[:patient_id]
       render json: OrdersSearchService.find_orders(filters)
     end
 
     def destroy
       OrdersService.void_order(params[:id], params[:reason])
+      Lab::VoidOrderJob.perform_later(params[:id])
 
       render status: :no_content
     end

data/app/controllers/lab/tests_controller.rb

@@ -14,7 +14,8 @@ class Lab::TestsController < ::ApplicationController
     order_id, test_concepts = test_params.require(%i[order_id tests])
     date = test_params[:date] || Date.today
 
-    tests = service.create_tests(Order.find(order_id), date, test_concepts)
+    tests = service.create_tests(Lab::LabOrder.find(order_id), date, test_concepts)
+    Lab::PushOrderJob.perform_later(order_id)
 
     render json: tests, status: :created
   end

data/app/models/lab/lab_order.rb

@@ -44,19 +44,11 @@ module Lab
     scope :drawn, -> { where.not(concept_id: ConceptName.where(name: 'Unknown').select(:concept_id)) }
     scope :not_drawn, -> { where(concept_id: ConceptName.where(name: 'Unknown').select(:concept_id)) }
 
-    after_commit :queue_lims_push
-
     def self.prefetch_relationships
       includes(:reason_for_test,
                :requesting_clinician,
                :target_lab,
                tests: [:result])
     end
-
-    private
-
-    def queue_lims_push
-      Lab::PushOrderJob.perform_later(order_id)
-    end
   end
 end

data/app/services/lab/lims/api/rest_api.rb

@@ -377,7 +377,7 @@ class Lab::Lims::Api::RestApi
     orders_without_specimen(patient_id).each { |order| orders[order.order_id] = order }
     orders_without_results(patient_id).each { |order| orders[order.order_id] = order }
     orders_without_reason(patient_id).each { |order| orders[order.order_id] = order }
-
+    
     orders.values
   end
 
@@ -394,8 +394,10 @@ class Lab::Lims::Api::RestApi
 
   def orders_without_results(patient_id = nil)
     Rails.logger.debug('Looking for orders without a result')
+    # Lab::OrdersSearchService.find_orders_without_results(patient_id: patient_id)
+    #                         .where.not(accession_number: Lab::LimsOrderMapping.select(:lims_id).where("pulled_at IS NULL"))
     Lab::OrdersSearchService.find_orders_without_results(patient_id: patient_id)
-                            .where.not(accession_number: Lab::LimsOrderMapping.select(:lims_id))
+                             .where(order_id: Lab::LimsOrderMapping.select(:order_id))
   end
 
   def orders_without_reason(patient_id = nil)

data/app/services/lab/lims/pull_worker.rb

@@ -45,7 +45,7 @@ module Lab
           end
 
           update_last_seq(context.current_seq)
-        rescue DuplicateNHID
+        rescue Lab::Lims::DuplicateNHID
           logger.warn("Failed to import order due to duplicate patient NHID: #{order_dto[:patient][:id]}")
           save_failed_import(order_dto, "Duplicate local patient NHID: #{order_dto[:patient][:id]}")
         rescue MissingAccessionNumber
@@ -102,7 +102,7 @@ module Lab
                           .distinct(:patient_id)
                           .all
 
-        raise DuplicateNHID, "Duplicate National Health ID: #{nhid}" if patients.size > 1
+        raise Lab::Lims::DuplicateNHID, "Duplicate National Health ID: #{nhid}" if patients.size > 1
 
         patients.first
       end
@@ -182,14 +182,14 @@ module Lab
 
       def update_results(order, lims_results)
         logger.debug("Updating results for order ##{order[:accession_number]}: #{lims_results}")
-
+        
         lims_results.each do |test_name, test_results|
           test = find_test(order['id'], test_name)
           unless test
             logger.warn("Couldn't find test, #{test_name}, in order ##{order[:id]}")
             next
           end
-
+          
           next if test.result || test_results['results'].blank?
 
           measures = test_results['results'].map do |indicator, value|
@@ -198,7 +198,7 @@ module Lab
 
             measure
           end
-
+          
           measures = measures.compact
           next if measures.empty?
 

data/app/services/lab/lims/push_worker.rb

@@ -98,7 +98,7 @@ module Lab
 
         Lab::LabOrder.left_joins(:results)
                      .where('orders.discontinued_date > :last_updated
-                             OR obs.date_created > :last_updated',
+                             OR obs.date_created > orders.date_created',
                             last_updated: last_updated)
                      .group('orders.order_id')
                      .order(discontinued_date: :desc, date_created: :desc)

data/app/services/lab/orders_search_service.rb

@@ -5,14 +5,6 @@ module Lab
   module OrdersSearchService
     class << self
       def find_orders(filters)
-        # A bit of hack-ish solution to have a patient's orders updated upon
-        # scanning of a patient. Done in this way to deal with LIMS' lack of
-        # a notification system for lab order updates. We are limited to polling
-        # for updates on a per order basis.
-        if filters[:patient_id]
-          Lab::UpdatePatientOrdersJob.perform_later(filters[:patient_id])
-        end
-
         extra_filters = pop_filters(filters, :date, :end_date, :status)
 
         orders = Lab::LabOrder.prefetch_relationships
@@ -39,13 +31,9 @@ module Lab
         date = date&.to_date
         end_date = end_date&.to_date
 
-        if date && end_date
-          return orders.where('start_date BETWEEN ? AND ?', date, end_date + 1.day)
-        end
+        return orders.where('start_date BETWEEN ? AND ?', date, end_date + 1.day) if date && end_date
 
-        if date
-          return orders.where('start_date BETWEEN ? AND ?', date, date + 1.day)
-        end
+        return orders.where('start_date BETWEEN ? AND ?', date, date + 1.day) if date
 
         return orders.where('start_date < ?', end_date + 1.day) if end_date
 

data/app/services/lab/orders_service.rb

@@ -100,7 +100,6 @@ module Lab
 
         order.tests.each { |test| test.void(reason) }
         voided = order.void(reason)
-        Lab::VoidOrderJob.perform_later(order_id)
 
         voided
       end

data/lib/lab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Lab
-  VERSION = '1.1.15'
+  VERSION = '1.1.19'
 end

data/lib/logger_multiplexor.rb

@@ -23,10 +23,16 @@ class LoggerMultiplexor
   end
 
   def method_missing(method_name, *args)
-    @loggers.each { |logger| logger.method(method_name).call(*args) }
+    if respond_to_missing?(method_name)
+      @loggers.each { |logger| logger.send(method_name, *args) }
+    else
+      super
+    end
   end
 
   def respond_to_missing?(method_name)
-    @loggers.all? { |logger| logger.respond_to_missing?(method_name) }
+    @loggers.all? do |logger|
+      logger.respond_to?(method_name)
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: his_emr_api_lab
 version: !ruby/object:Gem::Version
-  version: 1.1.15
+  version: 1.1.19
 platform: ruby
 authors:
 - Elizabeth Glaser Pediatric Foundation Malawi
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-14 00:00:00.000000000 Z
+date: 2021-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
@@ -319,7 +319,7 @@ licenses:
 - MIT
 metadata:
   source_code_uri: https://github.com/EGPAFMalawiHIS/his_emr_api_lab
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -334,8 +334,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
-signing_key: 
+rubygems_version: 3.0.9
+signing_key:
 specification_version: 4
 summary: Lab extension for the HIS-EMR-API
 test_files: []