Bug 33721

Summary: Show inactive funds in invoice.tt out of order
Product: Koha Reporter: Caroline Cyr La Rose <caroline.cyr-la-rose>
Component: TemplatesAssignee: Katrin Fischer <katrin.fischer>
Status: RESOLVED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: barbara.johnson, fridolin.somers, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.07
Bug Depends on:    
Bug Blocks: 34452, 34492    
Attachments: Screenshot of the invoice section with the fund line
Bug 33721: Fix display of shipping cost/fund on invoice summary page
Bug 33721: Fix display of shipping cost/fund on invoice summary page
Bug 33721: Fix display of shipping cost/fund on invoice summary page

Description Caroline Cyr La Rose 2023-05-11 20:50:29 UTC
Created attachment 151100 [details]
Screenshot of the invoice section with the fund line

When viewing an invoice, the elements in the 'Fund' line are out of order.

The is the 'Fund:' label, followed by 'Show inactive:' label, then the fund drop-down and then the show inactive checkbox.

The order should be 'Fund:' label, fund drop-down, 'Show inactive:' label, show inactive checkbox.


To recreate:
1. In acquisitions, create an order and close the basket
2. Click 'Receive shipment' and receive the order
3. Click 'Finish receiving'
   --> The Fund line in invoice.tt is out of order
Comment 1 Katrin Fischer 2023-05-14 11:22:33 UTC
Created attachment 151171 [details] [review]
Bug 33721: Fix display of shipping cost/fund on invoice summary page

When viewing an invoice, the elements in the 'Fund' line were out of order:
'Fund:' label, followed by 'Show inactive:' label, then the fund drop-down and then the show inactive checkbox.

The order should be:
'Fund:' label, fund drop-down, 'Show inactive:' label, show inactive checkbox.

This copies the markup that is used when you click on "Receive shipment".
It also changes 'Fund' to 'Shipping fund' to bring the forms even more in line.

To recreate:
1. In acquisitions, create an order and close the basket
2. Click 'Receive shipment' and receive the order
3. Click 'Finish receiving'
   --> The Fund line in invoice.tt is out of order
4. Apply patch
5. Verify the order is now fixed
Comment 2 ByWater Sandboxes 2023-05-14 16:44:46 UTC
Created attachment 151177 [details] [review]
Bug 33721: Fix display of shipping cost/fund on invoice summary page

When viewing an invoice, the elements in the 'Fund' line were out of order:
'Fund:' label, followed by 'Show inactive:' label, then the fund drop-down and then the show inactive checkbox.

The order should be:
'Fund:' label, fund drop-down, 'Show inactive:' label, show inactive checkbox.

This copies the markup that is used when you click on "Receive shipment".
It also changes 'Fund' to 'Shipping fund' to bring the forms even more in line.

To recreate:
1. In acquisitions, create an order and close the basket
2. Click 'Receive shipment' and receive the order
3. Click 'Finish receiving'
   --> The Fund line in invoice.tt is out of order
4. Apply patch
5. Verify the order is now fixed

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Comment 3 Katrin Fischer 2023-05-14 16:51:07 UTC
Thanks, Barbara!
Comment 4 Marcel de Rooy 2023-05-17 14:20:54 UTC
Created attachment 151342 [details] [review]
Bug 33721: Fix display of shipping cost/fund on invoice summary page

When viewing an invoice, the elements in the 'Fund' line were out of order:
'Fund:' label, followed by 'Show inactive:' label, then the fund drop-down and then the show inactive checkbox.

The order should be:
'Fund:' label, fund drop-down, 'Show inactive:' label, show inactive checkbox.

This copies the markup that is used when you click on "Receive shipment".
It also changes 'Fund' to 'Shipping fund' to bring the forms even more in line.

To recreate:
1. In acquisitions, create an order and close the basket
2. Click 'Receive shipment' and receive the order
3. Click 'Finish receiving'
   --> The Fund line in invoice.tt is out of order
4. Apply patch
5. Verify the order is now fixed

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Tomás Cohen Arazi 2023-05-18 14:04:50 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 6 Matt Blenkinsop 2023-06-08 16:43:20 UTC
Nice work everyone!

Pushed to stable for 22.11.x
Comment 7 Fridolin Somers 2023-08-04 18:39:27 UTC
Ah OK all places use style in label tag :

 > git grep 'Show inactive:</label>'
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt:                                                    <label for="all_showallbudgets" style="float:none;width:auto;">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/duplicate_orders.tt:                  <label for="all_showallbudgets" style="float:none;width:auto;margin-right:0;">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/invoice.tt:    <label for="showallfunds" style="float:none;width:auto;">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt:                <label for="showallbudgets" style="float:none;">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt:                                            <label for="showallbudgets">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt:            <label for="showallfunds" style="float:none;width:auto;">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/reports/orders_by_budget.tt:        <label for="showallbudgets" style="float:none;">&nbsp;Show inactive:</label>
koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt:                            <label for="showallfunds" style="float:none;width:auto;">&nbsp;Show inactive:</label>

It would be better to use a class and CSS rule in staff file.

For another bug report of course ;)