[Rt-commit] rt branch, 4.2-trunk, updated. rt-4.2.5-44-g989a2ac

[Alex Vandiver]

The branch, 4.2-trunk has been updated
       via  989a2acbadd7a65047f2cc78a4f7afeb890b24d5 (commit)
      from  e3ced30bb24cd35f30926d0f115c1c924f121702 (commit)

Summary of changes:
 etc/acl.Pg | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

- Log -----------------------------------------------------------------
commit 989a2acbadd7a65047f2cc78a4f7afeb890b24d5
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Tue Jun 17 16:22:02 2014 -0400

    RT 4.2 always requires Postgres 8.4 or higher; remove the conditional
    
    Also explain somewhat the rationale behind /^[a-z]/ rather that /_seq$/

diff --git a/etc/acl.Pg b/etc/acl.Pg
index 9cd06ec..a659d8e 100755
--- a/etc/acl.Pg
+++ b/etc/acl.Pg
@@ -69,14 +69,11 @@ sub acl {
          push @acls, "CREATE USER \"$db_user\" WITH PASSWORD '$db_pass' NOCREATEDB NOCREATEUSER;";
     }
 
-    my $sequence_right
-        = ( $dbh->{pg_server_version} >= 80200 )
-        ? "USAGE, SELECT, UPDATE"
-        : "SELECT, UPDATE";
     foreach my $table (@tables) {
         if ( $table =~ /^[a-z]/ && $table ne 'sessions' ) {
-# table like objectcustomfields_id_s
-            push @acls, "GRANT $sequence_right ON $table TO \"$db_user\";"
+            # Sequences; not all end with _seq because
+            # objectcustomfieldvalues_id_s is too long
+            push @acls, "GRANT USAGE, SELECT, UPDATE ON $table TO \"$db_user\";"
         }
         else {
             push @acls, "GRANT SELECT, INSERT, UPDATE, DELETE ON $table TO \"$db_user\";"

-----------------------------------------------------------------------