Populate items.cn_source in acq receive. However, items.cn_sort is not calculated. See bug 2403.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
This commit is contained in:
Ryan Higgins 2008-07-25 09:35:44 -05:00 committed by Joshua Ferraro
parent fa28afba18
commit 0d7265bd73

View file

@ -58,9 +58,8 @@ my $cnt=0;
my $error_url_str;
if ($quantityrec > $origquantityrec ) {
my @items_err ;
foreach my $bc (@barcode) {
my $itemRecord = TransformKohaToMarc({
my $item_hash = {
"items.replacementprice" => $replacement,
"items.price" => $cost,
"items.booksellerid" => $supplierid,
@ -71,13 +70,16 @@ if ($quantityrec > $origquantityrec ) {
"items.itype" => $itemtype[$cnt],
"items.location" => $location[$cnt],
"items.enumchron" => $enumchron[$cnt], # FIXME : No integration here with serials module.
"items.loan" => 0, });
"items.loan" => 0,
};
$item_hash->{'items.cn_source'} = C4::Context->preference('DefaultClassificationSource') if(C4::Context->preference('DefaultClassificationSource') );
# FIXME : cn_sort is populated by Items::_set_derived_columns_for_add , which is never called with AddItemFromMarc . Bug 2403
my $itemRecord = TransformKohaToMarc($item_hash);
$cnt++;
my $item_hash = TransformMarcToKoha(undef,$itemRecord,'','items');
# FIXME: possible race condition here. duplicate barcode check should happen in AddItem, but for now we have to do it here.
$item_hash = TransformMarcToKoha(undef,$itemRecord,'','items');
# FIXME: possible race condition. duplicate barcode check should happen in AddItem, but for now we have to do it here.
my %err = CheckItemPreSave($item_hash);
if(%err) {
push @items_err, \%err;
for my $err_cnd (keys %err) {
$error_url_str .= "&error=" . $err_cnd . "&error_param=" . $err{$err_cnd};
}