Add print receipt option to Koha Self-check. It should: * Bring up alert box upon hitting 'finish' ( like transfer/hold alert in CKI module) * "Would you like a receipt?" - options 'Yes' / 'No' * Print receipt if 'Yes' --- else, no receipt.
Created attachment 10648 [details] [review] Adds a print receipt to Self Checkout To Test: Log in to self checkout Enter a barcode and click submit You will be asked if you would like a receipt If you click ok you should be taken to the print receipt page and the item should check out if you click cancel the item should be checked out
Created attachment 10649 [details] [review] This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and your item should check out.
Hi Elliott This looks good, but a few little things that need to be fixed in opac/sco/printslip.pl 1/ Copyright headers (it says biblibre and katipo at the moment) 2/ use warnings is commented out, we shouldn't be making new scripts with warnings commented out 3/ The pod is still referring to moremember.pl which is where the script was copied from I guess 4/ $template->param( is a HTML::Template thing, they still exist in old scripts and we have a shim in C4::Templates to fix it, but its best to make no ones like this $template->{VARS}->{'slip'} = $slip;
Created attachment 10677 [details] [review] This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and your item should check out.
Chris, I made the changes you requested. Let me know if it looks ok. The reason I left the copyright headers was because it was basically a direct copy and I didn't want to take credit for someone else's work.
(In reply to comment #5) > Chris, > > I made the changes you requested. Let me know if it looks ok. The reason I > left the copyright headers was because it was basically a direct copy and I > didn't want to take credit for someone else's work. Yep, you could add parts copyright Bywater Solutions in that case. Also now we have no copyright headers :( And no POD. A follow up fixing those would be great. But I'll sign off on it because functionality wise it works, but you will probably need that follow up to get passed QA. (The POD is handy for this http://perldoc.koha-community.org/members/patronimage.html) http://perldoc.koha-community.org/ is a great resource for developing and it gets better the better we document our code. But that will be up to QA not me :)
Created attachment 10678 [details] [review] Bug 8033 : This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and your item should check out. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
QA comments * new script passes perlcritic * Eliott, if you don't want to copyright your code to bywatersolutions, please copyright it to HLT * your new script sco/printslip.pl seems not to be perltidy-ed (see PERL1 coding guidelines http://wiki.koha-community.org/wiki/Coding_Guidelines) * I tested, got the message, but the printed paper was empty. Filling ISSUEQSLIP notice make things looking better ! failed QA for now, when you've fixed 2 and 3, switch back to signed off, as the QA comments are just coding style related
Created attachment 11112 [details] [review] [Followup] Modified POD and copyright. Also perltidied
I find that submitting a barcode to check out results in a "Would you like to print a receipt" dialog. Shouldn't it only offer to print when you click "Finish?"
Owen, The reason I added it to the submit button rather than the finish button was because I feared someone would not click finish and just leave the terminal. If you think that this is an improbable case it is a pretty easy change. (In reply to comment #10) > I find that submitting a barcode to check out results in a "Would you like > to print a receipt" dialog. Shouldn't it only offer to print when you click > "Finish?"
(In reply to comment #11) > The reason I added it to the submit button rather than the finish button was > because I feared someone would not click finish and just leave the terminal. This was a potential problem before this patch, wasn't it? I think it's too much of a disruption to the process to pop up a message after each transaction. I think it would be better to leave it up to the user to decide when to print their receipt. Perhaps another button alongside "finish," "Finish and print receipt?"
(In reply to comment #12) > (In reply to comment #11) > > The reason I added it to the submit button rather than the finish button was > > because I feared someone would not click finish and just leave the terminal. > > This was a potential problem before this patch, wasn't it? I think it's too > much of a disruption to the process to pop up a message after each > transaction. I think it would be better to leave it up to the user to decide > when to print their receipt. Perhaps another button alongside "finish," > "Finish and print receipt?" I agree with Owen. If someone skips the receipt printing step, that's their problem.
(In reply to comment #13) > (In reply to comment #12) > > (In reply to comment #11) > > > The reason I added it to the submit button rather than the finish button was > > > because I feared someone would not click finish and just leave the terminal. > > > > This was a potential problem before this patch, wasn't it? I think it's too > > much of a disruption to the process to pop up a message after each > > transaction. I think it would be better to leave it up to the user to decide > > when to print their receipt. Perhaps another button alongside "finish," > > "Finish and print receipt?" > > I agree with Owen. If someone skips the receipt printing step, that's their > problem. Additionally, you're skipping the server-side validation when you have your script on the submit button. You generate a slip even though the item might not actually be checked out (because it is reference book, currently checked out to someone else, on hold for someone else, a renewal, and so on). While this isn't a crippling deficiency, it is certainly problematic. I also wanted to mention that I agree with Owen as well. I use the self-checkout at my local branch all the time, and you print the receipt by pressing the "Print Receipt" button on the screen. It's a much smoother behaviour. (I've also noticed recently that there is an "Email Receipt" button as an alternative to the paper receipt.)
Not to hassle you too much, but it's also worth noting that renewals aren't included in the print out. Just check outs. I haven't looked too much into the guts of this script, but they should probably be included too.
Created attachment 12316 [details] [review] Bug 8033 : This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. Click the finish button You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Added copyright to print slip for SCO Modified POD and copyright. Also perltidied http://bugs.koha-community.org/show_bug.cgi?id=8033 updated print slip option to show on the click of the finish button instead of the submit button
This seems to work fine, but I would feel more comfortable signing off if the "Finish" action wasn't so dependent on JavaScript. Instead of using a click handler on the submit button, I suggest using a submit handler: $("#logout_form").submit(function(){ if(confirm("Would you like to print a receipt?")){ window.open("/cgi-bin/koha/sco/printslip.pl?borrowernumber=[% borrowernumber %]&print=qslip"); } return true; }); You don't have to rewrite the "action" of the form because whether the user wants to print a receipt or not they want to log out. Keep the default action of the form and pop up the window if the user has JavaScript on and chooses to confirm.
Created attachment 13559 [details] [review] [Follow-up] Change javascipt behavior to degrade nicely
Please apply both the original patch and the follow up. The testing plan is the same as before.
Created attachment 13716 [details] [review] Change javascipt behavior to degrade nicely http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Mason James <mtj@kohaaloha.com>
(In reply to comment #20) > Created attachment 13716 [details] [review] > Change javascipt behavior to degrade nicely > > http://bugs.koha-community.org/show_bug.cgi?id=8033 > Signed-off-by: Mason James <mtj@kohaaloha.com> with jscript disabled, 'finish' button works as expected ... but printing pop-up is disabled Elliot, i assume this is the expected behaviour?
(In reply to comment #21) > with jscript disabled, 'finish' button works as expected > ... but printing pop-up is disabled This is to be expected since the trigger for the pop-up is JavaScript. If we wanted to be sure we offered a print view to non-JS users we might add a separate button and hide it with JavaScript.
Owen is right, it is expected behavior as the pop-up is javascript. Is adding a seperate button something that needs to be done before this patch sees some QA love? Elliott
(In reply to comment #23) > Is adding a seperate button something that needs to be done before this patch > sees some QA love? I would think it could be left for a follow-up.
QA comment: * passes koha-qa.pl although there are a few tabs I've replaced by 4 spaces in a follow-up * work as expected/announced, no side effect risk I can see * there are some missing images with ccsr theme, but this problem is not related to this patch * Idea for an improvement = make sco slip printing compatible with silent printing (see bug 7563) passed QA
Created attachment 14024 [details] [review] Bug 8033 : This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. Click the finish button You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Added copyright to print slip for SCO Modified POD and copyright. Also perltidied http://bugs.koha-community.org/show_bug.cgi?id=8033 updated print slip option to show on the click of the finish button instead of the submit button
Created attachment 14025 [details] [review] Change javascipt behavior to degrade nicely http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Mason James <mtj@kohaaloha.com> Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Created attachment 14026 [details] [review] Bug 8033 follow-up removing tabs in indentation
I followed the test plan, and I am definitely not asked if I want to print out a receipt. Is there a syspref that needs to be set?
Marking as "Failed QA" until guidance is provided on how to test this.
Created attachment 14091 [details] [review] Bug 8033: Change javascipt behavior to degrade nicely
Created attachment 14094 [details] [review] Bug 8033 : This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. Click the finish button You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Added copyright to print slip for SCO Modified POD and copyright. Also perltidied http://bugs.koha-community.org/show_bug.cgi?id=8033 updated print slip option to show on the click of the finish button instead of the submit button
Created attachment 14095 [details] [review] Bug 8033 follow-up removing tabs in indentation
Created attachment 14096 [details] [review] Bug 8033: Change javascipt behavior to degrade nicely
Created attachment 14097 [details] [review] Bug 8033: Change javascipt behavior to degrade nicely Signed-off-by: Elliott Davis <elliott@test.bywatersolutions.com>
Created attachment 14098 [details] [review] Bug 8033 follow-up removing tabs in indentation Signed-off-by: Elliott Davis <elliott@test.bywatersolutions.com>
Created attachment 14099 [details] [review] Bug 8033 : This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. Click the finish button You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Added copyright to print slip for SCO Modified POD and copyright. Also perltidied http://bugs.koha-community.org/show_bug.cgi?id=8033 updated print slip option to show on the click of the finish button instead of the submit button Signed-off-by: Elliott Davis <elliott@test.bywatersolutions.com>
Created attachment 14100 [details] [review] Bug 8033 : This patch adds print slips to self checkout. To Test: Sign in to self checkout. Enter a barcode and click submit. Click the finish button You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. http://bugs.koha-community.org/show_bug.cgi?id=8033 Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Added copyright to print slip for SCO Modified POD and copyright. Also perltidied http://bugs.koha-community.org/show_bug.cgi?id=8033 updated print slip option to show on the click of the finish button instead of the submit button Signed-off-by: Elliott Davis <elliott@test.bywatersolutions.com>
Created attachment 14101 [details] [review] Bug 8033 follow-up removing tabs in indentation Signed-off-by: Elliott Davis <elliott@test.bywatersolutions.com>
Created attachment 14102 [details] [review] Bug 8033: Change javascipt behavior to degrade nicely Signed-off-by: Elliott Davis <elliott@test.bywatersolutions.com>
This patch has been pushed to master.
Created attachment 14134 [details] [review] Bug 8033 [Follow-up] add print receipt option to Koha self-check This follow-up makes some corrections to JavaScript, most importantly by enabling translation of strings embedded in the script. Other corrections: Trailing whitespace, proper <![CDATA[ commenting, and === comparisons (Following coding/JSHint guidelines). To test, check something out in self checkout and click the "Finish" button. The JavaScript confirmation dialog should appear and the receipt should appear and self-close correctly.
Bug 8033 : This patch adds print slips to self checkout. Apply? [yn] y Applying: Bug 8033 : This patch adds print slips to self checkout. error: koha-tmpl/opac-tmpl/prog/en/modules/sco/printslip.tt: already exists in index error: patch failed: koha-tmpl/opac-tmpl/prog/en/modules/sco/sco-main.tt:8 error: koha-tmpl/opac-tmpl/prog/en/modules/sco/sco-main.tt: patch does not apply error: opac/sco/printslip.pl: already exists in index error: patch failed: opac/sco/sco-main.pl:239 error: opac/sco/sco-main.pl: patch does not apply Patch failed at 0001 Bug 8033 : This patch adds print slips to self checkout. When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". Patch left in /tmp/Bug-8033--This-patch-adds-print-slips-to-self-chec-33SuJQ.patch nengard@kohavm:~/kohaclone$
Created attachment 14496 [details] [review] Bug 8033 [Follow-up] add print receipt option to Koha self-check This follow-up makes some corrections to JavaScript, most importantly by enabling translation of strings embedded in the script. Other corrections: Trailing whitespace, proper <![CDATA[ commenting, and === comparisons (Following coding/JSHint guidelines). To test, check something out in self checkout and click the "Finish" button. The JavaScript confirmation dialog should appear and the receipt should appear and self-close correctly.
Created attachment 14498 [details] [review] [SIGNED-OFF] Bug 8033 [Follow-up] add print receipt option to Koha self-check This follow-up makes some corrections to JavaScript, most importantly by enabling translation of strings embedded in the script. Other corrections: Trailing whitespace, proper <![CDATA[ commenting, and === comparisons (Following coding/JSHint guidelines). To test, check something out in self checkout and click the "Finish" button. The JavaScript confirmation dialog should appear and the receipt should appear and self-close correctly. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Works as expected.
Obsoleted all patches that appeared to have been pushed already - if I was wrong feel free to restore. Follow up signed off. Nicole
I cannot test the modifications on the sco/receipt.tt file. Where is it used?
Click 'Finish' on /cgi-bin/koha/sco/sco-main.pl and it will prompt to print. Nicole
(In reply to comment #48) > Click 'Finish' on /cgi-bin/koha/sco/sco-main.pl and it will prompt to print. > > Nicole According to the code this action uses the sco/printslip.tt file, not sco/receipt.tt
Could someone confirm this file (sco/receipt.tt) can be removed?
(In reply to comment #50) > Could someone confirm this file (sco/receipt.tt) can be removed? I've searched in my text editor and grepped the files within the sco directories in cgi-bin and htdocs, and I cannot see any reference to sco/receipt.tt
Also, it looks like there is an issue with the HTML <title> tags. Chrome and IE9 (I haven't checked out browsers) seem to add a <title i18n-content="title"> tag, which causes the opening <title> tag to be rendered as part of the innerHTML, so you wind up with "<title> Print Receipt for..." I don't know if the best solution is to remove the opening <title> tag or if others have other ideas, but it would be worth a quick follow-up...
(In reply to comment #51) > (In reply to comment #50) > > Could someone confirm this file (sco/receipt.tt) can be removed? > > I've searched in my text editor and grepped the files within the sco > directories in cgi-bin and htdocs, and I cannot see any reference to > sco/receipt.tt interesting... despite sco/receipt.tt being added to the repo in commit a0b220f8551 it looks like it's probably never been used/referred to? so, delete this file for sure :)
(In reply to comment #53) > (In reply to comment #51) > > (In reply to comment #50) > > > Could someone confirm this file (sco/receipt.tt) can be removed? > > > > I've searched in my text editor and grepped the files within the sco > > directories in cgi-bin and htdocs, and I cannot see any reference to > > sco/receipt.tt > > interesting... > > despite sco/receipt.tt being added to the repo in commit a0b220f8551 > it looks like it's probably never been used/referred to? > > so, delete this file for sure :) ah, i see now... the orig. file was opac/sco/receipt.tmpl and was used by opac/sco/printer.pl opac/sco/printer.pl was removed, but its tmpl/tt file was not the file can still be removed safely
Created attachment 17373 [details] [review] Bug 8033 [Follow-up] add print receipt option to Koha self-check This follow-up makes some corrections to JavaScript, most importantly by enabling translation of strings embedded in the script. Other corrections: Trailing whitespace, proper <![CDATA[ commenting, and === comparisons (Following coding/JSHint guidelines). To test, check something out in self checkout and click the "Finish" button. The JavaScript confirmation dialog should appear and the receipt should appear and self-close correctly. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Works as expected. Signed-off-by: Mason James <mtj@kohaaloha.com>
(In reply to comment #55) > Created attachment 17373 [details] [review] [review] > Bug 8033 [Follow-up] add print receipt option to Koha self-check > > This follow-up makes some corrections to JavaScript, most importantly > by enabling translation of strings embedded in the script. Other > corrections: Trailing whitespace, proper <![CDATA[ commenting, > and === comparisons (Following coding/JSHint guidelines). > > To test, check something out in self checkout and click the "Finish" > button. The JavaScript confirmation dialog should appear and the receipt > should appear and self-close correctly. > > Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> > Works as expected. > Signed-off-by: Mason James <mtj@kohaaloha.com> passing QA on this ...with a follow-up patch to remove sco/receipt.tt
Created attachment 17374 [details] [review] Bug 8033 - add print receipt option to Koha self-check (edit) 4 misc follow-up fixes 1/ fixed printslip.tmpl typo 2/ fixed <title i18n-content="title"> display bug 3/ moved hardcoded text string from script to template. 4/ delete unused receipt.tt file To Test: - Sign in to self checkout. - Enter a barcode and click submit. - Click the finish button - You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. fyi: you should no longer see a "<title> Print Receipt for..." line in chrome/IE
I was going to sign off on the follow up but I can't apply the original signed off patch anymore. Bug 8033 - add print receipt option to Koha self-check (edit) Apply? [yn] y Applying: Bug 8033 - add print receipt option to Koha self-check (edit) error: patch failed: koha-tmpl/opac-tmpl/prog/en/modules/sco/receipt.tt:1 error: koha-tmpl/opac-tmpl/prog/en/modules/sco/receipt.tt: patch does not apply Patch failed at 0001 Bug 8033 - add print receipt option to Koha self-check (edit) When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". Patch left in /tmp/Bug-8033---add-print-receipt-option-to-Koha-self-c-gJKXQ0.patch
Created attachment 18053 [details] [review] Bug 8033 [Follow-up] add print receipt option to Koha self-check This follow-up makes some corrections to JavaScript, most importantly by enabling translation of strings embedded in the script. Other corrections: Trailing whitespace, proper <![CDATA[ commenting, and === comparisons (Following coding/JSHint guidelines). To test, check something out in self checkout and click the "Finish" button. The JavaScript confirmation dialog should appear and the receipt should appear and self-close correctly. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Works as expected. Signed-off-by: Mason James <mtj@kohaaloha.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 18054 [details] [review] Bug 8033 - add print receipt option to Koha self-check (edit) 4 misc follow-up fixes 1/ fixed printslip.tmpl typo 2/ fixed <title i18n-content="title"> display bug 3/ moved hardcoded text string from script to template. 4/ delete unused receipt.tt file To Test: - Sign in to self checkout. - Enter a barcode and click submit. - Click the finish button - You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out. fyi: you should no longer see a "<title> Print Receipt for..." line in chrome/IE Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(In reply to comment #60) > 2/ fixed <title i18n-content="title"> display bug What is 'i18n-content="title"' supposed to do? This template should not have a beginning <title> tag. <title> is supplied by doc-head-open.inc.
I would like to voice my annoyance at this bug. We have no printer, no need or want for receipts. Why should our users be forced to get out the mouse/keyboard and hit no every time they log out? This should be an option in the preferences and not forced upon everyone.
(In reply to Derryn from comment #62) > I would like to voice my annoyance at this bug. > > We have no printer, no need or want for receipts. > Why should our users be forced to get out the mouse/keyboard and hit no > every time they log out? > This should be an option in the preferences and not forced upon everyone. Awesome, I look forward to your patch implementing this.
Created attachment 22301 [details] [review] Bug 8033 [Follow-up] add print receipt option to Koha self-check This follow-up makes some corrections to JavaScript, most importantly by enabling translation of strings embedded in the script. Other corrections: Trailing whitespace, proper <![CDATA[ commenting, and === comparisons (Following coding/JSHint guidelines). To test, check something out in self checkout and click the "Finish" button. The JavaScript confirmation dialog should appear and the receipt should appear and self-close correctly. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Works as expected. Signed-off-by: Mason James <mtj@kohaaloha.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 22302 [details] [review] Bug 8033 - add print receipt option to Koha self-check (edit) Reimplementation of Kyle's follow-up, minus the addition of "<title i18n-content="title">" which is I don't understand. From the original commit message: 3 misc follow-up fixes 1/ fixed printslip.tmpl typo 2/ moved hardcoded text string from script to template. 3/ delete unused receipt.tt file To Test: - Sign in to self checkout. - Enter a barcode and click submit. - Click the finish button - You should be prompted with a message asking if you would like a receipt. If you click OK you should be taken to the page with the receipt. If you click Cancel you should not see the reciept and you should be logged out.
Patch applied cleanly, go forth and signoff
Attachment 22301 [details]: Merge conflict in sco-main.tt
Created attachment 37489 [details] [review] Bug 8033 - Remove unused template file. Bugs 11349 and 12062 implements part 1 and 2 of the 3 things listed before. This patch removes the unused receipt.tt file. TEST PLAN --------- 1) Confirm the file isn't used. $ git grep "sco/receipt" | grep -v "\.po:" -- this should list nothing, because there are still po files referencing it, but they aren't needed either. 2) ls Reimplementation of Kyle's follow-up, minus the addition of "<titleunused receipt.tt file. TEST PLAN --------- 1) Confirm the file isn't used. $ git grep "sco/receipt" | grep -v "\.po:" -- this should list nothing, because there are still po files referencing it, but they aren't needed either. 2) ls ./koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/receipt.tt -- there should be a file 3) apply patch 4) repeat step 2 -- the file should be removed
Created attachment 37490 [details] [review] Bug 8033 - Remove unused template file. All prog related patches were obsoleted from this bug. Bug 11349 changed the code to have the proper template name used (sco/printslip.tt instead of sco/printsplit.tmpl). Bug 12062 moved the hard coded string logic into the template, and passed the borrower number. This only leaves the patch to remove the unused receipt.tt file. TEST PLAN --------- 1) Confirm the file isn't used. $ git grep "sco/receipt" | grep -v "\.po:" -- this should list nothing, because there are still po files referencing it, but they aren't needed either. 2) ls ./koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/receipt.tt -- there should be a file 3) apply patch 4) repeat step 2 -- the file should be removed
Created attachment 37497 [details] [review] [Signed-off] Bug 8033 - Remove unused template file. All prog related patches were obsoleted from this bug. Bug 11349 changed the code to have the proper template name used (sco/printslip.tt instead of sco/printsplit.tmpl). Bug 12062 moved the hard coded string logic into the template, and passed the borrower number. This only leaves the patch to remove the unused receipt.tt file. TEST PLAN --------- 1) Confirm the file isn't used. $ git grep "sco/receipt" | grep -v "\.po:" -- this should list nothing, because there are still po files referencing it, but they aren't needed either. 2) ls ./koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/receipt.tt -- there should be a file 3) apply patch 4) repeat step 2 -- the file should be removed Followed test plan, file does no longer exist. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 37651 [details] [review] [PASSED QA] Bug 8033 - Remove unused template file. All prog related patches were obsoleted from this bug. Bug 11349 changed the code to have the proper template name used (sco/printslip.tt instead of sco/printsplit.tmpl). Bug 12062 moved the hard coded string logic into the template, and passed the borrower number. This only leaves the patch to remove the unused receipt.tt file. TEST PLAN --------- 1) Confirm the file isn't used. $ git grep "sco/receipt" | grep -v "\.po:" -- this should list nothing, because there are still po files referencing it, but they aren't needed either. 2) ls ./koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/receipt.tt -- there should be a file 3) apply patch 4) repeat step 2 -- the file should be removed Followed test plan, file does no longer exist. Signed-off-by: Marc Veron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch pushed to master. Thanks Owen! We miss you!