Kosten- und Erfolgstellen zu unterscheiden macht(e) mittlerweile keinen Sinn mehr...
authorJan Büren <jan@lx-office-hosting.de>
Wed, 12 Oct 2011 12:42:49 +0000 (14:42 +0200)
committerJan Büren <jan@lx-office-hosting.de>
Wed, 12 Oct 2011 12:42:49 +0000 (14:42 +0200)
Entsprechend die letzten Rudimente in der Form.pm entfernt und das Datenmodell für Rose angepasst, sowie natürlich die Tabelle

SL/Controller/Department.pm
SL/DB/MetaSetup/Department.pm
SL/Form.pm
sql/Pg-upgrade2/department_drop_role.sql [new file with mode: 0644]
templates/webpages/department/form.html
templates/webpages/department/list.html

index d1ba942..03d3fb6 100644 (file)
@@ -30,7 +30,6 @@ sub action_list {
 sub action_new {
   my ($self) = @_;
 
-  $self->{department} = SL::DB::Department->new(role => 'P');
   $self->render('department/form', title => $::locale->text('Create a new department'));
 }
 
@@ -89,7 +88,6 @@ sub create_or_update {
     $self->render('department/form', title => $is_new ? $::locale->text('Create a new department') : $::locale->text('Edit department'));
     return;
   }
-
   $self->{department}->save;
 
   flash_later('info', $is_new ? $::locale->text('The department has been created.') : $::locale->text('The department has been saved.'));
index 1d1d957..93aa92c 100644 (file)
@@ -12,14 +12,11 @@ __PACKAGE__->meta->setup(
   columns => [
     id          => { type => 'integer', not_null => 1, sequence => 'id' },
     description => { type => 'text' },
-    role        => { type => 'character', default => 'P', length => 1 },
     itime       => { type => 'timestamp', default => 'now()' },
     mtime       => { type => 'timestamp' },
   ],
 
   primary_key_columns => [ 'id' ],
-
-  allow_inline_column_values => 1,
 );
 
 1;
index b21f94c..85ba6f0 100644 (file)
@@ -2757,20 +2757,12 @@ sub all_vc {
   @{ $self->{all_employees} } =
     sort { $a->{name} cmp $b->{name} } @{ $self->{all_employees} };
 
-  if ($module eq 'AR') {
 
     # prepare query for departments
     $query = qq|SELECT id, description
                 FROM department
-                WHERE role = 'P'
                 ORDER BY description|;
 
-  } else {
-    $query = qq|SELECT id, description
-                FROM department
-                ORDER BY description|;
-  }
-
   $self->{all_departments} = selectall_hashref_query($self, $dbh, $query);
 
   # get languages
@@ -2840,15 +2832,9 @@ sub all_departments {
   my ($self, $myconfig, $table) = @_;
 
   my $dbh = $self->get_standard_dbh($myconfig);
-  my $where;
-
-  if ($table eq 'customer') {
-    $where = "WHERE role = 'P' ";
-  }
 
   my $query = qq|SELECT id, description
                  FROM department
-                 $where
                  ORDER BY description|;
   $self->{all_departments} = selectall_hashref_query($self, $dbh, $query);
 
diff --git a/sql/Pg-upgrade2/department_drop_role.sql b/sql/Pg-upgrade2/department_drop_role.sql
new file mode 100644 (file)
index 0000000..5a0ceb5
--- /dev/null
@@ -0,0 +1,8 @@
+-- @tag: department_drop_role
+-- @description: Kosten- und Erfolgstellen zu unterscheiden, macht(e) mittlerweile keinen Sinn mehr, da man ja entsprechend Kosten als Erfolg auf eine Kostenstelle buchen möchte. Ferner wird auch die Auswahlliste schon länger nicht mehr unterschieden
+-- @depends: release_2_6_3
+-- @charset: utf-8
+-- @ignore: 0
+
+
+ALTER TABLE department  DROP COLUMN role;
index 2dc0dd9..d009ea0 100644 (file)
     <td>[%- 'Description' | $T8 %]</td>
     <td>[% L.input_tag("department.description", SELF.department.description) %]</td>
    </tr>
-
-   <tr>
-    <td valign="top">[%- 'Type' | $T8 %]</td>
-    <td valign="top">
-     [%- IF is_used %]
-      [% L.hidden_tag("role", SELF.department.role) %]
-      [%- IF SELF.department.role == "C" %][%- LxERP.t8('Cost Center') %][%- ELSE %][%- LxERP.t8('Profit Center') %][%- END %]
-     [%- ELSE %]
-      [% L.radio_button_tag("department.role", "value", "C", "label", LxERP.t8("Cost Center"), "checked", SELF.department.role == "C") %]
-      <br>
-      [% L.radio_button_tag("department.role", "value", "P", "label", LxERP.t8("Profit Center"), "checked", SELF.department.role == "P") %]
-     [%- END %]
-    </td>
-   </tr>
   </table>
 
   <p>
index 059aa33..7433aa9 100644 (file)
@@ -16,8 +16,6 @@
     <thead>
     <tr class="listheading">
      <th width="100%">[%- 'Description' | $T8 %]</th>
-     <th>[%- 'Cost Center' | $T8 %]</th>
-     <th>[%- 'Profit Center' | $T8 %]</th>
     </tr>
     </thead>
 
@@ -29,8 +27,6 @@
        [%- HTML.escape(department.description) %]
       </a>
      </td>
-     <td align="center">[%- IF department.role == 'C' %]X[%- END %]</td>
-     <td align="center">[%- IF department.role == 'P' %]X[%- END %]</td>
     </tr>
     [%- END %]
     </tbody>