Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision 1ed28475

Von Jan Büren vor fast 13 Jahren hinzugefügt

  • ID 1ed2847584d7d52092876817cd6678126f9e9421
  • Vorgänger 74446031
  • Nachfolger 4af0c21e

Die Variable $readonly in display_row wird niemals ausgelesen.
S.a. Anmerkung von Sven:
...und hier wird es nur gesetzt, aber nicht konsumiert. Seit den strict Änderungen wird auch keine Variable mehr indirekt aufgerufen (a la $name = 'readobly'; print
$$name), und wird auch nicht implizit an html oder print Templates weitergeleitet.
git blame sagt zu io.pl:127:

commit d707f7ac60b9dbe7da50d733db1e73eae84952f7
Author: Moritz Bunkus <>
Date: Fri Jan 18 17:01:49 2008 +0000

Lieferscheine im Einkauf und Verkauf. Bisher nur gemerget, noch nicht getestet.

...der Commit ist berüchtigt, weil Mosu da nur die Aufträge copy&pasted hat, und tabellen und bezeichner geändert hat. In dem Commit sind ein paar tote Codestellen reingekommen.

Ich würd sagen, kannste rausschmeissen.

Unterschiede anzeigen:

bin/mozilla/io.pl
116 116

  
117 117
  my $numrows = shift;
118 118

  
119
  my ($readonly, $stock_in_out, $stock_in_out_title);
119
  my ($stock_in_out, $stock_in_out_title);
120 120

  
121 121
  my $is_purchase        = (first { $_ eq $form->{type} } qw(request_quotation purchase_order purchase_delivery_order)) || ($form->{script} eq 'ir.pl');
122 122
  my $show_min_order_qty =  first { $_ eq $form->{type} } qw(request_quotation purchase_order);
......
124 124
  my $is_s_p_order       = (first { $_ eq $form->{type} } qw(sales_order purchase_order));
125 125

  
126 126
  if ($is_delivery_order) {
127
    $readonly             = ' readonly' if ($form->{closed});
128

  
129 127
    if ($form->{type} eq 'sales_delivery_order') {
130 128
      $stock_in_out_title = $locale->text('Release From Stock');
131 129
      $stock_in_out       = 'out';

Auch abrufbar als: Unified diff