renalware-core 2.0.159 → 2.0.160

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fcf37115d2f693f8be9eac000f5f715889eef97f6d75b124631b1863172f40fb
-  data.tar.gz: cd91c4ca4d9caf7fa8f5d78dd1c604b2d43276b036c03163c362bcadc58eeeb2
+  metadata.gz: e50ba71bf73d49732a3e9be7cb75ef7267527b6af48cd3362794a21316dfbdfd
+  data.tar.gz: ad609dc32dc6d6e472089b5dc30dda67b8cfd17857e9f310d0e54c93c7d1a1ac
 SHA512:
-  metadata.gz: 0174c4a3b63c1af4aa60a195dfdb2b37bcdc179073f288886048601ef86efebd940a889e815dc52a77d48f3c878131e3962df86b17df42b50e1bc50ee0439c36
-  data.tar.gz: 9d08256e91e438cf5777478a1d2a3142f4265c6c5eac2a7daaf6d7e9dcb4c0743289d2b0f102403aeb6477846aab82a72b26f90564bea25f19c876cf96ecf91c
+  metadata.gz: e9db000533a7a9c8ef8b7042f40f25fdece027d809c94edb09064225c0a89e101b992cde7054e2902853f044e8c987f65c8e588b1957c61d7569b738ad170382
+  data.tar.gz: bd0ad2bf21e7b0369e8f6ff760e5669368fb3cf852387c6ea6c8d364c4c5d33b20e3ad0ea6094fee14bdedce44b6b2f58d2a59327cca6066542122b73de6ba2d

data/app/components/renalware/article_component.rb

@@ -6,7 +6,7 @@ module Renalware
   # build more and more useful components. Other components can use this component
   # in their markup.
   class ArticleComponent < ApplicationComponent
-    pattr_initialize [:title]
+    rattr_initialize [:title]
     with_content_areas :title_link, :actions
   end
 end

data/app/components/renalware/letters/letters_in_progress_component.rb

@@ -3,11 +3,7 @@
 module Renalware
   module Letters
     class LettersInProgressComponent < ApplicationComponent
-      attr_reader :current_user
-
-      def initialize(current_user:)
-        @current_user = current_user
-      end
+      pattr_initialize [:current_user!]
 
       # Note we want oldest letters ordered first here - elsewhere letters are newest first
       def letters_in_progress

data/app/components/renalware/letters/unread_electronic_ccs_component.rb

@@ -4,11 +4,8 @@ module Renalware
   module Letters
     class UnreadElectronicCCsComponent < ApplicationComponent
       include Pundit::Helper
-      attr_reader :current_user
 
-      def initialize(current_user:)
-        @current_user = current_user
-      end
+      pattr_initialize [:current_user!]
 
       def unread_electronic_ccs
         @unread_electronic_ccs ||= begin

data/app/components/renalware/patients/bookmarks_component.rb

@@ -3,11 +3,7 @@
 module Renalware
   module Patients
     class BookmarksComponent < ApplicationComponent
-      attr_reader :current_user
-
-      def initialize(current_user:)
-        @current_user = current_user
-      end
+      pattr_initialize [:current_user!]
 
       def bookmarks
         @bookmarks ||= begin

data/app/components/renalware/patients/messages_component.rb

@@ -8,13 +8,7 @@ module Renalware
       include Pagy::Frontend
       TITLE = "Messages"
 
-      attr_reader :patient, :current_user
-
-      def initialize(patient:, current_user:)
-        @patient = patient
-        @current_user = current_user
-        super
-      end
+      rattr_initialize [:patient!, :current_user!]
 
       def pagination
         load_messages unless @pagination

data/app/components/renalware/pd/adequacy_results_component.rb

@@ -9,7 +9,7 @@ module Renalware
       include Pagy::Backend
       include Pagy::Frontend
       include ToggleHelper
-      pattr_initialize [:patient!, :current_user!]
+      rattr_initialize [:patient!, :current_user!]
       attr_reader :pagination
 
       TITLE = "Adequacy Results"

data/app/components/renalware/surveys/eq5d_component.rb

@@ -5,11 +5,7 @@ require_dependency "renalware/surveys"
 module Renalware
   module Surveys
     class EQ5DComponent < ApplicationComponent
-      attr_reader :patient
-
-      def initialize(patient:)
-        @patient = patient
-      end
+      rattr_initialize [:patient!]
 
       def rows
         @rows ||= EQ5DPivotedResponse.where(patient_id: patient.id)

data/app/components/renalware/surveys/pos_s_component.rb

@@ -7,11 +7,7 @@ module Renalware
     class POSSComponent < ApplicationComponent
       include ToggleHelper
 
-      attr_reader :patient
-
-      def initialize(patient:)
-        @patient = patient
-      end
+      rattr_initialize [:patient!]
 
       # Backed by a SQL view
       def rows

data/app/components/renalware/system/admin_menu_component.rb

@@ -5,11 +5,8 @@ module Renalware
     # Experimental ActionView component
     class AdminMenuComponent < ApplicationComponent
       include Renalware::UsersHelper
-      attr_reader :current_user
 
-      def initialize(current_user:)
-        @current_user = current_user
-      end
+      rattr_initialize [:current_user!]
 
       def admin_menu_item(*args)
         return unless

data/app/components/renalware/system/users_awaiting_approval_component.rb

@@ -3,11 +3,7 @@
 module Renalware
   module System
     class UsersAwaitingApprovalComponent < ApplicationComponent
-      attr_reader :current_user
-
-      def initialize(current_user:)
-        @current_user = current_user
-      end
+      rattr_initialize [:current_user!]
 
       def users_needing_approval_count
         @users_needing_approval_count ||= User.unapproved.count

data/app/controllers/renalware/api/token_authenticated_api_controller.rb

@@ -7,7 +7,7 @@ require "devise"
 
 module Renalware
   module API
-    class TokenAuthenticatedApiController < ApplicationController
+    class TokenAuthenticatedAPIController < ApplicationController
       before_action :authenticate_user_from_token!
       before_action :authenticate_user! # fallback
 

data/app/controllers/renalware/api/v1/medications/prescriptions_controller.rb

@@ -7,7 +7,7 @@ module Renalware
   module API
     module V1
       module Medications
-        class PrescriptionsController < TokenAuthenticatedApiController
+        class PrescriptionsController < TokenAuthenticatedAPIController
           def index
             render locals: {
               patient: patient,

data/app/controllers/renalware/api/v1/patients/patients_controller.rb

@@ -7,7 +7,7 @@ module Renalware
   module API
     module V1
       module Patients
-        class PatientsController < TokenAuthenticatedApiController
+        class PatientsController < TokenAuthenticatedAPIController
           include Concerns::Pageable
 
           def show

data/app/controllers/renalware/events/events_controller.rb

@@ -2,7 +2,6 @@
 
 # rubocop:disable Metrics/ClassLength
 require_dependency "renalware/events"
-require "attr_extras"
 
 module Renalware
   module Events

data/app/jobs/renalware/hd/generate_monthly_statistics.rb

@@ -16,7 +16,7 @@ module Renalware
       def initialize(month: nil, year: nil)
         validate_args(month: month, year: year)
 
-        @period = Renalware::MonthPeriod.new(
+        @period = MonthPeriod.new(
           month: (month || default_month).to_i,
           year: (year || default_year).to_i
         )

data/app/jobs/renalware/hd/generate_monthly_statistics_for_patient_job.rb

@@ -10,7 +10,7 @@ module Renalware
 
       # :reek:UtilityFunction
       def perform(patient:, month:, year:)
-        period = Renalware::MonthPeriod.new(month: month, year: year)
+        period = MonthPeriod.new(month: month, year: year)
         GenerateMonthlyStatisticsForPatient.new(patient: patient, period: period).call
       end
     end

data/app/jobs/renalware/letters/calculate_page_count_job.rb

@@ -2,7 +2,6 @@
 
 require_dependency "renalware/letters"
 require "pdf/reader"
-require "attr_extras"
 
 module Renalware
   module Letters

data/app/jobs/renalware/letters/mailshots/create_mailshot_letters_job.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/letters"
-require "attr_extras"
 
 module Renalware
   module Letters

data/app/jobs/renalware/letters/printing/batch_print_job.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/letters"
-require "attr_extras"
 
 module Renalware
   module Letters

data/app/jobs/renalware/letters/save_rtf_letter_to_file_job.rb

@@ -7,7 +7,7 @@ module Renalware
   module Letters
     # Used for example in a host app like renalware-blt to generate an RTF letter for saving to
     # the electronic public register, aka EPR/CRS.
-    class SaveRtfLetterToFileJob < ApplicationJob
+    class SaveRTFLetterToFileJob < ApplicationJob
       queue_as :rtf_generation
       queue_with_priority 1
 

data/app/mailers/renalware/letters/{delivery/errors.rb → delivery.rb}

@@ -5,7 +5,8 @@ require_dependency "renalware/letters"
 module Renalware
   module Letters
     module Delivery
-      class LetterIsNotApprovedOrCompletedError < StandardError; end
+      # Test fail with the error class definition here
+      # class LetterIsNotApprovedOrCompletedError < StandardError; end
     end
   end
 end

data/app/mailers/renalware/letters/delivery/practice_mailer.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/letters"
-require "attr_extras"
-require_relative "./errors"
 
 module Renalware
   module Letters
     module Delivery
+      class LetterIsNotApprovedOrCompletedError < StandardError; end
+
       # Note PracticeMailer not the best name as we use this mailer to send a letter to any
       # recipient (practices, hospital secretaries etc)
       class PracticeMailer < ApplicationMailer

data/app/models/concerns/renalware/deviseable.rb

@@ -22,10 +22,10 @@ module Renalware
       end
 
       def inactive_message
-        if !approved?
-          :not_approved
-        else
+        if approved?
           super
+        else
+          :not_approved
         end
       end
     end

data/app/models/renalware/admissions/admission_query.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/admissions"
-require "attr_extras"
 
 module Renalware
   module Admissions

data/app/models/renalware/clinical/create_allergy.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/clinical"
-require_dependency "renalware/success"
-require_dependency "renalware/failure"
+require "success"
+require "failure"
 
 module Renalware
   module Clinical
@@ -17,9 +17,9 @@ module Renalware
         yield allergy if block_given?
         if allergy.valid?
           save_allergy(allergy)
-          ::Renalware::Success.new(allergy)
+          ::Success.new(allergy)
         else
-          ::Renalware::Failure.new(allergy)
+          ::Failure.new(allergy)
         end
       end
 

data/app/models/renalware/clinics/create_clinic_visit.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/clinics"
-require_dependency "renalware/success"
-require_dependency "renalware/failure"
+require "success"
+require "failure"
 
 module Renalware
   module Clinics
@@ -22,9 +22,9 @@ module Renalware
           visit = patient.clinic_visits.new(params)
           objects = OpenStruct.new(clinic_visit: visit, appointment: appointment)
           if visit.save && update_appointment_with(visit.id)
-            return ::Renalware::Success.new(objects)
+            return ::Success.new(objects)
           else
-            return ::Renalware::Failure.new(objects)
+            return ::Failure.new(objects)
           end
         end
       end

data/app/models/renalware/clinics/current_observations.rb

@@ -1,11 +1,10 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/letters"
-require "attr_extras"
 
 module Renalware
   module Clinics
-    # TODO: Move this to a view?
+    # TODO: Unused?
     class CurrentObservations
       NULL_DATE = nil
       pattr_initialize :patient
@@ -17,7 +16,7 @@ module Renalware
         @weight ||= begin
           result = ClinicVisit
                     .most_recent_for_patient(patient)
-                    .where("weight is not null")
+                    .where.not(weight: nil)
                     .pluck(:date, :weight).first || []
 
           Observation.new(result.first, result.last)
@@ -29,7 +28,7 @@ module Renalware
         @height ||= begin
           result = ClinicVisit
                     .most_recent_for_patient(patient)
-                    .where("height is not null")
+                    .where.not(height: nil)
                     .pluck(:date, :height).first || []
 
           Observation.new(result.first, result.last)

data/app/models/renalware/events/create_event.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/events"
-require "attr_extras"
 
 module Renalware
   module Events

data/app/models/renalware/feeds/files/exceptions.rb

@@ -7,6 +7,7 @@ module Renalware
     module Files
       class Exceptions
         class FileNotFoundError < StandardError; end
+
         class UnexpectedCSVFormatError < StandardError; end
       end
     end

data/app/models/renalware/feeds/files/practice_memberships/import_csv.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/feeds"
-require "attr_extras"
 
 module Renalware
   module Feeds

data/app/models/renalware/feeds/files/primary_care_physicians/import_csv.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/feeds"
-require "attr_extras"
 
 module Renalware
   module Feeds

data/app/models/renalware/feeds/message_processor.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_dependency "renalware/feeds"
-require "attr_extras"
 
 module Renalware
   module Feeds

data/app/models/renalware/hd/prescription_administration.rb

@@ -9,10 +9,12 @@ module Renalware
       acts_as_paranoid
 
       # Set to true by the parent hd_session if we are not signing off at this stage
-      attr_accessor :skip_witness_validation
-      attr_accessor :skip_administrator_validation
-      attr_accessor :administered_by_password
-      attr_accessor :witnessed_by_password
+      attr_accessor(
+        :skip_witness_validation,
+        :skip_administrator_validation,
+        :administered_by_password,
+        :witnessed_by_password
+      )
 
       belongs_to :hd_session, class_name: "HD::Session", touch: true
       belongs_to :prescription, class_name: "Medications::Prescription"