Add TT syntax for ACQORDER notices.
Had this on my wish list to file too - the default notice is not very nice. Did you see you can use [% basket %] now? (Bug 31858). That should allow us to build something much nicer.
(In reply to Katrin Fischer from comment #1) > Did you see you can use [% basket %] now? (Bug 31858). That should allow us Wrong bug number ?
(In reply to Alex Arnaud from comment #2) > (In reply to Katrin Fischer from comment #1) > > Did you see you can use [% basket %] now? (Bug 31858). That should allow us > Wrong bug number ? Typical me... bug 31587
Created attachment 142160 [details] [review] Bug 31858: add TT syntax for ACQORDER notices Test Plan: 1) Set your ACQORDER notice content to: [%- USE Price -%] [% bookseller.name %] [% FOREACH order IN orders %] Ordernumber [% order.ordernumber %] ([% order.biblio.title %] - [% order.biblio.biblioitem.publicationyear %]) (quantity: [% order.quantity %]) ($[% order.listprice | $Price %] each). [% END %] Thank you, [% branch.branchname %] 2) Set up a vendor with an email, set contact for email so the "E-mail order" button shows on a basket for that vendor 3) E-mail the basket 4) Check the content is correct
Created attachment 142161 [details] [review] Bug 31858: unit tests
Created attachment 142162 [details] [review] Bug 31858: add TT syntax for ACQORDER notices Test Plan: 1) Set your ACQORDER notice content to: [%- USE Price -%] [% bookseller.name %] [% FOREACH order IN orders %] Ordernumber [% order.ordernumber %] ([% order.biblio.title %] - [% order.biblio.biblioitem.publicationyear %]) (quantity: [% order.quantity %]) ($[% order.listprice | $Price %] each). [% END %] Thank you, [% branch.branchname %] 2) Set up a vendor with an email, set contact for email so the "E-mail order" button shows on a basket for that vendor 3) E-mail the basket 4) Check the content is correct
Created attachment 142163 [details] [review] Bug 31858: unit tests
Created attachment 145246 [details] [review] Bug 31858: unit tests Signed-off-by: Marius Mandrescu <marius.mandrescu@inLibro.com>
Created attachment 146904 [details] [review] Bug 31858: add TT syntax for ACQORDER notices Test Plan: 1) Set your ACQORDER notice content to: [%- USE Price -%] [% bookseller.name %] [% FOREACH order IN orders %] Ordernumber [% order.ordernumber %] ([% order.biblio.title %] - [% order.biblio.biblioitem.publicationyear %]) (quantity: [% order.quantity %]) ($[% order.listprice | $Price %] each). [% END %] Thank you, [% branch.branchname %] 2) Set up a vendor with an email, set contact for email so the "E-mail order" button shows on a basket for that vendor 3) E-mail the basket 4) Check the content is correct Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 146905 [details] [review] Bug 31858: add TT syntax for ACQORDER notices Test Plan: 1) Set your ACQORDER notice content to: [%- USE Price -%] [% bookseller.name %] [% FOREACH order IN orders %] Ordernumber [% order.ordernumber %] ([% order.biblio.title %] - [% order.biblio.biblioitem.publicationyear %]) (quantity: [% order.quantity %]) ($[% order.listprice | $Price %] each). [% END %] Thank you, [% branch.branchname %] 2) Set up a vendor with an email, set contact for email so the "E-mail order" button shows on a basket for that vendor 3) E-mail the basket 4) Check the content is correct Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 146906 [details] [review] Bug 31858: unit tests Signed-off-by: Marius Mandrescu <marius.mandrescu@inLibro.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to stable for 22.11.x
Could someone confirm that there is no way to access additional fields from baskets yet? I got as far as [% order.basket... %], but we seem to be missing a method here.
Enhancement will not be backported to 22.05.x
(In reply to Katrin Fischer from comment #14) > Could someone confirm that there is no way to access additional fields from > baskets yet? I got as far as [% order.basket... %], but we seem to be > missing a method here. What data do you need to get to?
(In reply to Kyle M Hall from comment #16) > (In reply to Katrin Fischer from comment #14) > > Could someone confirm that there is no way to access additional fields from > > baskets yet? I got as far as [% order.basket... %], but we seem to be > > missing a method here. > > What data do you need to get to? additional fields for baskets in additional_fields_values.