his_emr_api_lab 2.1.4.pre.beta → 2.1.5.streaming

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2cb84913a3c09866a51aa40a98812803bfee6f66a36c3e5c7ea4bdcb9a530724
-  data.tar.gz: 833074b20f473b97b515a11392fc768dabc0b3289b1717474e08b903120c6216
+  metadata.gz: 4aeefd371ffd0e3a4c9d460b7d7bd280b3d65868438b0760f992cf66dced202b
+  data.tar.gz: 48ca67bb59e2daebe7776f62fdafa6bc158d18351bbb161690a9a598a442b905
 SHA512:
-  metadata.gz: fa6487f57604f5ee24b2fc15e730dfc765cea9eff12051d1875560aa49832e36dc7ffab4320517eeb1f6ffe0ccb706b0e40e4bffc0bcb56d3055b77c9f72c650
-  data.tar.gz: 02a6a0b02074c8fcec8c3d9dbeaeee0a29ea6f96a62f30459f96ee904720c72a854d8251bfb111c5e6e80ae6552a658dabb409547b92119062ada27723d2d0bf
+  metadata.gz: fabfe624347d8724f40a063266d04619057111f92f512be9e847587c53d1d435caaf46dda9055cf8adde7af5d791091bc5898c54a1a4857b5a4536e11e5f723e
+  data.tar.gz: b24da9322346b8efdf05b3e671e550d3d6b6b59c8cd02b52415747acdd2b968e9f30eba0ef7b4df63e6cf9e25b4811189114f074139eac7fdaf967954569dc3c

data/app/jobs/lab/process_lab_result_job.rb

@@ -6,6 +6,7 @@ module Lab
   class ProcessLabResultJob < ApplicationJob
     queue_as :default
     def perform(results_obs_id, serializer, result_enter_by)
+      Rails.logger.info("Lab::ProcessLabResultJob: Processing result completion for #{serializer}")
       results_obs = Lab::LabResult.find(results_obs_id)
       Lab::ResultsService.process_result_completion(results_obs, serializer, result_enter_by)
     end

data/app/serializers/lab/result_serializer.rb

@@ -7,11 +7,11 @@ module Lab
     def self.serialize(result)
       result.children.map do |measure|
         value, value_type = read_value(measure)
-        concept_name = ConceptName.find_by_concept_id(measure.concept_id)
-        
-        program_id = ""
+        # Get the test catalog name instead of any random concept name
+        concept_name = get_test_catalog_concept_name(measure.concept_id)
+        program_id = ''
         if measure.obs_id.present?
-          obs = Observation.find(measure.obs_id)
+          obs = Observation.unscope(where: :obs_group_id).find(measure.obs_id)
           encounter = Encounter.find(obs.encounter_id)
           program_id = encounter.program_id
         end
@@ -19,8 +19,8 @@ module Lab
         {
           id: measure.obs_id,
           indicator: {
-            concept_id: concept_name&.concept_id,
-            name: concept_name&.name
+            concept_id: measure.concept_id,
+            name: concept_name
           },
           date: measure.obs_datetime,
           value:,
@@ -31,6 +31,12 @@ module Lab
       end
     end
 
+    def self.get_test_catalog_concept_name(concept_id)
+      return nil unless concept_id
+
+      ::ConceptAttribute.find_by(concept_id:, attribute_type: ConceptAttributeType.test_catalogue_name)&.value_reference
+    end
+
     def self.read_value(measure)
       %w[value_numeric value_coded value_boolean value_text].each do |field|
         value = measure.send(field) if measure.respond_to?(field)

data/app/services/lab/concepts_service.rb

@@ -61,7 +61,7 @@ module Lab
         INNER JOIN concept_attribute ca2 ON ca.concept_id = ca2.concept_id
           AND ca2.attribute_type_id = #{ConceptAttributeType.nlims_code.concept_attribute_type_id}
         WHERE ca.attribute_type_id = #{ConceptAttributeType.test_catalogue_name.concept_attribute_type_id}
-        AND ca.concept_id IN (#{concept_set.select(:concept_id).to_sql})
+        AND ca.concept_id IN (#{concept_set.select(:concept_set).to_sql})
         GROUP BY ca.concept_id
       SQL
     end

data/app/services/lab/json_web_token_service.rb

@@ -4,7 +4,7 @@ module Lab
   # This class is used to encode and decode the JWT token
   module JsonWebTokenService
     class << self
-      SECRET_KEY = Rails.application.secrets.secret_key_base.to_s
+      SECRET_KEY = Rails.application.credentials[:secret_key_base].to_s
 
       def encode(payload, request_ip, exp = 18.hours.from_now)
         payload[:exp] = exp.to_i

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

@@ -341,7 +341,7 @@ module Lab
         def nlims_order_exists?(tracking_number)
           response = in_authenticated_session do |headers|
             Rails.logger.info("Verifying order ##{tracking_number}")
-            RestClient.get(expand_uri("orders/#{tracking_number}", api_version: 'v2'), headers)
+            RestClient.get(expand_uri("orders/#{tracking_number}/exists", api_version: 'v2'), headers)
           end
 
           Rails.logger.info("Order ##{tracking_number} verified... Parsing...")

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

@@ -216,7 +216,7 @@ module Lab
           end
 
           next if test.result || test_results['results'].blank?
-          
+
           result_date = Time.now
           measures = test_results['results'].map do |indicator, value|
             measure = find_measure(order, indicator, value)

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

@@ -78,8 +78,12 @@ module Lab
       end
 
       def self.find_concept_by_name(name)
-        ConceptName.joins("INNER JOIN concept_attribute ON concept_attribute.concept_id = concept_name.concept_id")
-                   .where("concept_attribute.value_reference = ?", CGI.unescapeHTML(name))
+        unescaped_name = CGI.unescapeHTML(name)
+        attribute_type_id = ConceptAttributeType.test_catalogue_name.concept_attribute_type_id
+
+        ConceptName.joins('INNER JOIN concept_attribute ON concept_attribute.concept_id = concept_name.concept_id')
+                   .where('concept_attribute.attribute_type_id = ? AND concept_attribute.value_reference = ? AND concept_name.name = ?',
+                          attribute_type_id, unescaped_name, unescaped_name)
                    .first
       end
     end

data/app/services/lab/results_service.rb

@@ -20,7 +20,11 @@ module Lab
         serializer = {}
         results_obs = {}
         ActiveRecord::Base.transaction do
-          test = Lab::LabTest.find(test_id) rescue nil
+          test = begin
+            Lab::LabTest.find(test_id)
+          rescue StandardError
+            nil
+          end
           test = Lab::LabTest.find_by_uuid(test_id) if test.blank?
           encounter = find_encounter(test, encounter_id: params[:encounter_id],
                                            encounter_uuid: params[:encounter],
@@ -37,9 +41,9 @@ module Lab
 
         # force commit all transactions
         ActiveRecord::Base.connection.commit_db_transaction
-        
-        # delay job by a second
-        ProcessLabResultJob.set(wait: 1.second).perform_later(results_obs.id, serializer, result_enter_by)
+
+        # Execute job synchronously
+        ProcessLabResultJob.perform_now(results_obs.id, serializer, result_enter_by)
 
         Rails.logger.info("Lab::ResultsService: Result created for test #{test_id} #{serializer}")
         serializer
@@ -55,10 +59,10 @@ module Lab
       def precess_notification_message(result, values, result_enter_by)
         order = Order.find(result.order_id)
         data = { Type: result_enter_by,
-                 Specimen: ConceptName.find_by(concept_id: order.concept_id)&.name,
-                 'Test type': ConceptName.find_by(concept_id: result.test.value_coded)&.name,
+                 Specimen: get_test_catalog_name(order.concept_id) || ConceptName.find_by(concept_id: order.concept_id)&.name,
+                 'Test type': get_test_catalog_name(result.test.value_coded) || ConceptName.find_by(concept_id: result.test.value_coded)&.name,
                  'Accession number': order&.accession_number,
-                 'Orde date': Order.columns.include?('start_date') ? order.start_date : order.date_created,
+                 order_date: Order.columns.include?('start_date') ? order.start_date : order.date_created,
                  'ARV-Number': find_arv_number(result.person_id),
                  PatientID: result.person_id,
                  'Ordered By': Order.columns.include?('provider_id') ? order&.provider&.person&.name : Person.find(order.creator)&.name,
@@ -82,19 +86,21 @@ module Lab
       def find_encounter(test, encounter_id: nil, encounter_uuid: nil, date: nil, provider_id: nil)
         return Encounter.find(encounter_id) if encounter_id
         return Encounter.find_by_uuid(encounter_uuid) if encounter_uuid
-        encounter_type = EncounterType.find_by_name!(Lab::Metadata::ENCOUNTER_TYPE_NAME)
+
+        lab_encounter_type = EncounterType.find_by_name!(Lab::Metadata::ENCOUNTER_TYPE_NAME)
 
         encounter = Encounter.new
         encounter.patient_id = test.person_id
         encounter.program_id = test.encounter.program_id if Encounter.column_names.include?('program_id')
         encounter.visit_id = test.encounter.visit_id if Encounter.column_names.include?('visit_id')
-        encounter.type = encounter_type
-        encounter.encounter_type = encounter_type if (encounter&.encounter_type.nil? || encounter&.type.nil?)
+        # Use bracket notation to set the encounter_type column directly (bypasses association)
+        # This handles both Integer and EncounterType object
+        encounter_type_value = lab_encounter_type.is_a?(Integer) ? lab_encounter_type : lab_encounter_type.encounter_type_id
+        encounter[:encounter_type] = encounter_type_value
         encounter.encounter_datetime = date || Date.today
         encounter.provider_id = provider_id || User.current.user_id if Encounter.column_names.include?('provider_id')
-
         encounter.save!
-
+        encounter.reload
         encounter
       end
 
@@ -130,7 +136,8 @@ module Lab
       def add_measure_to_results(results_obs, params, date)
         validate_measure_params(params)
 
-        concept_id = params[:indicator][:concept_id] || Concept.find_concept_by_uuid(params.dig(:indicator, :concept))&.id
+        concept_id = params[:indicator][:concept_id] || Concept.find_concept_by_uuid(params.dig(:indicator,
+                                                                                                :concept))&.id
 
         Observation.create!(
           person_id: results_obs.person_id,
@@ -174,6 +181,13 @@ module Lab
         else raise InvalidParameterError, "Invalid boolean value: #{string}"
         end
       end
+
+      def get_test_catalog_name(concept_id)
+        return nil unless concept_id
+
+        ::ConceptAttribute.find_by(concept_id:,
+                                   attribute_type: ConceptAttributeType.test_catalogue_name)&.value_reference
+      end
     end
   end
 end

data/lib/lab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Lab
-  VERSION = '2.1.4-beta'
+  VERSION = '2.1.5.streaming'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: his_emr_api_lab
 version: !ruby/object:Gem::Version
-  version: 2.1.4.pre.beta
+  version: 2.1.5.streaming
 platform: ruby
 authors:
 - Elizabeth Glaser Pediatric Foundation Malawi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-01-29 00:00:00.000000000 Z
+date: 2026-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
@@ -42,16 +42,16 @@ dependencies:
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 7.0.6
+        version: 8.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 7.0.6
+        version: 8.0.0
 - !ruby/object:Gem::Dependency
   name: socket.io-client-simple
   requirement: !ruby/object:Gem::Requirement