The patron statistics report is missing the page section on the results table.
Created attachment 161021 [details] [review] Bug 33452: Reindent borrowers_stats.tt This patch only contains whitespace changes, including switching from tabs to 4 spaces for indenting consistently. To test: * There shoudl be no change in appearance and behavior
Created attachment 161022 [details] [review] Bug 33452: Update patron statistics search form and result list This makes changes to terminology, spelling and capitalization. Terminology used should be the same as on the normal patron form. To test: * Go to Reports > Patrons statistics * Compare search form before and after patch. List of changes: * Gender: * Add option 'Other' * Change men and women to 'Male' and 'Female' * Sort*: * Labels changed from "Sortx" to "Sort x" * Note: to see Sort 1 you need to add an AV for bsort1 * Patron status: * Options changed to start with a capital letter * 'Lost' changed to 'Lost card' * Patron activity: * Options changed to start with a capital letter * 'no active' changed to 'Not active' * Patron attributes: * Switched code and description in label, so description is first * Do a search, compare result table before and after patch. * Added white background (page-section) to results
Created attachment 161023 [details] [review] Bug 33452: Improve display of applied filters and results table This makes various improvements on the display of the results: * Filters didn't display the value that was filtered on * Filter labels were not translatable * Results table showed line/column using untranslatable codes * Results table showed library and patron category codes instead of descriptions To test: * Use the various filters in the form * Use various combinations of row/column * Verify it all works as expected and display is much improved Note: * In the filters, the code for extended patron attributs will be used as label. * In the results table only patron category codes and library codes translate to descriptions. Other values from pull downs haven't been implemented yet.
I tested this out and the changes to the form labels and the display of the table look good to me. It's especially nice that the filters display accurately above the results table. While I was testing and looking at the code I noticed another small improvement could be made to the results table by right aligning the <td> elements that have numerical data. It's easily done by adding class="text-right" in a few places. Question about Bugzilla testing and submitting patches: would it be ok to add an attachment to this bug with the proposed alignment change? or should I just sign off on this bug and file a new bug for the numerical alignment and attach that change separately? Thanks!
Hi Brendan, thanks for testing! As this already includes a lot of different changes, I think it would make it a little easier to put any additional ones into a separate bug report that is dependent on this one.
Created attachment 161471 [details] [review] Bug 33452: Reindent borrowers_stats.tt This patch only contains whitespace changes, including switching from tabs to 4 spaces for indenting consistently. To test: * There shoudl be no change in appearance and behavior Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Created attachment 161472 [details] [review] Bug 33452: Update patron statistics search form and result list This makes changes to terminology, spelling and capitalization. Terminology used should be the same as on the normal patron form. To test: * Go to Reports > Patrons statistics * Compare search form before and after patch. List of changes: * Gender: * Add option 'Other' * Change men and women to 'Male' and 'Female' * Sort*: * Labels changed from "Sortx" to "Sort x" * Note: to see Sort 1 you need to add an AV for bsort1 * Patron status: * Options changed to start with a capital letter * 'Lost' changed to 'Lost card' * Patron activity: * Options changed to start with a capital letter * 'no active' changed to 'Not active' * Patron attributes: * Switched code and description in label, so description is first * Do a search, compare result table before and after patch. * Added white background (page-section) to results Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Created attachment 161473 [details] [review] Bug 33452: Improve display of applied filters and results table This makes various improvements on the display of the results: * Filters didn't display the value that was filtered on * Filter labels were not translatable * Results table showed line/column using untranslatable codes * Results table showed library and patron category codes instead of descriptions To test: * Use the various filters in the form * Use various combinations of row/column * Verify it all works as expected and display is much improved Note: * In the filters, the code for extended patron attributs will be used as label. * In the results table only patron category codes and library codes translate to descriptions. Other values from pull downs haven't been implemented yet. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>