From 2a9651328ee18e9fccf369db093ccda637d7df7b Mon Sep 17 00:00:00 2001
From: Kirk Wang <kirk.wang@scientist.com>
Date: Wed, 27 Sep 2023 06:54:36 -0700
Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=A7=20Override=20QA=20gem=20for=20mult?=
 =?UTF-8?q?iple=20paths?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This commit will add overrides to the Questioning Authority gem to allow
for multiple paths to be used.  This is necessary if you want to use
authorities in addition to the ones in Hyku overwise those would not get
loaded.
---
 config/initializers/knapsack_authorities.rb | 70 +++++++++++++++++++++
 1 file changed, 70 insertions(+)
 create mode 100644 config/initializers/knapsack_authorities.rb

diff --git a/config/initializers/knapsack_authorities.rb b/config/initializers/knapsack_authorities.rb
new file mode 100644
index 00000000..d69c2e59
--- /dev/null
+++ b/config/initializers/knapsack_authorities.rb
@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+# The main purpose of these overrides is to handle multiple authorities directories.
+# Questioning Authority gem only allows one directory for local authorities, which by
+# default is looking at the Hyku's `config/authorities directory` instead of Knapsack's
+# `config/authorities` directory.
+#
+# We want have the ability to add HykuKnapsack's authorities directory in addition to
+# Hyku's.  If two are of the same name, then the one in HykuKnapsack should override the
+# one in Hyku.
+
+# Using relative path because when this gets called we are in the submodule
+HykuKnapsack::AUTHORITIES_PATH = File.expand_path('../config/authorities', Rails.root)
+
+module Qa
+  module Authorities
+    # OVERRIDE Questioning Authority v5.10.0 to load knapsack authorities
+    # see: https://github.com/samvera/questioning_authority/blob/14d233a8404a8d805f1213ae9431ad4ff82a9937/lib/qa/authorities/local.rb
+    module LocalDecorator
+      # Overidding to handle to return an array of paths
+      def subauthorities_path(knapsack_authorities_path: HykuKnapsack::AUTHORITIES_PATH)
+        # knapsack_authorities_path should be first to allow for overriding in case of duplicate names
+        authorities_paths = [knapsack_authorities_path, config[:local_path]]
+
+        authorities_paths.map do |path|
+          path if File.directory?(path)
+        end.compact
+      end
+
+      # Overriding to handle an array of paths to return names of all files
+      def names
+        paths = subauthorities_path # Assuming this now returns an array
+        all_names = []
+
+        paths.each do |path|
+          unless Dir.exist? path
+            raise Qa::ConfigDirectoryNotFound,
+                  "There's no directory at #{path}. You must create it in order to use local authorities"
+          end
+
+          all_names += Dir.entries(path).map { |f| File.basename(f, '.yml') if f =~ /yml$/ }.compact
+        end
+
+        all_names.uniq
+      end
+    end
+  end
+end
+
+module Qa
+  module Authorities
+    module Local
+      # OVERRIDE Questioning Authority v5.10.0 to return first file found from an array of paths
+      # see: https://github.com/samvera/questioning_authority/blob/14d233a8404a8d805f1213ae9431ad4ff82a9937/lib/qa/authorities/local/file_based_authority.rb
+      module FileBasedAuthorityDecorator
+        def subauthority_filename
+          subauthorities_paths = Local.subauthorities_path
+
+          subauthorities_paths.each do |path|
+            yaml_file = File.join(path, "#{subauthority}.yml")
+            return yaml_file if File.exist?(yaml_file)
+          end
+        end
+      end
+    end
+  end
+end
+
+Qa::Authorities::Local.singleton_class.send(:prepend, Qa::Authorities::LocalDecorator)
+Qa::Authorities::Local::FileBasedAuthority.prepend(Qa::Authorities::Local::FileBasedAuthorityDecorator)