his_emr_api_lab 2.3.6 → 2.3.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0a4ea1a6183fe75cd7d56a11e934391903cab098f8892b91380d6f6e67b3b752
-  data.tar.gz: 4b9f2c6e7f8beabf94c5c474cd11a2cbafe833a4111d975786b9b65b85a093fd
+  metadata.gz: aa10ff237ade4cbd630628c0511ca04f8ab31e076a8b9b6f8413a4e0dde444f9
+  data.tar.gz: 4ca6cf07011ac9117fa00b0fee436e353dd5faad5cb71ed50656b89b4826c98e
 SHA512:
-  metadata.gz: b5562e97cd505fed7dc56e3529e7b5b38510cf3ec5d0a94dbd2fbff91a8286e7715a96bb77c6ff339cf7edf60d0eda539e57ec758109c32de26cd11892e387e7
-  data.tar.gz: '097f26ec972cd4f79e57f5719142c6224a975dad0b4c31de4e6063df96470f01885c000544b743f4d0470b09c2d63ab1180f97dad6eb585ab0d7ff6d01350e1b'
+  metadata.gz: 1a6aba5b3b1339126fd2f7f16959b1d1f1a11381d1d790eeeaca9909383fcd8324c3896b1a94d75aa9c19edc7ae7b7aafdbfb95b59073ff68b66be35732fc97e
+  data.tar.gz: 4964fbb342fc54ff534c0d90d6c43191492deb9401b7e4278ed5921be74bc32ea84cd6adbc6f6dca5c82a1fd8f6120a8412ce08b58fa55634825ae1bccb45193

data/app/controllers/lab/orders_controller.rb

@@ -58,6 +58,10 @@ module Lab
     def verify_tracking_number
       tracking_number = params.require(:accession_number)
       render json: { exists: OrdersService.check_tracking_number(tracking_number) }, status: :ok
+    rescue Lab::Lims::ValidationUnavailable => e
+      # Return 502 Bad Gateway to indicate the external service is unavailable
+      # This allows the frontend to prompt user for confirmation
+      render json: { errors: [e.message] }, status: :bad_gateway
     end
 
     def destroy
@@ -74,7 +78,7 @@ module Lab
 
     def order_status
       order_params = params.permit(:tracking_number, :status, :status_time, :comments, :status_id,
-                                    updated_by: [:first_name, :last_name, :id, :phone_number])
+                                   updated_by: %i[first_name last_name id phone_number])
       OrdersService.update_order_status(order_params)
       render json: { message: "Status for order #{order_params['tracking_number']} successfully updated" }, status: :ok
     end

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

@@ -8,17 +8,18 @@ module Lab
       class MissingAccessionNumber < LimsException; end
       class UnknownSpecimenType < LimsException; end
       class UnknownTestType < LimsException; end
+      class ValidationUnavailable < LimsException; end
     end
   end
 end
 
-
 module Lab
   module Lims
-      class LimsException < StandardError; end
-      class DuplicateNHID < LimsException; end
-      class MissingAccessionNumber < LimsException; end
-      class UnknownSpecimenType < LimsException; end
-      class UnknownTestType < LimsException; end
+    class LimsException < StandardError; end
+    class DuplicateNHID < LimsException; end
+    class MissingAccessionNumber < LimsException; end
+    class UnknownSpecimenType < LimsException; end
+    class UnknownTestType < LimsException; end
+    class ValidationUnavailable < LimsException; end
   end
 end

data/app/services/lab/orders_service.rb

@@ -56,8 +56,14 @@ module Lab
 
         Order.transaction do
           encounter = find_encounter(order_params)
-          if order_params[:accession_number].present? && check_tracking_number(order_params[:accession_number])
-            raise 'Accession number already exists'
+          if order_params[:accession_number].present?
+            begin
+              raise 'Accession number already exists' if check_tracking_number(order_params[:accession_number])
+            rescue Lab::Lims::ValidationUnavailable => e
+              # Log warning but allow order creation to proceed
+              # User should have been prompted to confirm on the frontend
+              Rails.logger.warn("Creating order with unvalidated accession number: #{e.message}")
+            end
           end
 
           order = create_order(encounter, order_params)
@@ -364,6 +370,15 @@ module Lab
         # fetch from the rest api and check if it exists
         lims_api = Lab::Lims::ApiFactory.create_api
         lims_api.verify_tracking_number(accession_number).present?
+      rescue StandardError => e
+        # Log the error for debugging
+        Rails.logger.error("Failed to verify accession number with NLIMS: #{e.message}")
+        Rails.logger.error(e.backtrace.join("\n"))
+
+        # Raise a specific exception indicating validation is unavailable
+        # This allows the controller to handle it differently than "accession exists"
+        raise Lab::Lims::ValidationUnavailable,
+              "Failed to communicate with external service: #{e.message}"
       end
 
       ##

data/lib/lab/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: his_emr_api_lab
 version: !ruby/object:Gem::Version
-  version: 2.3.6
+  version: 2.3.7
 platform: ruby
 authors:
 - Elizabeth Glaser Pediatric Foundation Malawi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-06-15 00:00:00.000000000 Z
+date: 2026-06-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest