Cleaned up so it shouldn't ruin the database creation (hopefully).

Signed-off-by: Joseph Lewis <joehms22@gmail.com>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
Signed-off-by: Jason Etheridge <jason@esilibrary.com>

diff --git a/Open-ILS/src/sql/Pg/002.schema.config.sql b/Open-ILS/src/sql/Pg/002.schema.config.sql
index 476cc2f..2e032a2 100644 (file)
--- a/Open-ILS/src/sql/Pg/002.schema.config.sql
+++ b/Open-ILS/src/sql/Pg/002.schema.config.sql
@@ -924,6 +924,14 @@ CREATE TABLE config.org_unit_setting_type_log (
     field_name      TEXT      REFERENCES config.org_unit_setting_type (name)
 );
 
+COMMENT ON TABLE config.org_unit_setting_log IS $$
+Org Unit setting Logs
+
+This table contains the most recent changes to each setting 
+in actor.org_unit_setting, allowing for mistakes to be undone.
+This is NOT meant to be an auditor, but rather an undo/redo.
+$$;
+
 CREATE OR REPLACE FUNCTION limit_oustl() RETURNS TRIGGER AS $oustl_limit$
     BEGIN
         -- Only keeps the most recent five settings changes.
@@ -942,24 +950,5 @@ $oustl_limit$ LANGUAGE plpgsql;
 CREATE TRIGGER limit_logs_oust
     BEFORE INSERT OR UPDATE ON config.org_unit_setting_type_log
     FOR EACH ROW EXECUTE PROCEDURE limit_oustl();
-    
-
--- Log each change in oust to oustl, so admins can see what they messed up if someting stops working.
-CREATE OR REPLACE FUNCTION ous_change_log() RETURNS TRIGGER AS $ous_change_log$
-    DECLARE
-    original TEXT;
-    BEGIN
-        -- Check for which setting is being updated, and log it.
-        SELECT INTO original value FROM actor.org_unit_setting WHERE name = NEW.name AND org_unit = NEW.org_unit;
-                
-        INSERT INTO config.org_unit_setting_type_log (org,original_value,new_value,field_name) VALUES (NEW.org_unit, original, NEW.value, NEW.name);
-        
-        RETURN NEW;
-    END;
-$ous_change_log$ LANGUAGE plpgsql;    
-
-CREATE TRIGGER log_ous_change
-    BEFORE INSERT OR UPDATE ON actor.org_unit_setting
-    FOR EACH ROW EXECUTE PROCEDURE ous_change_log();
 
 COMMIT;

diff --git a/Open-ILS/src/sql/Pg/005.schema.actors.sql b/Open-ILS/src/sql/Pg/005.schema.actors.sql
index 4208cd0..587b785 100644 (file)
--- a/Open-ILS/src/sql/Pg/005.schema.actors.sql
+++ b/Open-ILS/src/sql/Pg/005.schema.actors.sql
@@ -468,6 +468,24 @@ $$;
 
 CREATE INDEX actor_org_unit_setting_usr_idx ON actor.org_unit_setting (org_unit);
 
+-- Log each change in oust to oustl, so admins can see what they messed up if someting stops working.
+CREATE OR REPLACE FUNCTION ous_change_log() RETURNS TRIGGER AS $ous_change_log$
+    DECLARE
+    original TEXT;
+    BEGIN
+        -- Check for which setting is being updated, and log it.
+        SELECT INTO original value FROM actor.org_unit_setting WHERE name = NEW.name AND org_unit = NEW.org_unit;
+                
+        INSERT INTO config.org_unit_setting_type_log (org,original_value,new_value,field_name) VALUES (NEW.org_unit, original, NEW.value, NEW.name);
+        
+        RETURN NEW;
+    END;
+$ous_change_log$ LANGUAGE plpgsql;    
+
+CREATE TRIGGER log_ous_change
+    BEFORE INSERT OR UPDATE ON actor.org_unit_setting
+    FOR EACH ROW EXECUTE PROCEDURE ous_change_log();
+
 
 CREATE TABLE actor.usr_address (
        id                      SERIAL  PRIMARY KEY,