Revision 1ed28475
Von Jan Büren vor fast 13 Jahren hinzugefügt
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
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 <m.bunkus@linet-services.de>
Date: Fri Jan 18 17:01:49 2008 +0000
...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.