X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;ds=sidebyside;f=SL%2FDBUpgrade2%2FBase.pm;h=5d0d26c9b572b66908b929b3ddfb9d0081e23099;hb=ac1a9e8e357375b2426041eefee1f555fc5139b5;hp=2c1e111140cc63d25d15294c3f5f05e92aedba74;hpb=61cdba5d566357f3beabe0e7f3f0cb2d7bdccd73;p=kivitendo-erp.git
diff --git a/SL/DBUpgrade2/Base.pm b/SL/DBUpgrade2/Base.pm
index 2c1e11114..5d0d26c9b 100644
--- a/SL/DBUpgrade2/Base.pm
+++ b/SL/DBUpgrade2/Base.pm
@@ -5,11 +5,14 @@ use strict;
use parent qw(Rose::Object);
use Carp;
+use Encode;
use English qw(-no_match_vars);
use File::Basename ();
use File::Copy ();
use File::Path ();
use List::MoreUtils qw(uniq);
+use SL::DBUtils qw(selectfirst_hashref_query);
+use version;
use Rose::Object::MakeMethods::Generic (
scalar => [ qw(dbh myconfig) ],
@@ -27,9 +30,10 @@ sub execute_script {
die $EVAL_ERROR;
}
+ my $auth = $file_name =~ m{/Pg-upgrade2-auth/} ? 'Auth::' : '';
my $package = delete $params{tag};
$package =~ s/[^a-zA-Z0-9_]+/_/g;
- $package = "SL::DBUpgrade2::${package}";
+ $package = "SL::DBUpgrade2::${auth}${package}";
$package->new(%params)->run;
}
@@ -37,18 +41,34 @@ sub execute_script {
sub db_error {
my ($self, $msg) = @_;
- die $::locale->text("Database update error:") . "
$msg
" . $DBI::errstr;
+ die $::locale->text("Database update error:") . "
$msg
" . $self->db_errstr('DBI');
}
sub db_query {
my ($self, $query, %params) = @_;
- return if $self->dbh->do($query, undef, @{ $params{bind} || [] });
+ my $dbh = $params{dbh} || $self->dbh;
+
+ return if $dbh->do($query, undef, @{ $params{bind} || [] });
$self->db_error($query) unless $params{may_fail};
- $self->dbh->rollback;
- $self->dbh->begin_work;
+ $dbh->rollback;
+ $dbh->begin_work;
+}
+
+sub db_errstr {
+ my ($self, $handle) = @_;
+
+ # DBD::Pg before 2.16.1 doesn't set the UTF-8 flag for error
+ # messages even if the connection has UTF-8 enabled. Therefore we
+ # have to convert it to Perl's internal encoding ourselves. See
+ # https://rt.cpan.org/Public/Bug/Display.html?id=53854
+
+ my $error = $handle ? $handle->errstr : $self->dbh->errstr;
+
+ return $error if version->new("$DBD::Pg::VERSION")->numify >= version->new("2.16.1")->numify;
+ return Encode::decode('utf-8', $error);
}
sub check_coa {
@@ -79,35 +99,71 @@ sub add_print_templates {
croak "File '${src_dir}/$_' does not exist" unless -f "${src_dir}/$_";
}
- my %users = $::auth->read_all_users;
- my @template_dirs = uniq map { $_ = $_->{templates}; s:/+$::; $_ } values %users;
-
- $::lxdebug->message(LXDebug::DEBUG1(), "add_print_templates: template_dirs " . join(' ', @template_dirs));
+ # can't use Rose or InstanceConf here because defaults might not be fully upgraded yet.
+ my $defaults = selectfirst_hashref_query($::form, $::form->get_standard_dbh, "SELECT * FROM defaults");
+ return 1 unless my $template_dir = $defaults->{template};
+ $::lxdebug->message(LXDebug::DEBUG1(), "add_print_templates: template_dir $template_dir");
foreach my $src_file (@files) {
- foreach my $template_dir (@template_dirs) {
- my $dest_file = $template_dir . '/' . $src_file;
+ my $dest_file = $template_dir . '/' . $src_file;
- if (-f $dest_file) {
- $::lxdebug->message(LXDebug::DEBUG1(), "add_print_templates: dest_file exists, skipping: ${dest_file}");
- next;
- }
+ if (-f $dest_file) {
+ $::lxdebug->message(LXDebug::DEBUG1(), "add_print_templates: dest_file exists, skipping: ${dest_file}");
+ next;
+ }
- my $dest_dir = File::Basename::dirname($dest_file);
+ my $dest_dir = File::Basename::dirname($dest_file);
- if ($dest_dir && !-d $dest_dir) {
- File::Path::make_path($dest_dir) or die "Cannot create directory '${dest_dir}': $!";
- }
+ if ($dest_dir && !-d $dest_dir) {
+ File::Path::make_path($dest_dir) or die "Cannot create directory '${dest_dir}': $!";
+ }
- File::Copy::copy($src_dir . '/' . $src_file, $dest_file) or die "Cannot copy '${src_dir}/${src_file}' to '${dest_file}': $!";
+ File::Copy::copy($src_dir . '/' . $src_file, $dest_file) or die "Cannot copy '${src_dir}/${src_file}' to '${dest_file}': $!";
- $::lxdebug->message(LXDebug::DEBUG1(), "add_print_templates: copied '${src_dir}/${src_file}' to '${dest_file}'");
- }
+ $::lxdebug->message(LXDebug::DEBUG1(), "add_print_templates: copied '${src_dir}/${src_file}' to '${dest_file}'");
}
return 1;
}
+sub drop_constraints {
+ my ($self, %params) = @_;
+
+ croak "Missing parameter 'table'" unless $params{table};
+ $params{type} ||= 'FOREIGN KEY';
+ $params{schema} ||= 'public';
+
+ my $constraints = $self->dbh->selectall_arrayref(<
}g;
+ $new_content =~ s{\n}{
}g;
+ $new_content = "
${new_content}
" if $new_content; + + $sth->execute($new_content, $row->{id}) if $new_content ne $row->{$column}; + } + + $sth->finish; +} + 1; __END__ @@ -221,6 +277,48 @@ current transaction will be rolled back, a new one will be started. An optional array reference containing bind parameter for the query. +=item C