Fehler #273

Bei Schweizer Kontenplänen erscheint beim Aufruf der Maske zum Dialogbuchen eine Fehlermeldung

Von Andreas Rudin vor 9 Monaten hinzugefügt. Vor 9 Monaten aktualisiert.

Status:GelöstBeginn:16.07.2017
Priorität:SofortAbgabedatum:
Zugewiesen an:-% erledigt:

100%

Kategorie:-
Zielversion:3.5

Beschreibung

kivitendo 3.4.1 unstable seit commit ad635c66023bc107d55b2f2f41f6fcadabd02cc3, sowie weiterhin in aktueller 3.5-beta

Bei einer Neu-Installation mit irgendeinem CH Kontenplan können keine Dialogbuchungen ausgeführt werden.

Beim Aufruf von "Finanzbuchhaltung → Dialogbuchen" erscheint folgende Fehlermeldung:

Can't call method "id" on an undefined value at bin/mozilla/gl.pl line 704 (commit ad635c66023bc107d55b2f2f41f6fcadabd02cc3)

bzw.

Can't call method "id" on an undefined value at bin/mozilla/gl.pl line 835 (3.5-beta vom 16.7.2017)

Bei Neu-Installationen mit SKR03 oder SKR04 tritt der Fehler nicht auf.

Hier der Diff aus commit ad635c66 der direkt betroffenen Zeilen in der bin/mozilla/gl.pl:

@@ -690,49 +681,31 @@ sub display_rows {
       <input type="hidden" name="memo_$i" value="$form->{"memo_$i"}" size="16">|;
     }

-    my $selected_accno_full;
-    my ($accno_row) = split(/--/, $form->{"accno_$i"});
-    my $item = $charts{$accno_row};
-    $selected_accno_full = "$item->{accno}--$item->{tax_id}";
-
-    my $selected_taxchart = $form->{"taxchart_$i"};
-    my ($selected_accno, $selected_tax_id) = split(/--/, $selected_accno_full);
-    my ($previous_accno, $previous_tax_id) = split(/--/, $form->{"previous_accno_$i"});
-
     my %taxchart_labels = ();
     my @taxchart_values = ();
-    my %taxcharts = ();
-    my $filter_accno;
-    $filter_accno = $::form->{ALL_CHARTS}[0]->{accno};
-    $filter_accno = $selected_accno if (!$init and $i < $form->{rowcount});
-    foreach my $item ( GL->get_tax_dropdown($filter_accno) ) {
-      my $key = $item->{id} . "--" . $item->{rate};
-      $taxchart_init = $key if ($taxchart_init == $item->{id});
-      push(@taxchart_values, $key);
-      $taxchart_labels{$key} = $item->{taxdescription} . " " . $item->{rate} * 100 . ' %';
-      $taxcharts{$item->{id}} = $item;
-    }

-    if ($previous_accno &&
-        ($previous_accno eq $selected_accno) &&
-        ($previous_tax_id ne $selected_tax_id)) {
-      my $item = $taxcharts{$selected_tax_id};
-      $selected_taxchart = "$item->{id}--$item->{rate}";
+    my $accno_id          = $::form->{"accno_id_$i"};
+    my $chart             = $charts_by_id{$accno_id} // $default_chart;
+    $accno_id             = $chart->{id};
+    my $chart_has_changed = $::form->{"previous_accno_id_$i"} && ($accno_id != $::form->{"previous_accno_id_$i"});
+    my ($first_taxchart, $default_taxchart, $taxchart_to_use);
+
+    foreach my $item ( GL->get_active_taxes_for_chart($accno_id, $transdate) ) {
+      my $key             = $item->id . "--" . $item->rate;
+      $first_taxchart   //= $item;
+      $default_taxchart   = $item if $item->{is_default};
+      $taxchart_to_use    = $item if $key eq $form->{"taxchart_$i"};
+
+      push(@taxchart_values, $key);
+      $taxchart_labels{$key} = $item->taxdescription . " " . $item->rate * 100 . ' %';
     }

-    $selected_accno      = '' if ($init);
-    $selected_taxchart ||= $taxchart_init;
+    $taxchart_to_use      = $default_taxchart // $first_taxchart if $chart_has_changed || !$taxchart_to_use;
+    my $selected_taxchart = $taxchart_to_use->id . '--' . $taxchart_to_use->rate;

     my $accno = qq|<td>| .
-      NTI($cgi->popup_menu('-name' => "accno_$i",
-                           '-id' => "accno_$i",
-                           '-onChange' => "updateTaxes($i);",
-                           '-style' => 'width:200px',
-                           '-values' => \@chart_values,
-                           '-labels' => \%chart_labels,
-                           '-default' => $selected_accno_full))
-      . $cgi->hidden('-name' => "previous_accno_$i",
-                     '-default' => $selected_accno_full)
+      $::request->presenter->chart_picker("accno_id_$i", $accno_id, style => "width: 300px") .
+      $::request->presenter->hidden_tag("previous_accno_id_$i", $accno_id)
       . qq|</td>|;
     my $tax_ddbox = qq|<td>| .
       NTI($cgi->popup_menu('-name' => "taxchart_$i",

Fehler_Dialogbuchen_SwissCharts.png - Fehlermeldung bei commit ad635c66 (14,287 KB) Andreas Rudin, 16.07.2017 13:28

Zugehörige Revisionen

Revision 4f99ff61
Von Jan Büren vor 9 Monaten hinzugefügt

behebt #273 Überschriftskonten nicht im Dialog bebuchen

Historie

#1 Von Jan Büren vor 9 Monaten aktualisiert

-  foreach my $item (@{ $form->{ALL_CHARTS} }) {
-    if ($item->{charttype} eq 'H'){ # skip headings
-      next;
-    }

Das wurde zuviel entfernt.

#2 Von Jan Büren vor 9 Monaten aktualisiert

  • Status wurde von Neu zu Gelöst geändert
  • % erledigt wurde von 0 zu 100 geändert

Status geändert durch Changeset kivitendo-erp|commit:4f99ff6136d6198ba3f7a842c903deaaf4e47a1b.

Auch abrufbar als: Atom PDF