Attachment 158644 Details for Bug 31041 – Bug 31041: Prevent duplicate display of cashup summary when printing

[patch] Bug 31041: Prevent duplicate display of cashup summary when printing

Bug-31041-Prevent-duplicate-display-of-cashup-summ.patch (text/plain), 2.83 KB, created by Martin Renvoize (ashimema) on 2023-11-08 07:32:23 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2023-11-08 07:32:23 UTC

Size: 2.83 KB

patch

obsolete

>From 1d70ea4a0807f635245233e25cc4287870ceb44f Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Thu, 2 Nov 2023 19:56:00 +0000
>Subject: [PATCH] Bug 31041: Prevent duplicate display of cashup summary when
> printing
>
>To test:
>1. Have POS on.
>2. Have some registers and items for sale. Make some sales to 'Cashup'.
>3. If you have enough enough lines in the table so that 'register.pl' would print on 2 or more pages, the summary will be duplicated that same number of time.
>
>If you are lazy like me and don't want to make that many transaction you can also add some content to the page like this:
>
>for (let i = 0; i < 100; i++) {
>  $('#register').append('<h1>TEST</h1>');
>}
>
>4. APPLY PATCH, clear browser cahche
>5. Try printing again, you should only get one cashup summary when printing.
>
>Signed-off-by: Juliet Heltibridle <jheltibridle@rcplib.org>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> .../intranet-tmpl/prog/js/cashup_modal.js     | 42 ++++++++++++++++++-
> 1 file changed, 41 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/cashup_modal.js b/koha-tmpl/intranet-tmpl/prog/js/cashup_modal.js
>index ad7bd6aa83b..dceca2f07bf 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/cashup_modal.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/cashup_modal.js
>@@ -55,6 +55,46 @@ $(document).ready(function() {
>     });
> 
>     $('.printModal').click(function() {
>-        window.print();
>+
>+        let contents = $('#cashupSummaryModal .modal-body').html();
>+        let win = window.open('','');
>+        win.document.write(`
>+            <style>
>+                table {
>+                    background-color: #FFFFFF;
>+                    border-bottom: 1px solid #CCCCCC;
>+                    border-collapse: collapse;
>+                    border-left: 1px solid #CCCCCC;
>+                    margin: 3px 0 5px 0;
>+                    padding: 0;
>+                    width: 99%;
>+                }
>+
>+                td {
>+                    background-color: #FFF;
>+                    border-bottom: 1px solid #CCCCCC;
>+                    border-left: 0;
>+                    border-right: 1px solid #CCCCCC;
>+                    border-top: 0;
>+                    font-size: 12px;
>+                    padding: 5px 5px 5px 5px;
>+                }
>+
>+                th {
>+                    background-color: #E9E9E9;
>+                    border-bottom: 1px solid #CCCCCC;
>+                    border-left: 0;
>+                    border-right: 1px solid #CCCCCC;
>+                    border-top: 0;
>+                    font-size: 14px;
>+                    font-weight: bold;
>+                    padding: 5px 5px 5px 5px;
>+                    text-align: left;
>+                }
>+            </style>
>+        `)
>+        win.document.write( contents );
>+        win.print();
>+        win.close();
>     });
> });
>-- 
>2.41.0

Actions: View | Diff | Splinter Review

Attachments on bug 31041: 158283 | 158342 | 158589 | 158590 | 158591 | 158592 | 158636 | 158644 | 158645 | 158646 | 158647