Buchungsjournal: Filter "Bearbeiter" gefixt

[mfinanz.git] / sql / Pg-upgrade2 / tax_constraints.pl

diff --git a/sql/Pg-upgrade2/tax_constraints.pl b/sql/Pg-upgrade2/tax_constraints.pl
index 7567930a21a639bfb3381e9cfcb521e86b20e434..23d9b84b2c0fe4faa83bdaa15b55f39460fdfaec 100644 (file)
--- a/sql/Pg-upgrade2/tax_constraints.pl
+++ b/sql/Pg-upgrade2/tax_constraints.pl
@@ -1,304 +1,290 @@
 # @tag: tax_constraints
 # @description: Setzt Fremdschlüssel und andere constraints auf die Tabellen tax und taxkeys
 # @tag: tax_constraints
 # @description: Setzt Fremdschlüssel und andere constraints auf die Tabellen tax und taxkeys

-# @depends: release_3_0_0
-# @charset: utf-8
+# @depends: release_3_0_0 charts_without_taxkey
+package SL::DBUpgrade2::tax_constraints;

 
 

-use utf8;

 use strict;
 use strict;

-use SL::Locale;
-
-die("This script cannot be run from the command line.") unless ($main::form);
-
-sub mydberror {
-  my ($msg) = @_;
-  die($dbup_locale->text("Database update error:") . "<br>$msg<br>" . $DBI::errstr);
-}
+use utf8;

 
 

-sub do_query {
-  my ($query, $may_fail) = @_;
+use parent qw(SL::DBUpgrade2::Base);

 
 

-  if (!$dbh->do($query)) {
-    mydberror($query) unless ($may_fail);
-    $dbh->rollback();
-    $dbh->begin_work();
-  }
-}
+sub run {
+  my ($self) = @_;

 
 

-sub do_update {
-#CHECK CONSISTANCY OF tax
+  #CHECK CONSISTANCY OF tax

   #update tax.rate and tax.taxdescription in order to set later NOT NULL constraints
   #update tax.rate and tax.taxdescription in order to set later NOT NULL constraints

-my $query= <<SQL;
-UPDATE tax SET rate=0 WHERE rate IS NULL;
-UPDATE tax SET taxdescription='-' WHERE taxdescription IS NULL;
+  my $query= <<SQL;
+    UPDATE tax SET rate=0 WHERE rate IS NULL;
+    UPDATE tax SET taxdescription='-' WHERE COALESCE(taxdescription, '') = '';

 SQL
 
 SQL
 

-  do_query($query);
+  $self->db_query($query);

 
   #check automatic tax accounts
   $query= <<SQL;
 
   #check automatic tax accounts
   $query= <<SQL;

-SELECT count(*) FROM tax WHERE chart_id NOT IN (SELECT id FROM chart);
+    SELECT count(*) FROM tax WHERE chart_id NOT IN (SELECT id FROM chart);

 SQL
 SQL

-   
-  my ($invalid_tax_account) = $dbh->selectrow_array($query);
-  
+
+  my ($invalid_tax_account) = $self->dbh->selectrow_array($query);
+

   if ($invalid_tax_account > 0){
     #list all invalid tax accounts
     $query = <<SQL;
   if ($invalid_tax_account > 0){
     #list all invalid tax accounts
     $query = <<SQL;

-SELECT id,
-  taxkey,
-  taxdescription, 
-  round(rate * 100, 2) AS rate 
-FROM tax WHERE chart_id NOT IN (SELECT id FROM chart);
+      SELECT id,
+        taxkey,
+        taxdescription,
+        round(rate * 100, 2) AS rate
+      FROM tax WHERE chart_id NOT IN (SELECT id FROM chart);

 SQL
 
 SQL
 

-    my $sth = $dbh->prepare($query);
-    $sth->execute || $main::form->dberror($query);
+    my $sth = $self->dbh->prepare($query);
+    $sth->execute || $::form->dberror($query);

 
 

-    $main::form->{TAX} = [];
+    $::form->{TAX} = [];

     while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {
     while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {

-      push @{ $main::form->{TAX} }, $ref;
+      push @{ $::form->{TAX} }, $ref;

     }
     $sth->finish;
 
     }
     $sth->finish;
 

-    $main::form->{invalid_tax_account} = 1;
+    $::form->{invalid_tax_account} = 1;

     print_error_message();
     return 0;
     print_error_message();
     return 0;

-  } 
+  }

 
   #check entry tax.taxkey of NOT NULL
   $query= <<SQL;
 
   #check entry tax.taxkey of NOT NULL
   $query= <<SQL;

-SELECT count(*) FROM tax WHERE taxkey IS NULL;
+    SELECT count(*) FROM tax WHERE taxkey IS NULL;

 SQL
 SQL

-   
-  my ($taxkey_is_null) = $dbh->selectrow_array($query);
-  
+
+  my ($taxkey_is_null) = $self->dbh->selectrow_array($query);
+

   if ($taxkey_is_null > 0){
     #list all invalid tax accounts
     $query = <<SQL;
   if ($taxkey_is_null > 0){
     #list all invalid tax accounts
     $query = <<SQL;

-SELECT id, 
-  taxdescription, 
-  round(rate * 100, 2) AS rate, 
-  (SELECT accno FROM chart WHERE id = chart_id) AS taxnumber, 
-  (SELECT description FROM chart WHERE id = chart_id) AS account_description
-FROM tax 
-WHERE taxkey IS NULL;
+      SELECT id,
+        taxdescription,
+        round(rate * 100, 2) AS rate,
+        (SELECT accno FROM chart WHERE id = chart_id) AS taxnumber,
+        (SELECT description FROM chart WHERE id = chart_id) AS account_description
+      FROM tax
+      WHERE taxkey IS NULL;

 SQL
 
 SQL
 

-    my $sth = $dbh->prepare($query);
-    $sth->execute || $main::form->dberror($query);
+    my $sth = $self->dbh->prepare($query);
+    $sth->execute || $::form->dberror($query);

 
 

-    $main::form->{TAX} = [];
+    $::form->{TAX} = [];

     while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {
     while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {

-      push @{ $main::form->{TAX} }, $ref;
+      push @{ $::form->{TAX} }, $ref;

     }
     $sth->finish;
 
     }
     $sth->finish;
 

-    $main::form->{taxkey_is_null} = 1;
+    $::form->{taxkey_is_null} = 1;

     print_error_message();
     return 0;
     print_error_message();
     return 0;

-  } 
-#END CHECK OF tax
+  }
+  #END CHECK OF tax

 
 

-#CHECK CONSISTANCY OF taxkeys
+  #CHECK CONSISTANCY OF taxkeys

   #delete invalide entries in taxkeys
   $query= <<SQL;
   #delete invalide entries in taxkeys
   $query= <<SQL;

-DELETE FROM taxkeys 
-WHERE chart_id IS NULL 
-OR chart_id NOT IN (SELECT id FROM chart) 
-OR startdate IS NULL;
+    DELETE FROM taxkeys
+    WHERE chart_id IS NULL
+    OR chart_id NOT IN (SELECT id FROM chart)
+    OR startdate IS NULL;

 SQL
 
 SQL
 

-  do_query($query);
+  $self->db_query($query);

 
 

-#There are 3 cases for taxkeys.tax_id and taxkeys.taxkey_id
-#taxkeys.taxkey_id is NULL and taxkeys.tax_id is not NULL:
+  #There are 3 cases for taxkeys.tax_id and taxkeys.taxkey_id
+  #taxkeys.taxkey_id is NULL and taxkeys.tax_id is not NULL:

 
   #Update taxkeys.taxkey_id with tax.taxkey
   $query= <<SQL;
 
   #Update taxkeys.taxkey_id with tax.taxkey
   $query= <<SQL;

-UPDATE taxkeys 
-SET taxkey_id = (SELECT t.taxkey 
-                        FROM tax t 
+    UPDATE taxkeys
+    SET taxkey_id = (SELECT t.taxkey
+                        FROM tax t

                         WHERE t.id=tax_id)
                         WHERE t.id=tax_id)

-WHERE taxkey_id IS NULL 
-AND tax_id IS NOT NULL;
+    WHERE taxkey_id IS NULL
+    AND tax_id IS NOT NULL;

 SQL
 SQL

-  
-  do_query($query);

 
 

-#taxkeys.taxkey_id and taxkeys.tax_id are NULL:
-  
+  $self->db_query($query);
+
+  #taxkeys.taxkey_id and taxkeys.tax_id are NULL:
+

   #Set taxkey 0 in this case:
   $query= <<SQL;
   #Set taxkey 0 in this case:
   $query= <<SQL;

-UPDATE taxkeys
-SET taxkey_id = 0, tax_id = (SELECT id FROM tax WHERE taxkey=0)
-WHERE taxkey_id IS NULL
-AND tax_id IS NULL;
+    UPDATE taxkeys
+    SET taxkey_id = 0, tax_id = (SELECT id FROM tax WHERE taxkey=0)
+    WHERE taxkey_id IS NULL
+    AND tax_id IS NULL;

 SQL
 SQL

-   
-  do_query($query);

 
 

-#Last case where taxkeys.taxkey_id is not null and taxkeys.tax_id is null
+  $self->db_query($query);
+
+  #Last case where taxkeys.taxkey_id is not null and taxkeys.tax_id is null

 
   #If such entries exist we update with an entry in tax where tax.rate=0
   #and tax.taxkey corresponds to taxkeys.taxkey_id.
 
   #If such entries exist we update with an entry in tax where tax.rate=0
   #and tax.taxkey corresponds to taxkeys.taxkey_id.

-  #If no entry in tax with rate 0 and taxkey taxkeys.taxkey_id exists 
+  #If no entry in tax with rate 0 and taxkey taxkeys.taxkey_id exists

   #we create one.
   $query= <<SQL;
   #we create one.
   $query= <<SQL;

-SELECT DISTINCT taxkey_id
-FROM taxkeys 
-WHERE taxkey_id IS NOT NULL 
-AND tax_id IS NULL;
+    SELECT DISTINCT taxkey_id
+    FROM taxkeys
+    WHERE taxkey_id IS NOT NULL
+    AND tax_id IS NULL;

 SQL
 
 SQL
 

-  my $sth = $dbh->prepare($query);
-  $sth->execute || $main::form->dberror($query);
+  my $sth = $self->dbh->prepare($query);
+  $sth->execute || $::form->dberror($query);

 
 

-  $main::form->{TAXID} = [];
+  $::form->{TAXID} = [];

   my $rowcount = 0;
   while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {
   my $rowcount = 0;
   while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {

-    push @{ $main::form->{TAXID} }, $ref;
+    push @{ $::form->{TAXID} }, $ref;

     $rowcount++;
   }
   $sth->finish;
     $rowcount++;
   }
   $sth->finish;

-  
+

   my $insertquery;
   my $updatequery;
   my $tax_id;
   for my $i (0 .. $rowcount-1){
     $query= qq|
   my $insertquery;
   my $updatequery;
   my $tax_id;
   for my $i (0 .. $rowcount-1){
     $query= qq|

-SELECT id FROM tax WHERE rate = 0 and taxkey=| . $main::form->{TAXID}[$i]->{taxkey_id} . qq| LIMIT 1
-|;  
-    ($tax_id) = $dbh->selectrow_array($query); 
+      SELECT id FROM tax WHERE rate = 0 and taxkey=| . $::form->{TAXID}[$i]->{taxkey_id} . qq| LIMIT 1
+|;
+    ($tax_id) = $self->dbh->selectrow_array($query);

     if ( not $tax_id ){
       $insertquery=qq|
     if ( not $tax_id ){
       $insertquery=qq|

-INSERT INTO tax (rate, taxdescription, taxkey) VALUES (0, '| . $::locale->text('0% tax with taxkey') . $main::form->{TAXID}[$i]->{taxkey_id} .  $::locale->text('. Automatically generated.') . 
-qq|', | . $main::form->{TAXID}[$i]->{taxkey_id} . qq|);
+        INSERT INTO tax (rate, taxdescription, taxkey) VALUES (0, '| . $::locale->text('0% tax with taxkey') . $::form->{TAXID}[$i]->{taxkey_id} .  $::locale->text('. Automatically generated.') .
+        qq|', | . $::form->{TAXID}[$i]->{taxkey_id} . qq|);

 |;
 |;

-      do_query($insertquery);
-      ($tax_id) = $dbh->selectrow_array($query);
-      $tax_id || $main::form->dberror($query); 
+      $self->db_query($insertquery);
+      ($tax_id) = $self->dbh->selectrow_array($query);
+      $tax_id || $::form->dberror($query);

     }
     $updatequery = qq|
     }
     $updatequery = qq|

-UPDATE taxkeys SET tax_id= | . $tax_id . qq| WHERE taxkey_id = | . $main::form->{TAXID}[$i]->{taxkey_id} . qq| AND tax_id IS NULL
+      UPDATE taxkeys SET tax_id= | . $tax_id . qq| WHERE taxkey_id = | . $::form->{TAXID}[$i]->{taxkey_id} . qq| AND tax_id IS NULL

 |;
 |;

-    do_query($updatequery);
+    $self->db_query($updatequery);

   }
 
   }
 

-#The triple taxkey_id, chart_id, startdate in taxkeys has to be unique
+  #The triple taxkey_id, chart_id, startdate in taxkeys has to be unique

   #Select these entries:
   $query= <<SQL;
   #Select these entries:
   $query= <<SQL;

-SELECT DISTINCT tk1.chart_id AS chart_id, tk1.startdate AS startdate
-FROM taxkeys tk1 
-WHERE (SELECT count(*) 
-       FROM taxkeys tk2 
-       WHERE tk2.chart_id  = tk1.chart_id 
-       AND   tk2.startdate = tk1.startdate) > 1;
+    SELECT DISTINCT tk1.chart_id AS chart_id, tk1.startdate AS startdate
+    FROM taxkeys tk1
+    WHERE (SELECT count(*)
+           FROM taxkeys tk2
+           WHERE tk2.chart_id  = tk1.chart_id
+           AND   tk2.startdate = tk1.startdate) > 1;

 SQL
 
 SQL
 

-  $sth = $dbh->prepare($query);
-  $sth->execute || $main::form->dberror($query);
+  $sth = $self->dbh->prepare($query);
+  $sth->execute || $::form->dberror($query);

 
 

-  $main::form->{TAXKEYS} = [];
+  $::form->{TAXKEYS} = [];

   $rowcount = 0;
   while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {
   $rowcount = 0;
   while (my $ref = $sth->fetchrow_hashref("NAME_lc")) {

-    push @{ $main::form->{TAXKEYS} }, $ref;
+    push @{ $::form->{TAXKEYS} }, $ref;

     $rowcount++;
   }
   $sth->finish;
 
   for my $i (0 .. $rowcount-1){
     $rowcount++;
   }
   $sth->finish;
 
   for my $i (0 .. $rowcount-1){

-    $query= qq| 
-DELETE FROM taxkeys tk1 
-WHERE (SELECT count(*) 
-       FROM taxkeys tk2 
-       WHERE tk2.chart_id  = tk1.chart_id 
-       AND   tk2.startdate = tk1.startdate) > 1 
-AND NOT tk1.id = (SELECT id 
-                  FROM taxkeys 
-                  WHERE chart_id  = | . $main::form->{TAXKEYS}[$i]->{chart_id} . qq|
-                  AND   startdate = '| . $main::form->{TAXKEYS}[$i]->{startdate} . qq|'
-                  LIMIT 1)
+    $query= qq|
+      DELETE FROM taxkeys tk1
+      WHERE (SELECT count(*)
+            FROM taxkeys tk2
+            WHERE tk2.chart_id  = tk1.chart_id
+            AND   tk2.startdate = tk1.startdate) > 1
+      AND NOT tk1.id = (SELECT id
+                        FROM taxkeys
+                        WHERE chart_id  = | . $::form->{TAXKEYS}[$i]->{chart_id} . qq|
+                        AND   startdate = '| . $::form->{TAXKEYS}[$i]->{startdate} . qq|'
+                        LIMIT 1)

 |;
 
 |;
 

-    do_query($query);
+    $self->db_query($query);

   }
 
   }
 

-#END CHECK OF taxkeys
+  #END CHECK OF taxkeys

 
 

-#ADD CONSTRAINTS:
-#Now the database is consistent, so we can add constraints:
+  #ADD CONSTRAINTS:
+  #Now the database is consistent, so we can add constraints:

   #Crate NOT NULL constraint for tax.rate with default value 0
   $query= <<SQL;
   #Crate NOT NULL constraint for tax.rate with default value 0
   $query= <<SQL;

-ALTER TABLE tax ALTER COLUMN rate SET NOT NULL;
-ALTER TABLE tax ALTER COLUMN rate SET DEFAULT 0;
+    ALTER TABLE tax ALTER COLUMN rate SET NOT NULL;
+    ALTER TABLE tax ALTER COLUMN rate SET DEFAULT 0;

 SQL
 
 SQL
 

-  do_query($query);
+  $self->db_query($query);

 
   #Create NOT NULL constraint for tax.description
   $query= <<SQL;
 
   #Create NOT NULL constraint for tax.description
   $query= <<SQL;

-ALTER TABLE tax ALTER COLUMN taxdescription SET NOT NULL;
+    ALTER TABLE tax ALTER COLUMN taxdescription SET NOT NULL;

 SQL
 
 SQL
 

-  do_query($query);
+  $self->db_query($query);

 
   #Create foreign key for tax.chart_id to chart.id
   $query= <<SQL;
 
   #Create foreign key for tax.chart_id to chart.id
   $query= <<SQL;

-ALTER TABLE tax ADD FOREIGN KEY (chart_id) REFERENCES chart(id);
+    ALTER TABLE tax ADD FOREIGN KEY (chart_id) REFERENCES chart(id);

 SQL
 SQL

-  
-  do_query($query);
-  
+
+  $self->db_query($query);
+

   #Create NOT NULL constraint for tax.taxkey
   $query= <<SQL;
   #Create NOT NULL constraint for tax.taxkey
   $query= <<SQL;

-ALTER TABLE tax ALTER COLUMN taxkey SET NOT NULL;
+    ALTER TABLE tax ALTER COLUMN taxkey SET NOT NULL;

 SQL
 
 SQL
 

-  do_query($query);
+  $self->db_query($query);

 
   #Create NOT NULL constraint for taxkey.chart_id and foreign key for taxkey.chart_id
   $query= <<SQL;
 
   #Create NOT NULL constraint for taxkey.chart_id and foreign key for taxkey.chart_id
   $query= <<SQL;

-ALTER TABLE taxkeys ALTER COLUMN chart_id SET NOT NULL;
-ALTER TABLE taxkeys ADD FOREIGN KEY (chart_id) REFERENCES chart(id);
+    ALTER TABLE taxkeys ALTER COLUMN chart_id SET NOT NULL;
+    ALTER TABLE taxkeys ADD FOREIGN KEY (chart_id) REFERENCES chart(id);

 SQL
 SQL

-  
-  do_query($query);
+
+  $self->db_query($query);

 
   #Create NOT NULL constraint for taxkey.startdate
   $query= <<SQL;
 
   #Create NOT NULL constraint for taxkey.startdate
   $query= <<SQL;

-ALTER TABLE taxkeys ALTER COLUMN startdate SET NOT NULL;
+    ALTER TABLE taxkeys ALTER COLUMN startdate SET NOT NULL;

 SQL
 SQL

-  
-  do_query($query);
+
+  $self->db_query($query);

 
   #Create NOT NULL constraint for taxkey.taxkey_id
   $query= <<SQL;
 
   #Create NOT NULL constraint for taxkey.taxkey_id
   $query= <<SQL;

-ALTER TABLE taxkeys ALTER COLUMN taxkey_id SET NOT NULL;
+    ALTER TABLE taxkeys ALTER COLUMN taxkey_id SET NOT NULL;

 SQL
 SQL

-   
-  do_query($query);
+
+  $self->db_query($query);

 
   #Create NOT NULL constraint for taxkey.tax_id
   $query= <<SQL;
 
   #Create NOT NULL constraint for taxkey.tax_id
   $query= <<SQL;

-ALTER TABLE taxkeys ALTER COLUMN tax_id SET NOT NULL;
+    ALTER TABLE taxkeys ALTER COLUMN tax_id SET NOT NULL;

 SQL
 SQL

-   
-  do_query($query);
+
+  $self->db_query($query);

 
   #The triple chart_id, taxkey_id, startdate should be unique:
   $query= <<SQL;
 
   #The triple chart_id, taxkey_id, startdate should be unique:
   $query= <<SQL;

-CREATE UNIQUE INDEX taxkeys_chartid_startdate ON taxkeys(chart_id, startdate);
+    CREATE UNIQUE INDEX taxkeys_chartid_startdate ON taxkeys(chart_id, startdate);

 SQL
 SQL

-  
-  do_query($query);
-#ALL CONSTRAINTS WERE ADDED
+
+  $self->db_query($query);
+  #ALL CONSTRAINTS WERE ADDED

 
   return 1;
 
   return 1;

-}; # end do_update
+} # end run

 
 
 sub print_error_message {
 
 
 sub print_error_message {

-  print $main::form->parse_html_template("dbupgrade/tax_constraints");
+  print $::form->parse_html_template("dbupgrade/tax_constraints");

 }
 
 }
 

-return do_update();
+1;