Statistics wizard for suggestions
Created attachment 34124 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page
Created attachment 34125 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests)
Test plan: 1/ Go to Reports > Statistics wizard > Suggestions 2/ Play with it and check if result is consistent with what you have in your database. All fields used in this wizard correspond to a column in the suggestions table, except for categorycode, sort1 and sort2, which correspond to the borrowers fields of the user who made the suggestion (suggestedby).
Created attachment 34135 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests)
Created attachment 34136 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests)
All fields used in this wizard correspond to a column in the suggestions table : it's ok.
Patch tested with a sandbox, by Beroud <isabelle.beroud@univ-lyon3.fr>
Created attachment 34328 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Beroud <isabelle.beroud@univ-lyon3.fr>
Created attachment 34329 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Beroud <isabelle.beroud@univ-lyon3.fr>
Hi Julian, I feel like this would have been a bit better split up into 2 separate bugs/features. Tried a few things and have some first feedback from code review and tests: Bug 13392: Add suggestions.ordereddate and suggestions.orderedby 1) I am not sure about adding ordereddate and orderedby as those seem duplicating the information we can get from aqorders.entrydate and aqbasket.authorisedby. Bug 13392: New statistics assistant for suggestions 1) The new tests don't give any information when run, please add a description. 2) Using the Koha:I18N is an option now, but I think we haven't created the po file that is needed for it yet. Those strings appear already multiple times in the templates, so the translators have to make sure to translate them the same as everywhere else. I feel it might be easier if we moved them to the template. But more important, the custom suggestion statuses are not supported by the new wizard (authorized value SUGGEST_STATUS) 3) For MARC21 the publicationyear is saved in the copyrightdate column in the database. Please change so that copyrightdate is used in the SQL query for MARC21 databases. 4) The bsort1 and bsort2 fields can be freetext if there are no entries in the authorised value categories - maybe show an input field in that case? 5) Please rename branch to library on the suggestion form. And maybe just "Reason" instead of "Reject reason"? The reasons can also be positive. In testing this I had difficulties with getting a suggestion into the ORDERED status with and without the patches - can you check if that works for you? The new database fields stayed empty too :(
> 1) I am not sure about adding ordereddate and orderedby as those seem > duplicating the information we can get from aqorders.entrydate and > aqbasket.authorisedby. It seems there is no direct link between a suggestion and an order line. Are you suggesting we should create this link ? > 3) For MARC21 the publicationyear is saved in the copyrightdate column > in the database. Please change so that copyrightdate is used in the > SQL query for MARC21 databases. It seems in fact that suggestions.publicationyear is not used at all (my own database only contains NULL or 0 in this column whereas copyrightdate contains the actual data) > In testing this I had difficulties with getting a suggestion into the > ORDERED status with and without the patches - can you check if that > works for you? The new database fields stayed empty too :( Yes, it worked for me, but I will re-check.
Created attachment 35835 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Rebased on master
Created attachment 35836 [details] [review] Bug 13392: New statistics assistant for suggestions Rebased on master
Created attachment 35837 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason'
> > In testing this I had difficulties with getting a suggestion into > > the ORDERED status with and without the patches - can you check if > > that works for you? The new database fields stayed empty too :( > > Yes, it worked for me, but I will re-check. Apparently it works only when the suggestion is not linked to an existing record (ie. the record is created while creating the order line). I don't know if this is a bug or not. And of course you need to run updatedatabase and misc/devel/update_dbix_class_files.pl (I always forget the latter)
Hi Julian, sorry, I don't understand the last comment - can you give an example/step by step for both cases maybe? Thx!
Here's a simple step by step example that worked for me: 1/ Create a new suggestion 2/ Mark it as ACCEPTED 3/ Create a new order from this suggestion After these steps a new biblio record should be created, the suggestion should be linked to it and its status should be ORDERED. In the basket view, you should see a link to the suggestion. But before that make sure you've run the updatedatabase script and the script in misc/devel/update_dbix_class_files.pl My remark was about if we create a suggestion from a suggestion that is already linked to a biblionumber (maybe the suggestion was already ORDERED and someone reset status to ACCEPTED), then the suggestion status is not updated.
Created attachment 36188 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page
Created attachment 36189 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests)
Created attachment 36190 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason'
Last patches fixed an easy conflict on updatedatabase.pl
Patch tested with a sandbox, by Nicole Engard <nengard@bywatersolutions.com>
Created attachment 36343 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 36344 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 36345 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
The suggest date wizard works good. But it seems that Koha reports some suggestions that were rejected, even if these suggestions do not appear in the staff interface. Some rejected suggestions that were refused for a lack of budget in the pro interface do not appear later in the wizard report. Similar issue of inconsistency of results for pending suggestions : 9 reported in the wizard while 19 appear in Koha pro interface. Unfortunately, we miss a running phpmyadmin interface to check the real content of the database.
Hi Courret, you could create a sql report with the following statement: select * from suggestions; This will show you everything that is in your suggestions table.
Hi Julian, working on this again: 1) The QA script outputs some warnings, could you please take a look? I can also see those in my logs on accessing the new page: Processing files before patches |========================>| 13 / 13 (100.00%) Processing files after patches Argument "" isn't numeric in numeric gt (>) at /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt line 52. Argument "" isn't numeric in numeric gt (>) at /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt line 52. Argument "" isn't numeric in numeric gt (>) at /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt line 52. Argument "" isn't numeric in numeric gt (>) at /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt line 52. Argument "" isn't numeric in numeric gt (>) at /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt line 52. Argument "" isn't numeric in numeric gt (>) at /home/katrin/kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt line 52. |========================>| 13 / 13 (100.00%) 2) Some strings need to be changed from 'Acquisitions' to 'Suggestions' - like the table legend on the form and the heading on the result page. 3) We need some POD for the new GetBudgetDescendantsIds. 4) I was worried about some of the TT constructs translatability and ran some tests, but looking good so far :)
Created attachment 40305 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Patch updated: Use atomic update
Created attachment 40306 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 40307 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 40308 [details] [review] [Follow-up] Bug 13392 - Fix QA failures: - Fix a perl warning in suggestions_stats.tt, - Replace term "Acquisitions" with "Suggestions", - Add POD for GetBudgetDescendantsIds.
I rebased patches on master / updated them to use atomic update and fix Katrin' QA comments.
Sorry, does not apply CONFLICT (content): Merge conflict in t/db_dependent/Budgets.t Auto-merging C4/Budgets.pm Failed to merge in the changes. Patch failed at 0001 Bug 13392: New statistics assistant for suggestions
Created attachment 48291 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Patch updated: Use atomic update
Created attachment 48292 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 48293 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 48294 [details] [review] Bug 13392 - Fix QA failures: - Fix a perl warning in suggestions_stats.tt, - Replace term "Acquisitions" with "Suggestions", - Add POD for GetBudgetDescendantsIds.
Created attachment 49088 [details] [review] Bug 13392 - Replace using of C4::Dates and C4::ItemType by Koha::DateUtils and Koha::ItemTypes
Created attachment 51995 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Patch updated: Use atomic update
Created attachment 51996 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 51997 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 51998 [details] [review] Bug 13392 - Fix QA failures: - Fix a perl warning in suggestions_stats.tt, - Replace term "Acquisitions" with "Suggestions", - Add POD for GetBudgetDescendantsIds.
Created attachment 51999 [details] [review] Bug 13392 - Replace using of C4::Dates and C4::ItemType by Koha::DateUtils and Koha::ItemTypes
Patches rebased on master
This patch doesn't apply, I get this message : Can't locate C4/Branch.pm in @INC (@INC contains: /home/sandbox1/src /etc/perl /usr/local/lib/perl/5.14.2 /usr/local/share/perl/5.14.2 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.14 /usr/share/perl/5.14 /usr/local/lib/site_perl .) at /home/sandbox1/src/reports/suggestions_stats.pl line 27. BEGIN failed--compilation aborted at /home/sandbox1/src/reports/suggestions_stats.pl line 27.
Created attachment 56110 [details] [review] Bug 13392: Remove use of C4::Branch and C4::Category + fix a 'ambiguous column name' error when using borrowers table
Created attachment 56889 [details] [review] Bug 13392 - Follow-up on terminology Standardizing a few terms - Patron sort 1/2 to match the holds statistics wizard - Patron category instead of borrower category
Hi Julian, I wrote a tiny follow up changing some of the terms used - can you please check? I concentrated on the changes/new columns in suggestions to see if this works alright. While testing I found that when ordering from a suggestion the status did no longer switch to ORDERED and the suggestion remains as ACCEPTED. Please fix!
Created attachment 60956 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Patch updated: Use atomic update
Created attachment 60957 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 60958 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Created attachment 60959 [details] [review] Bug 13392 - Fix QA failures: - Fix a perl warning in suggestions_stats.tt, - Replace term "Acquisitions" with "Suggestions", - Add POD for GetBudgetDescendantsIds.
Created attachment 60960 [details] [review] Bug 13392 - Replace using of C4::Dates and C4::ItemType by Koha::DateUtils and Koha::ItemTypes
Created attachment 60961 [details] [review] Bug 13392: Remove use of C4::Branch and C4::Category + fix a 'ambiguous column name' error when using borrowers table
Created attachment 60962 [details] [review] Bug 13392 - Follow-up on terminology Standardizing a few terms - Patron sort 1/2 to match the holds statistics wizard - Patron category instead of borrower category
Katrin, sorry but I wasn't able to reproduce your problem. When I create a new order line from a suggestion, the suggestion status is always changed to ORDERED. Could you describe step by step what you do ?
I've tried testing this by applying the patches, running the database update, regenerating the Schema. Every time I try to order from my accepted suggestion, an internal server error is shown (running under Plack with kohadevbox). Without Plack the error shown is: Template process failed: undef error - The method selected is not covered by tests! at /home/vagrant/kohaclone/C4/Templates.pm line 121. This prevents me from testing the problem I had earlier :( Filed bug 18525 as I also see this without the patches. But would be good if someone could confirm.
Created attachment 64459 [details] [review] [SIGNED-OFF] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Patch updated: Use atomic update Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 64460 [details] [review] [SIGNED-OFF] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 64461 [details] [review] [SIGNED-OFF] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 64462 [details] [review] [SIGNED-OFF] Bug 13392 - Fix QA failures: - Fix a perl warning in suggestions_stats.tt, - Replace term "Acquisitions" with "Suggestions", - Add POD for GetBudgetDescendantsIds. Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 64463 [details] [review] [SIGNED-OFF] Bug 13392 - Replace using of C4::Dates and C4::ItemType by Koha::DateUtils and Koha::ItemTypes Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 64464 [details] [review] [SIGNED-OFF] Bug 13392: Remove use of C4::Branch and C4::Category + fix a 'ambiguous column name' error when using borrowers table Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 64465 [details] [review] [SIGNED-OFF] Bug 13392 - Follow-up on terminology Standardizing a few terms - Patron sort 1/2 to match the holds statistics wizard - Patron category instead of borrower category Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
(In reply to Katrin Fischer from comment #59) > I've tried testing this by applying the patches, running the database > update, regenerating the Schema. Every time I try to order from my accepted > suggestion, an internal server error is shown (running under Plack with > kohadevbox). > Without Plack the error shown is: > Template process failed: undef error - The method selected is not covered by > tests! at /home/vagrant/kohaclone/C4/Templates.pm line 121. > > This prevents me from testing the problem I had earlier :( Filed bug 18525 > as I also see this without the patches. But would be good if someone could > confirm. I can't confirm it, it does work for me...
The blocking bug was fixed since, but I didn't get back to this - thanks for testing Josef!
Hi Julian, Minor conflict in tests (wrong number) Missing comma in kohastructure.sql between constraints I tried adding a suggestion, ordering it, but never saw the suggestion status update to 'ORDERED' - nothing relevant in error log Reporting seems to work, however, if I don't select both a row and column the page simply reloads and clears data, maybe a warning could be added? Can you provide a clear test plan and verify things are working for you? -Nick
Created attachment 78622 [details] [review] Bug 13392: Add suggestions.ordereddate and suggestions.orderedby Those fields are updated when suggestion status is changed to 'ORDERED' and are displayed on suggestion/suggestion.pl page Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Patch updated: Use atomic update Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78623 [details] [review] Bug 13392: New statistics assistant for suggestions New sub C4::Budgets::GetBudgetDescendantsIds (with unit tests) Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78624 [details] [review] Bug 13392: QA fixes 1/ Add tests descriptions 2/ Do not use Koha::I18N for suggestion statuses, and add statuses from SUGGEST_STATUS authorised values 3/ Use copyrightdate instead of publicationyear which seems to be unused 4/ Display free text filters if there is no values in corresponding authorised values list 5/ Rename 'Branch' to 'Library' and 'Reject reason' to 'Reason' Signed-off-by: Nicole Engard <nengard@bywatersolutions.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78625 [details] [review] Bug 13392 - Fix QA failures: - Fix a perl warning in suggestions_stats.tt, - Replace term "Acquisitions" with "Suggestions", - Add POD for GetBudgetDescendantsIds. Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78626 [details] [review] Bug 13392 - Replace using of C4::Dates and C4::ItemType by Koha::DateUtils and Koha::ItemTypes Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78627 [details] [review] Bug 13392: Remove use of C4::Branch and C4::Category + fix a 'ambiguous column name' error when using borrowers table Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78628 [details] [review] Bug 13392: Follow-up on terminology Standardizing a few terms - Patron sort 1/2 to match the holds statistics wizard - Patron category instead of borrower category Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 78629 [details] [review] Bug 13392: Select a row and a column by default It prevents users to submit the form without having selected anything
(In reply to Nick Clemens from comment #69) > Hi Julian, > > Minor conflict in tests (wrong number) > Missing comma in kohastructure.sql between constraints Tests should pass now, and the syntax error in kohastructure.sql is fixed > > I tried adding a suggestion, ordering it, but never saw the suggestion > status update to 'ORDERED' - nothing relevant in error log It worked for me. Have you run updatedatabase + update_dbix_class_files ? > > Reporting seems to work, however, if I don't select both a row and column > the page simply reloads and clears data, maybe a warning could be added? The last patch fixes this. I did not add a warning but this scenario should not happen anymore > > Can you provide a clear test plan and verify things are working for you? The only test plan I can come up with is: 1. Create a lot of suggestions from different libraries and with different status 2. Try all the buttons on the form and see what happens :) Request directly mysql to compare results if needed
Please run the QA script (make sure to update it first) and fix all errors: FAIL koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats.tt FAIL filters missing_filter at line 12 ( <td>[% filter %]</td>) missing_filter at line 51 ( <input type="text" name="filter_[% name %]">) missing_filter at line 18 ( <select name="groupby_[% name %]">) missing_filter at line 10 ( <td><input type="radio" name="column" value="[% name %]" [% IF checked == "column" %]checked[% END %]/></td>) missing_filter at line 157 ( [% budget.budget_name %]) missing_filter at line 33 ( <input type="text" size="10" id="filter_[% name %]_to") missing_filter at line 155 ( <option value="[% budget.budget_id %]">) missing_filter at line 47 ( <option value="[% value %]">[% display_values.$name.$value %]</option>) missing_filter at line 34 ( name="filter_[% name %]_to"/>) missing_filter at line 9 ( <td><input type="radio" name="line" value="[% name %]" [% IF checked == "line" %]checked[% END %] /></td>) missing_filter at line 44 ( <select name="filter_[% name %]">) missing_filter at line 29 ( <input type="text" size="10" id="filter_[% name %]_from") missing_filter at line 11 ( <td>[% groupby %]</td>) missing_filter at line 30 ( name="filter_[% name %]_from"/>) FAIL tt_valid lines 9, 10 FAIL koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt FAIL filters missing_filter at line 193 ( <td>[% IF ( orderedby_borrowernumber ) %]<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% orderedby_borrowernumber %]">[% orderedby_surname %], [% orderedby_firstname %]</a> [% Branches.GetName( orderedby_branchcode ) %] ([% orderedby_description %])[% END %]</td>) FAIL koha-tmpl/intranet-tmpl/prog/en/modules/reports/suggestions_stats_results.tt FAIL filters missing_filter at line 36 ( [% display_values.$filter.$value || value %]) missing_filter at line 74 ( <th>[% total %]</th>) missing_filter at line 66 ( <td>[% totalrow %]</td>) missing_filter at line 59 ( <td>[% value %]</td>) missing_filter at line 72 ( <th>[% totalcols.$col %]</th>) missing_filter at line 56 ( <td>[% display_values.$line.$row || row %]</td>) missing_filter at line 48 ( <th>[% display_values.$column.$col || col %]</th>) * Commit title does not contain 'follow-up' correctly spelt - 9e5d85c * Commit title does not start with 'Bug XXXXX: ' - 33c4ee9 * Commit title does not start with 'Bug XXXXX: ' - 4165976