From: Chris Sharp Date: Tue, 7 Feb 2017 23:03:50 +0000 (-0500) Subject: Revert "LP#1596595 AOUS lookup batch by org id" X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=6137bb1d7cf34bb5c65dea520e24902b5acdf5a5;p=evergreen%2Fpines.git Revert "LP#1596595 AOUS lookup batch by org id" This reverts commit 483d641d4292f73e0473dd9be9bfc29b7d294958. --- diff --git a/Open-ILS/src/perlmods/live_t/21-batch-org-settings.t b/Open-ILS/src/perlmods/live_t/21-batch-org-settings.t deleted file mode 100644 index 27d88b5e02..0000000000 --- a/Open-ILS/src/perlmods/live_t/21-batch-org-settings.t +++ /dev/null @@ -1,43 +0,0 @@ -#!perl -use strict; use warnings; -use Test::More tests => 180; # 15 orgs * 12 settings -use OpenILS::Utils::TestUtils; -use OpenILS::Application::AppUtils; -my $U = 'OpenILS::Application::AppUtils'; - -diag("Tests batch org setting retrieval"); - -my $script = OpenILS::Utils::TestUtils->new(); -$script->bootstrap; - -my $org_ids = [1 .. 15]; -# All settings at time of writing. None of these have view perms. -my @settings = qw/ - circ.patron_search.diacritic_insensitive - circ.checkout_auto_renew_age - cat.label.font.weight - cat.spine.line.height - circ.grace.extend - cat.label.font.size - circ.booking_reservation.default_elbow_room - cat.spine.line.width - lib.info_url - circ.hold_go_home_interval - cat.label.font.family - cat.spine.line.margin -/; - -# compare the values returned from the batch-by-org setting to the -# traditional setting value lookup call. -for my $setting (@settings) { - my %batch_settings = - $U->ou_ancestor_setting_batch_by_org_insecure($org_ids, $setting); - - for my $org_id (@$org_ids) { - my $value = $U->ou_ancestor_setting_value($org_id, $setting); - is($value, $batch_settings{$org_id}->{value}, - "Value matches for setting $setting and org $org_id"); - } -} - - diff --git a/Open-ILS/src/sql/Pg/upgrade/YYYY.schema.batch_settings_by_org.sql b/Open-ILS/src/sql/Pg/upgrade/YYYY.schema.batch_settings_by_org.sql deleted file mode 100644 index de24985a7e..0000000000 --- a/Open-ILS/src/sql/Pg/upgrade/YYYY.schema.batch_settings_by_org.sql +++ /dev/null @@ -1,24 +0,0 @@ -BEGIN; - -CREATE OR REPLACE FUNCTION actor.org_unit_ancestor_setting_batch_by_org( - setting_name TEXT, VARIADIC org_ids INT[]) - RETURNS SETOF actor.org_unit_setting AS -$FUNK$ -DECLARE - setting RECORD; - org_id INTEGER; -BEGIN - /* Returns one actor.org_unit_setting row per org unit ID provided. - When no setting exists for a given org unit, the setting row - will contain all empty values. */ - FOREACH org_id IN ARRAY org_ids LOOP - SELECT INTO setting * FROM - actor.org_unit_ancestor_setting(setting_name, org_id); - RETURN NEXT setting; - END LOOP; - RETURN; -END; -$FUNK$ LANGUAGE plpgsql STABLE; - -COMMIT; -