From 549c50478af823b2d5fd408f522eff8547e2f9a3 Mon Sep 17 00:00:00 2001 From: Jason Boyer Date: Fri, 11 Sep 2020 12:43:20 -0400 Subject: [PATCH] LP1849212: Correct upgrade script Re-align permission ids and correct syntax errors in the course reserves upgrade script. I'm not convinced the boolean_facet metabib class should exist simply to add "course reserves, Y/N?" and in any case can not function as is because adding metabib field classes requires additional field_entry tables which are missing from the seed data and upgrade script. Signed-off-by: Michele Morgan Signed-off-by: Galen Charlton --- Open-ILS/src/sql/Pg/950.data.seed-values.sql | 2 +- Open-ILS/src/sql/Pg/upgrade/XXXX.schema.course-materials-module.sql | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Open-ILS/src/sql/Pg/950.data.seed-values.sql b/Open-ILS/src/sql/Pg/950.data.seed-values.sql index 214aacce25..626f1ce651 100644 --- a/Open-ILS/src/sql/Pg/950.data.seed-values.sql +++ b/Open-ILS/src/sql/Pg/950.data.seed-values.sql @@ -1943,7 +1943,7 @@ INSERT INTO permission.perm_list ( id, code, description ) VALUES 'View booking resources', 'ppl', 'description')), ( 623, 'UPDATE_ORG_UNIT_SETTING.opac.matomo', oils_i18n_gettext(623, 'Allows a user to configure Matomo Analytics org unit settings', 'ppl', 'description')), - ( 624, 'MANAGE_RESERVES', oils_i18n_gettext(623, + ( 624, 'MANAGE_RESERVES', oils_i18n_gettext(624, 'Allows user to manage Courses, Course Materials, and associate Users with Courses.', 'ppl', 'description')) ; diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.course-materials-module.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.course-materials-module.sql index 00a78f5e44..8699d37519 100644 --- a/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.course-materials-module.sql +++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.course-materials-module.sql @@ -8,7 +8,7 @@ CREATE TABLE asset.course_module_course ( course_number TEXT NOT NULL, section_number TEXT, owning_lib INT REFERENCES actor.org_unit (id), - is_archived BOOLEAN NOT NULL DEFAULT false; + is_archived BOOLEAN NOT NULL DEFAULT false ); CREATE TABLE asset.course_module_role ( @@ -86,7 +86,7 @@ VALUES ( ), oils_i18n_gettext( 'circ.course_materials_opt_in', - 'If enabled, the Org Unit will utilize Course Material functionality.' + 'If enabled, the Org Unit will utilize Course Material functionality.', 'coust', 'description' ), null @@ -101,7 +101,7 @@ VALUES ( ), oils_i18n_gettext( 'circ.course_materials_browse_by_instructor', - 'If enabled, the Org Unit will allow OPAC users to browse Courses by instructor name.' + 'If enabled, the Org Unit will allow OPAC users to browse Courses by instructor name.', 'coust', 'description' ), null -- 2.11.0