Spool-Dateien: Handling vom wiederholten Spoolen desselben Beleges gefixt

[kivitendo-erp.git] / bin / mozilla / ic.pl
diff --git a/bin/mozilla/ic.pl b/bin/mozilla/ic.pl

index 1eddce3..b86623b 100644 (file)
--- a/bin/mozilla/ic.pl
+++ b/bin/mozilla/ic.pl
@@ -38,6 +38,7 @@ use List::MoreUtils qw(any);
  use SL::AM;
  use SL::CVar;
  use SL::IC;
+use SL::Helper::Flash;
  use SL::ReportGenerator;
  
  #use SL::PE;
@@ -288,7 +289,6 @@ sub update_prices {
  #  $form->header;
  #
  #  print qq|
-#<body>
  #  <form method=post action=ic.pl>
  #    <table width=100%>
  #     <tr>
@@ -388,8 +388,6 @@ sub update_prices {
  #    . $locale->text('TOP100') . qq|">
  #
  #</form>
-#</body>
-#</html>
  #|;
  #  $lxdebug->leave_sub();
  #}    #end list()
@@ -725,8 +723,6 @@ sub addtop100 {
    my $colspan = $#column_index + 1;
  
    print qq|
-<body>
-
  <table width=100%>
    <tr>
      <th class=listtop colspan=$colspan>$form->{title}</th>
@@ -981,9 +977,6 @@ sub addtop100 {
      . $locale->text('choice') . qq|">
  
    </form>
-
-</body>
-</html>
  |;
  
    $lxdebug->leave_sub();
@@ -1032,6 +1025,7 @@ sub generate_report {
      'bin'                => { 'text' => $locale->text('Bin'), },
      'deliverydate'       => { 'text' => $locale->text('deliverydate'), },
      'description'        => { 'text' => $locale->text('Part Description'), },
+    'notes'              => { 'text' => $locale->text('Notes'), },
      'drawing'            => { 'text' => $locale->text('Drawing'), },
      'ean'                => { 'text' => $locale->text('EAN'), },
      'image'              => { 'text' => $locale->text('Image'), },
@@ -1214,7 +1208,7 @@ sub generate_report {
    IC->all_parts(\%myconfig, \%$form);
  
    my @columns = qw(
-    partnumber description partsgroup bin onhand rop soldtotal unit listprice
+    partnumber description notes partsgroup bin onhand rop soldtotal unit listprice
      linetotallistprice sellprice linetotalsellprice lastcost linetotallastcost
      priceupdate weight image drawing microfiche invnumber ordnumber quonumber
      transdate name serialnumber deliverydate ean projectnumber projectdescription
@@ -1331,11 +1325,11 @@ sub generate_report {
      $row->{description}->{link} = $edit_link;
  
      foreach (qw(sellprice listprice lastcost)) {
-      $row->{$_}{data}            = $form->format_amount(\%myconfig, $ref->{$_}, -2);
+      $row->{$_}{data}            = $form->format_amount(\%myconfig, $ref->{$_}, 2);
        $row->{"linetotal$_"}{data} = $form->format_amount(\%myconfig, $ref->{onhand} * $ref->{$_}, 2);
      }
      foreach ( @pricegroup_columns ) {
-      $row->{$_}{data}            = $form->format_amount(\%myconfig, $ref->{"$_"}, -2);
+      $row->{$_}{data}            = $form->format_amount(\%myconfig, $ref->{"$_"}, 2);
      };
  
  
@@ -1592,6 +1586,10 @@ sub form_header {
    IC->retrieve_buchungsgruppen(\%myconfig, $form);
    @{ $form->{BUCHUNGSGRUPPEN} } = grep { $_->{id} eq $form->{buchungsgruppen_id} || ($form->{id} && $form->{orphaned}) || !$form->{id} } @{ $form->{BUCHUNGSGRUPPEN} };
  
+  if (!SL::TransNumber->new(number => $form->{partnumber}, type => $form->{item}, id => $form->{id})->is_unique) {
+    flash('info', $::locale->text('This partnumber is not unique. You should change it.'));
+  }
+
    # use JavaScript Calendar or not (yes!)
    $form->{jsscript} = 1;
  
@@ -1600,7 +1598,7 @@ sub form_header {
  
    $form->{defaults} = AM->get_defaults();
  
-  $form->{fokus} = "ic.partnumber";
+  $::request->{layout}->focus("#partnumber");
  
    $form->{CUSTOM_VARIABLES} = CVar->get_custom_variables('module' => 'IC', 'trans_id' => $form->{id});
  
@@ -1841,7 +1839,6 @@ sub save {
    $form->error($locale->text('Partnumber must not be set to empty!')) if $form->{id} && !$form->{partnumber};
  
    # save part
-  $lxdebug->message($LXDebug::DEBUG1, "ic.pl: sellprice in save = $form->{sellprice}\n");
    if (IC->save(\%myconfig, \%$form) == 3) {
      $form->error($locale->text('Partnumber not unique!'));
    }
@@ -1909,8 +1906,6 @@ sub save {
          $form->{"sellprice_$i"} /= $form->{exchangerate};
        }
  
-      $lxdebug->message($LXDebug::DEBUG1, qq|sellprice_$i in previousform 2 = | . $form->{"sellprice_$i"} . qq|\n|);
-
        map { $form->{"taxaccounts_$i"} .= "$_ " } split / /, $newform{taxaccount};
        chop $form->{"taxaccounts_$i"};
        foreach my $item (qw(description rate taxnumber)) {
@@ -1959,7 +1954,6 @@ sub save {
      }
      $form->{callback} = $callback;
    }
-  $lxdebug->message($LXDebug::DEBUG1, qq|ic.pl: sellprice_$i nach sub save = | . $form->{"sellprice_$i"} . qq|\n|);
  
    # redirect
    $form->redirect;