[Koha-bugs] [Bug 6934] New report Cash Register Statistics

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6934

--- Comment #39 from Rémi Mayrand-Provencher <remi.mayrand-provencher at inLibro.com> ---
Created attachment 48634
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=48634&action=edit
Bug 6934 - QA Follow-up

Tried to address every single thing that QA mentioned :
1)Removed isNull function
2)Changed datepicker properly
3)We have to fill in two dates not to be flooded with old entries if we don't
want to.
4)The first two fieldsets are now in one bigger field set
5)Removed the use of $fullreportname variable
6)Removed the use of dt_from_string when it already recieved a DT.
7)Removed comments starting with "###" in cash_register_stats.pl
8)Used Text::CSV::Encoded to generate the CSV file (used guided_reports.pl as
an example)
9)I am not sure if using C4::Reports::Guided is the right choice here. I
instead used the same column names as the one recieved by the query in
cash_register_stats because this report uses columns from 5 different tables in
addition of using both firstname and surname twice.

For the ninth point, I would like to know more about C4::Reports::Guided if you
still think using it would be a better solution. Thank you!

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the QA Contact for the bug.