Bugzilla – Attachment 27300 Details for
Bug 9416
We should have 2 kinds of notes for each order line
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PATCH 2/3] Bug 9416 QA Followup: fix neworderempty and templates
0002-follow.patch (text/plain), 5.69 KB, created by
Mathieu Saby
on 2014-04-19 08:34:29 UTC
(
hide
)
Description:
[PATCH 2/3] Bug 9416 QA Followup: fix neworderempty and templates
Filename:
MIME Type:
Creator:
Mathieu Saby
Created:
2014-04-19 08:34:29 UTC
Size:
5.69 KB
patch
obsolete
>From 9b0001abbab590d2f58c2af377bebc929ea2b95b Mon Sep 17 00:00:00 2001 >From: Mathieu Saby <mathsabypro@gmail.com> >Date: Sat, 19 Apr 2014 10:21:51 +0200 >Subject: [PATCH 2/3] Bug 9416 QA Followup: fix neworderempty and templates >Content-Type: text/plain; charset="utf-8" > >This followup answer QA remarks : >- neworderempty.pl updated so that the 2 new variables are passed to the template >- modordernotes.tt fixed to make the translation easier >- in CSV headers, to make clear that no change are made for the moment, rename "note" to "vendor note" > >Additionnaly, "Publisher code" was wrong in the csv headers. I changed it to "Publisher" (the field in database is publishercode, but the content is a real publisher name, not a code) >I did not change "Note:" in modordernotes.tt, because it is just under a h1 tag which specifies the type of note the librarian is being editing > >Test plan : >- edit an existing order, and try to change/add/suppress the vendor note, and the internal note. Check the changes are properly saved >- export a basket and a basketgroup in CSV. Check the columns headers are "Publisher" and "Vendor note" > >--- > acqui/neworderempty.pl | 7 ++++--- > .../prog/en/includes/csv_headers/acqui/basket.tt | 2 +- > .../en/includes/csv_headers/acqui/basketgroup.tt | 2 +- > .../prog/en/modules/acqui/modordernotes.tt | 4 ++-- > 4 files changed, 8 insertions(+), 7 deletions(-) > >diff --git a/acqui/neworderempty.pl b/acqui/neworderempty.pl >index 9fdf032..879d2f8 100755 >--- a/acqui/neworderempty.pl >+++ b/acqui/neworderempty.pl >@@ -335,7 +335,8 @@ if ( defined $subscriptionid ) { > $data->{ecost} = $lastOrderReceived->{ecost}; > $data->{quantity} = $lastOrderReceived->{quantity}; > $data->{unitprice} = $lastOrderReceived->{unitprice}; >- $data->{notes} = $lastOrderReceived->{notes}; >+ $data->{order_internalnote} = $lastOrderReceived->{order_internalnote}; >+ $data->{order_vendornote} = $lastOrderReceived->{order_vendornote}; > $data->{sort1} = $lastOrderReceived->{sort1}; > $data->{sort2} = $lastOrderReceived->{sort2}; > >@@ -384,6 +385,8 @@ $template->param( > discount => $bookseller->{'discount'}, > orderdiscount_2dp => sprintf( "%.2f", $data->{'discount'} || 0 ), > orderdiscount => $data->{'discount'}, >+ order_internalnote => $data->{'order_internalnote'}, >+ order_vendornote => $data->{'order_vendornote'}, > listincgst => $bookseller->{'listincgst'}, > invoiceincgst => $bookseller->{'invoiceincgst'}, > name => $bookseller->{'name'}, >@@ -418,8 +421,6 @@ $template->param( > (uc(C4::Context->preference("marcflavour"))) => 1 > ); > >-$template->param ( notes => $data->{'notes'} ) if ( $ordernumber ); >- > output_html_with_http_headers $input, $cookie, $template->output; > > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basket.tt >index efcf292..8a5125b 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basket.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basket.tt >@@ -1 +1 @@ >-Contract name,Order number,Entry date,ISBN,Author,Title,Publication year,Publisher code,Collection title,Notes,Quantity,RRP,Delivery place,Billing place >+Contract name,Order number,Entry date,ISBN,Author,Title,Publication year,Publisher,Collection title,Vendor note,Quantity,RRP,Delivery place,Billing place >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basketgroup.tt b/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basketgroup.tt >index d7650c1..783ae49 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basketgroup.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/csv_headers/acqui/basketgroup.tt >@@ -1 +1 @@ >-Account number,Basket name,Order number,Author,Title,Publisher code,Publication year,Collection title,ISBN,Quantity,RRP,Discount,Estimated cost,Notes,Entry date,Bookseller name,Bookseller physical address,Bookseller postal address,Contract number,Contract name,Basket group delivery place,Basket group billing place,Basket delivery place,Basket billing place >+Account number,Basket name,Order number,Author,Title,Publisher,Publication year,Collection title,ISBN,Quantity,RRP,Discount,Estimated cost,Vendor note,Entry date,Bookseller name,Bookseller physical address,Bookseller postal address,Contract number,Contract name,Basket group delivery place,Basket group billing place,Basket delivery place,Basket billing place >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/modordernotes.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/modordernotes.tt >index ee60b0e..6ade209 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/modordernotes.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/modordernotes.tt >@@ -1,5 +1,5 @@ > [% INCLUDE 'doc-head-open.inc' %] >-<title>Koha › Acquisition › Change order [% IF (type == "vendor") %]vendor[% ELSE %]internal[% END %] note</title> >+<title>Koha › Acquisition › Change order [% IF (type == "vendor") %]vendor note[% ELSE %]internal note[% END %]</title> > [% INCLUDE 'doc-head-close.inc' %] > </head> > >@@ -12,7 +12,7 @@ > > <div id="bd"> > <div id="yui-main"> >- <h1>Change order [% IF (type == "vendor") %]vendor[% ELSE %]internal[% END %] note (order no. [% ordernumber %])</h1> >+ <h1>Change order [% IF (type == "vendor") %]vendor note[% ELSE %]internal note[% END %] (order no. [% ordernumber %])</h1> > <form action="/cgi-bin/koha/acqui/modordernotes.pl" method="post"> > <fieldset class="brief"> > <label for="ordernotes">Note:</label> >-- >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 9416
:
21293
|
21964
|
23798
|
25329
|
25335
|
27231
|
27274
|
27299
|
27300
|
27301
|
27302
|
27304
|
27305
|
27306