Bugzilla – Attachment 15996 Details for
Bug 7449
billingplace and freedeliveryplace not saved when creating a new basketgroup
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Attachment to Bug 7449 - billingplace and freedeliveryplace not saved when creating a new basketgroup
Bug-7449-billingplace-and-freedeliveryplace-not-sa.patch (text/plain), 3.38 KB, created by
Mathieu Saby
on 2013-03-09 00:36:45 UTC
(
hide
)
Description:
Attachment to Bug 7449 - billingplace and freedeliveryplace not saved when creating a new basketgroup
Filename:
MIME Type:
Creator:
Mathieu Saby
Created:
2013-03-09 00:36:45 UTC
Size:
3.38 KB
patch
obsolete
>From 2bff02067ba2d443f4a3fcfcaf42c5abd0c9ffd3 Mon Sep 17 00:00:00 2001 >From: Fridolyn SOMERS <fridolyn.somers@biblibre.com> >Date: Fri, 22 Feb 2013 10:14:15 +0100 >Subject: [PATCH] Bug 7449: billingplace and freedeliveryplace not saved when > creating a new basketgroup >Content-Type: text/plain; charset="utf-8" > >billingplace and freedeliveryplace are missing in C4::Acquisition::NewBasketgroup. > >Test plan : >- Go to a vendor basket groups >- Create a new basket group >- Enter a name >- Choose a billing place >- Do not choose a delivery place in combobox but enter a text in delivery place textarea >- Enter a comment >- Save >- Edit created basket group >=> Check that billing place and free delivery place are ok > >Signed-off-by: Mathieu Saby <mathieu.saby@univ-rennes2.fr> >--- > C4/Acquisition.pm | 10 ++++++++-- > acqui/basketgroup.pl | 1 + > 2 files changed, 9 insertions(+), 2 deletions(-) > >diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm >index 4a9eae6..2d1da3e 100644 >--- a/C4/Acquisition.pm >+++ b/C4/Acquisition.pm >@@ -643,8 +643,12 @@ $hashref->{'name'} is the 'name' field of the basketgroup in the aqbasketgroups > > $hashref->{'basketlist'} is a list reference of the 'id's of the baskets that belong to this group, > >+$hashref->{'billingplace'} is the 'billingplace' field of the basketgroup in the aqbasketgroups table, >+ > $hashref->{'deliveryplace'} is the 'deliveryplace' field of the basketgroup in the aqbasketgroups table, > >+$hashref->{'freedeliveryplace'} is the 'freedeliveryplace' field of the basketgroup in the aqbasketgroups table, >+ > $hashref->{'deliverycomment'} is the 'deliverycomment' field of the basketgroup in the aqbasketgroups table, > > $hashref->{'closed'} is the 'closed' field of the aqbasketgroups table, it is false if 0, true otherwise. >@@ -656,8 +660,8 @@ sub NewBasketgroup { > die "booksellerid is required to create a basketgroup" unless $basketgroupinfo->{'booksellerid'}; > my $query = "INSERT INTO aqbasketgroups ("; > my @params; >- foreach my $field ('name', 'deliveryplace', 'deliverycomment', 'closed') { >- if ( $basketgroupinfo->{$field} ) { >+ foreach my $field (qw(name billingplace deliveryplace freedeliveryplace deliverycomment closed)) { >+ if ( defined $basketgroupinfo->{$field} ) { > $query .= "$field, "; > push(@params, $basketgroupinfo->{$field}); > } >@@ -700,6 +704,8 @@ $hashref->{'billingplace'} is the 'billingplace' field of the basketgroup in the > > $hashref->{'deliveryplace'} is the 'deliveryplace' field of the basketgroup in the aqbasketgroups table, > >+$hashref->{'freedeliveryplace'} is the 'freedeliveryplace' field of the basketgroup in the aqbasketgroups table, >+ > $hashref->{'deliverycomment'} is the 'deliverycomment' field of the basketgroup in the aqbasketgroups table, > > $hashref->{'closed'} is the 'closed' field of the aqbasketgroups table, it is false if 0, true otherwise. >diff --git a/acqui/basketgroup.pl b/acqui/basketgroup.pl >index 6b7e5bf..a71d9e8 100755 >--- a/acqui/basketgroup.pl >+++ b/acqui/basketgroup.pl >@@ -453,6 +453,7 @@ if ( $op eq "add" ) { > name => $basketgroupname, > booksellerid => $booksellerid, > basketlist => \@baskets, >+ billingplace => $billingplace, > deliveryplace => $deliveryplace, > freedeliveryplace => $freedeliveryplace, > deliverycomment => $deliverycomment, >-- >1.7.9.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 7449
:
15608
|
15996
|
17230