renalware-core 2.0.149 → 2.0.151

data/app/models/renalware/pd/unified_pet_adequacy_form.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require_dependency "renalware/pd"
+
+# Captures data used for PET (Peritoneal Equilibration Test) and Adequacy.
+module Renalware
+  module PD
+    class UnifiedPETAdequacyForm
+      include ActiveModel::Model
+      include Virtus::Model
+
+      attribute :pet, PETResult
+      attribute :adequacy, AdequacyResult
+      attribute :patient, Patient
+
+      attribute :pet_missing, Boolean
+      attribute :adequacy_missing, Boolean
+
+      def valid?
+        objects_to_save.each(&:validate).all?(&:valid?)
+      end
+
+      def save_by!(user)
+        PETResult.transaction do
+          objects_to_save.each { |obj| obj.save_by!(user) }
+        end
+      end
+
+      def objects_to_save
+        @objects_to_save ||= begin
+          [].tap do |arr|
+            arr << pet unless pet_missing
+            arr << adequacy unless adequacy_missing
+          end
+        end
+      end
+
+      # This method lets us just use eg simple_form_for in the view without
+      # having to specify the url option i.e. PatientPDUnifiedPetAdequacy here will
+      # resolve to 'patient_pd_pet_adequacy_results_path'
+      def self.model_name
+        ActiveModel::Name.new(self, nil, "PatientPDUnifiedPetAdequacy")
+      end
+    end
+  end
+end

data/app/models/renalware/renal/registry/preflight_checks/missing_esrf_query.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require_dependency "renalware/renal"
+
+module Renalware
+  module Renal
+    module Registry
+      module PreflightChecks
+        # Finds patients with a current modality of HD, PD or Transplant, who have no ESRF date
+        class MissingESRFQuery
+          include ModalityScopes
+          MODALITY_NAMES = %w(HD PD Transplant).freeze
+
+          attr_reader :relation, :query_params
+
+          def initialize(relation: nil, query_params: {})
+            @relation = relation || default_relation
+            @query_params = query_params
+            @query_params[:s] = "modality_descriptions_name ASC" if @query_params[:s].blank?
+          end
+
+          def default_relation
+            Renalware::Renal::Patient.left_outer_joins(:profile)
+          end
+
+          def call
+            search
+              .result
+              .extending(ModalityScopes)
+              .preload(current_modality: [:description])
+              .merge(HD::Patient.with_profile)
+              .eager_load(:profile)
+              .with_current_modality_matching(MODALITY_NAMES)
+              .where("renal_profiles.esrf_on IS NULL")
+          end
+
+          def search
+            @search ||= relation.ransack(query_params)
+          end
+
+          def self.missing_data_for(_patient)
+            [
+              :esrf
+            ]
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/renalware/snippets/duplicate_snippet_title_generator.rb

@@ -8,6 +8,7 @@ module Renalware
       MAX_ATTEMPTS = 100
 
       attr_reader :snippet, :title_suffix
+
       delegate :valid?, to: :snippet
 
       def initialize(snippet, title_suffix: "[CLONED]")

data/app/models/renalware/transplants/recipient_workup_builder.rb

@@ -20,6 +20,7 @@ module Renalware
       private
 
       attr_reader :workup, :default_consenter_name
+
       delegate :document, to: :workup
 
       def assign_default_consenter

data/app/models/renalware/transplants/registration_status.rb

@@ -14,7 +14,7 @@ module Renalware
       scope :reversed, -> { order(started_on: :desc, created_at: :desc) }
 
       validates :description_id, presence: true
-      validates :started_on, timeliness: { type: :date, allow_blank: false }
+      validates :started_on, presence: true, timeliness: { type: :date, allow_blank: false }
       validates :terminated_on, timeliness: { type: :date, allow_blank: true }
 
       validate :constraint_started_on_today_or_before, if: :started_on

data/app/models/renalware/ukrdc/outgoing/rendering/patient_number.rb

@@ -15,7 +15,7 @@ module Renalware
 
           def name_element
             create_node("PatientNumber") do |elem|
-              elem << create_node("Number", number)
+              elem << create_node("Number", number&.delete(" "))
               elem << create_node("Organization", organisation)
               elem << create_node("NumberType", type)
             end

data/app/models/renalware/user.rb

@@ -42,6 +42,8 @@ module Renalware
     scope :hidden, -> { where(hidden: true) }
     scope :picklist, -> { visible.ordered }
 
+    store_accessor :preferences, :experimental_features
+
     # Non-persistent attribute to signify we want to use extended validation.
     # We need to refactor this by ising a form object for updating a user.
     attr_accessor :with_extended_validation
@@ -100,6 +102,14 @@ module Renalware
     #   Pundit.policy(self, record).public_send(method.to_sym)
     # end
 
+    # We can enable experiment features for particular users using the bitmask user#feature_flags
+    # property and bitwise operators.
+    # For example given the the feature flag FANCY_GRAPHS = 2, we if they user should see these with
+    #   FANCY_GRAPHS & feature_flags == FANCY_GRAPHS
+    def wants_feature?(flag)
+      (flag & feature_flags) == flag
+    end
+
     private
 
     def build_authentication_token

data/app/pdfs/renalware/events/event_pdf.rb

@@ -12,6 +12,7 @@ module Renalware
     class EventPdf
       include Prawn::View
       attr_reader :event
+
       delegate :patient, :event_type, to: :event
 
       def initialize(event)

data/app/policies/renalware/events/advanced_care_plan_policy.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+require_dependency "renalware/messaging"
+
+module Renalware
+  module Events
+    class AdvancedCarePlanPolicy < BasePolicy
+    end
+  end
+end

data/app/policies/renalware/events/clinical_frailty_score_policy.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+require_dependency "renalware/messaging"
+
+module Renalware
+  module Events
+    class ClinicalFrailtyScorePolicy < BasePolicy
+    end
+  end
+end

data/app/policies/renalware/pd/adequacy_result_policy.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module Renalware
+  module PD
+    class AdequacyResultPolicy < BasePolicy
+      def destroy?
+        user_is_admin? || user_is_super_admin?
+      end
+
+      def edit?
+        destroy?
+      end
+    end
+  end
+end

data/app/policies/renalware/pd/pet_result_policy.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module Renalware
+  module PD
+    class PETResultPolicy < BasePolicy
+      def destroy?
+        user_is_admin? || user_is_super_admin?
+      end
+
+      def edit?
+        destroy?
+      end
+    end
+  end
+end

data/app/policies/renalware/renal/registry_preflight_check_policy.rb

@@ -12,6 +12,10 @@ module Renalware
       def patients?
         index?
       end
+
+      def missing_esrf?
+        index?
+      end
     end
   end
 end

data/app/presenters/renalware/clinical/profile_presenter.rb

@@ -6,6 +6,7 @@ module Renalware
   module Clinical
     class ProfilePresenter
       attr_reader :patient, :params
+
       delegate :allergies, :document, to: :patient
 
       def initialize(patient:, params:)

data/app/presenters/renalware/hd/dashboard_presenter.rb

@@ -6,6 +6,7 @@ module Renalware
   module HD
     class DashboardPresenter
       attr_accessor :patient
+
       delegate :has_ever_been_on_hd?, to: :patient
 
       def initialize(patient, view_context, current_user)

data/app/presenters/renalware/hd/profile_presenter.rb

@@ -4,6 +4,7 @@ module Renalware
   module HD
     class ProfilePresenter < DumbDelegator
       attr_reader :preference_set
+
       delegate :dialysis,
                :anticoagulant,
                :transport,

data/app/presenters/renalware/hd/scheduling/diary_presenter.rb

@@ -8,6 +8,7 @@ module Renalware
     module Scheduling
       class DiaryPresenter
         attr_reader :user, :weekly_diary, :master_diary, :null_diary
+
         delegate :id, :hospital_unit_id, :to_s, :week, :created_at, to: :weekly_diary
 
         # https://github.com/avdi/naught

data/app/presenters/renalware/hd/session_presenter.rb

@@ -10,6 +10,7 @@ module Renalware
       }.freeze
 
       attr_reader :preference_set
+
       delegate :info,
                :observations_before,
                :observations_after,

data/app/presenters/renalware/hd/unmet_preferences_presenter.rb

@@ -28,7 +28,7 @@ module Renalware
         value = yield(value) if block_given?
         return value if preference_satisfied?(attribute)
 
-        content_tag(:b, value)
+        tag.b(value)
       end
 
       def preference_satisfied?(attribute)

data/app/presenters/renalware/mdm_patients_presenter.rb

@@ -5,6 +5,7 @@ module Renalware
   class MDMPatientsPresenter
     include PresenterHelper
     attr_reader :patients, :view_proc, :page_title, :q
+
     def initialize(patients:,
                    view_proc:,
                    page_title:,

data/app/presenters/renalware/pd/dashboard_presenter.rb

@@ -29,13 +29,21 @@ module Renalware
 
       def peritonitis_episodes
         @peritonitis_episodes ||= begin
-          episodes = PeritonitisEpisode.for_patient(patient).ordered.includes(:episode_types)
+          episodes = PeritonitisEpisode
+            .for_patient(patient)
+            .ordered
+            .includes(:episode_types, infection_organisms: :organism_code)
           present(episodes, PeritonitisEpisodePresenter)
         end
       end
 
       def exit_site_infections
-        @exit_site_infections ||= ExitSiteInfection.for_patient(patient).ordered
+        @exit_site_infections ||= begin
+          ExitSiteInfection
+            .for_patient(patient)
+            .includes(infection_organisms: :organism_code)
+            .ordered
+        end
       end
 
       def pet_adequacies

data/app/presenters/renalware/ukrdc/patient_presenter.rb

@@ -9,6 +9,7 @@ module Renalware
       UKRDC_MAX_PHONE_LEN = 80
 
       attr_reader :changes_since, :changes_up_until
+
       delegate :prd_description, to: :profile, allow_nil: true
       delegate :code, :term, to: :prd_description, allow_nil: true, prefix: true
 

data/app/presenters/renalware/user_session_presenter.rb

@@ -31,7 +31,7 @@ module Renalware
             "login-path": urls.new_user_session_path,
             "check-alive-path": urls.check_session_expired_path,
             "keep-alive-path": urls.keep_session_alive_path,
-            debug: Rails.env.development?.to_s,
+            debug: Rails.env.development?.to_s, # eg "true" or "false"
             "register-user-activity-after":
               Renalware.config.session_register_user_user_activity_after.to_i,
             timeout: ::Devise.timeout_in

data/app/validators/renalware/patients/nhs_number_validator.rb

@@ -0,0 +1,88 @@
+# frozen_string_literal: true
+
+module Renalware
+  module Patients
+    # Validates an NHS number which should be 10 characters long and confirm to the modulo 11
+    # algorithm where the 10th character is a check digit.
+    class NHSNumberValidator < ActiveModel::EachValidator
+      def validate_each(record, attribute, value)
+        return if value.blank?
+
+        Validator.new(record, attribute, value).validate
+      end
+
+      class Validator
+        pattr_initialize :record, :attribute, :number
+        LENGTH = 10
+
+        def validate
+          strip_spaces_from_nhs_number
+          validate_length
+          validate_numeric
+          validate_using_modulus_11
+        end
+
+        private
+
+        def strip_spaces_from_nhs_number
+          @record[attribute] = @number = @number.delete(" ")
+        end
+
+        def validate_length
+          ActiveModel::Validations::LengthValidator.new(
+            is: LENGTH,
+            attributes: [attribute]
+          ).validate(record)
+        end
+
+        def validate_numeric
+          return if record.errors[attribute].any?
+
+          unless number =~ /\d{10}/
+            record.errors.add(attribute, :nhs_number_not_numeric)
+          end
+        end
+
+        # See https://www.datadictionary.nhs.uk/data_dictionary/attributes/n/nhs/nhs_number_de.asp
+        # rubocop:disable Metrics/AbcSize
+        def validate_using_modulus_11
+          return if record.errors[attribute].any?
+
+          # Step 1 Multiply each of the first nine digits by a weighting factor
+          # Digit Position  Factor
+          # 1                10
+          # 2                9
+          # 3                8
+          # 4                7
+          # 5                6
+          # 6                5
+          # 7                4
+          # 8                3
+          # 9                2
+          weighting_factors = [10, 9, 8, 7, 6, 5, 4, 3, 2]
+          first_9_digits = number.chars.map(&:to_i).take(9)
+          first_9_digits_weighted = first_9_digits.each_with_index.map do |num, idx|
+            num * weighting_factors[idx]
+          end
+
+          # Step 2 Add the results of each multiplication together.
+          # Step 3 Divide the total by 11 and establish the remainder.
+          # Step 4 Subtract the remainder from 11 to give the check digit.
+          checkdigit = 11 - (first_9_digits_weighted.sum % 11)
+
+          # If the result is 11 then a check digit of 0 is used.
+          checkdigit = 0 if checkdigit == 11
+
+          # If the result is 10 then the NHS NUMBER is invalid and not used.
+          # Step 5 Check the remainder matches the check digit. Otherwise the NHS NUMBER is invalid.
+          if checkdigit == 10 || number[9].to_i != checkdigit
+            # record.errors[attribute] << "Invalid NHS number (checkdigit mismatch)"
+            record.errors.add(attribute, :nhs_number_invalid_checkdigit)
+          end
+        end
+        # rubocop:enable Metrics/AbcSize
+      end
+      private_constant :Validator
+    end
+  end
+end

data/app/values/renalware/address.rb

@@ -11,6 +11,7 @@ module Renalware
     # Set to true to avoid address validation - useful when archiving a letter with a migrated
     # address that might not have a postcode for instance
     attr_accessor :skip_validation
+
     validates_with AddressValidator, unless: :skip_validation
 
     def self.reject_if_blank

data/app/views/renalware/accesses/assessments/show.html.slim

@@ -8,7 +8,7 @@
   - return unless assessment
 
   .document-view
-    .grid
+    .mgrid
       .row
         - attributes_list = assessment.document.results.class.attributes_list
         = render "renalware/shared/attributes_group",