his_emr_api_lab 0.0.8 → 0.0.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b410e215fad55950c150d2ec7a82bda3f434cca4cc287e0d4fca2a9116e9182
-  data.tar.gz: 6b3a0c75c303c2139c1f6f963875667c4606bce42d23f6a22c277412e24db960
+  metadata.gz: '00645812f301ba486329bfe76c91406af972fb152d9b08be82ca09bedb1715a2'
+  data.tar.gz: 9c1f1136ed92e903059e40adb0c66aa6f4ed7057fce3d71e70b973655df08679
 SHA512:
-  metadata.gz: 4f6068aecbc02d5c3df75ac422c4d4f9baeac86c6559a1e75078ef68088b4e6702239a366fb5e5879a2925592e3440a2880668bcc50f4730c9d2051f522ac94a
-  data.tar.gz: 5e3b4d61f507f2de7847951dcdd738a50dd29ba4ae5f981ce8284f3ba11c35d6d1c337787c1b78154b96eb318fc5343d404a3827fd02c89ffc3da60eb2d617c3
+  metadata.gz: 5894ad876b761667077801c771b39d4f8dd7347a0cebe10886c513d6d2c9401dec7296c6dbf66788f5e2ceff05b15368de96c1c49a4223a560c1f44e5154ea95
+  data.tar.gz: c85f640154e2612b0b3943cc2332f7de7ee65db3a8401a241c55e53d9061cadb5ea24e69a4ca9d4a647f6468a6d5fa6160e2a8126d0e68865f551cab85e1e2c3

data/app/services/lab/labelling_service/order_label.rb

@@ -60,11 +60,14 @@ module Lab
       def drawer
         return 'N/A' if order.concept_id == unknown_concept.concept_id
 
-        name = PersonName.find_by_person_id(order.creator)
-        return "#{name.given_name} #{name.family_name}" if name
+        drawer_id = order.discontinued_by || order.creator
+        draw_date = (order.discontinued_date || order.start_date).strftime('%d/%^b/%Y %H:%M:%S')
 
-        user = User.find(order.creator)
-        user&.username || 'N/A'
+        name = PersonName.find_by_person_id(drawer_id)
+        return "#{name.given_name} #{name.family_name} #{draw_date}" if name
+
+        user = User.find_by_user_id(drawer_id)
+        user ? "#{user.username} #{draw_date}" : 'N/A'
       end
 
       def specimen

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

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require_relative './api'
+require_relative './order_dto'
+
+module Lab
+  module Lims
+    ##
+    # Manage LIMS orders that failed to import
+    module FailedImports
+      class << self
+        ##
+        # Retrieve all imports that failed
+        def failed_imports(start_id = 0, limit = 20)
+          Lab::LimsFailedImport.where('id >= ?', start_id).limit(limit)
+        end
+
+        ##
+        # Forcefully imports a failed import into a patient
+        def force_import(failed_import_id, _patient_id)
+          failed_import = Lab::LimsFailedImport.find(failed_import_id)
+          order_dto = Lab::Lims::OrderDTO.new(lims_api.find_order(failed_import.lims_id))
+          byebug
+        end
+
+        private
+
+        def lims_api
+          @lims_api ||= Lab::Lims::Api.new
+        end
+      end
+    end
+  end
+end

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

@@ -44,7 +44,7 @@ module Lab
   module Lims
     module Migrator
       class MigratorApi < Api
-        MAX_THREADS = 6
+        MAX_THREADS = (ENV.fetch('MIGRATION_WORKERS') { 6 }).to_i
 
         attr_reader :rejections
 

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

@@ -69,6 +69,10 @@ module Lab
       end
 
       def start_date
+        if self['date_created'].blank?
+          raise LimsException, 'Order missing created date'
+        end
+
         Utils.parse_date(self['date_created'])
       end
 

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

@@ -66,6 +66,8 @@ module Lab
           Date.strptime(str_date, '%d-%m-%Y').strftime('%Y-%m-%d')
         elsif str_date.match?(/(\d{4}\d{2}\d{2})\d+/)
           Date.strptime(str_date, '%Y%m%d').strftime('%Y-%m-%d')
+        elsif str_date.match?(%r{\d{2}/\d{2}/\d{4}})
+          str_date.to_date.to_s
         else
           Rails.logger.warn("Invalid date: #{str_date}")
           parse_date(fallback_date)

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

@@ -139,9 +139,17 @@ module Lab
       def find_patient_by_nhid(nhid)
         national_id_type = PatientIdentifierType.where(name: ['National id', 'Old Identification Number'])
         identifiers = PatientIdentifier.where(type: national_id_type, identifier: nhid)
+                                       .joins('INNER JOIN person ON person.person_id = patient_identifier.patient_id AND person.voided  = 0')
         if identifiers.count.zero?
-          identifiers = PatientIdentifier.unscoped.where(voided: 1, type: national_id_type, identifier: nhid)
+          identifiers = PatientIdentifier.unscoped
+                                         .where(voided: 1, type: national_id_type, identifier: nhid)
+                                         .joins('INNER JOIN person ON person.person_id = patient_identifier.patient_id AND person.voided  = 0')
         end
+
+        # Joining to person above to ensure that the person is not voided,
+        # it was noted at one site that there were some people that were voided
+        # upon merging but the patient and patient_identifier was not voided
+
         return nil if identifiers.count.zero?
 
         patients = Patient.where(patient_id: identifiers.select(:patient_id))

data/lib/lab/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: his_emr_api_lab
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.13
 platform: ruby
 authors:
 - Elizabeth Glaser Pediatric Foundation Malawi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-27 00:00:00.000000000 Z
+date: 2021-05-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
@@ -252,6 +252,7 @@ files:
 - app/services/lab/lims/api.rb
 - app/services/lab/lims/config.rb
 - app/services/lab/lims/exceptions.rb
+- app/services/lab/lims/failed_imports.rb
 - app/services/lab/lims/migrator.rb
 - app/services/lab/lims/order_dto.rb
 - app/services/lab/lims/order_serializer.rb