[Patches] [PATCH] Bug 7118: Remove 'any' from pull downs in acquisitions

koha-patchbot at kohaaloha.com koha-patchbot at kohaaloha.com
Fri Nov 4 19:55:40 NZDT 2011


From: Katrin Fischer <Katrin.Fischer.83 at web.de>
Date: Mon, 31 Oct 2011 01:40:26 +0100
Subject: [PATCH] Bug 7118: Remove 'any' from pull downs in acquisitions

The filters on the suggestion page show 'Any' and an empty entry for the
filters 'Suggested by', 'Accepted by' and 'Managed by'.
Both do the same. It should only be 'Any' or an empty entry, but not both.

To test:
- check 'Any' is gone, but empty entry still shows
- check you can still unset filters 'managed by', 'suggested by' and 'accepted by'
---
 .../prog/en/modules/suggestion/suggestion.tt       |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
index 47ee96a..dfaad78 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
@@ -347,7 +347,7 @@ $(document).ready(function() { calcNewsuggTotal(); });
                         [% IF ( statusselected_REJECTED ) %]<option value="REJECTED" selected="selected">Rejected</option>
                         [% ELSE %]<option value="REJECTED">Rejected</option>[% END %]
                       </select></li>
-                    <li><label for="suggestedby"> Suggested By:</label><select id="suggestedby" name="suggestedby"><option value="">Any</option>
+                    <li><label for="suggestedby"> Suggested By:</label><select id="suggestedby" name="suggestedby">
 [% FOREACH suggestedby_loo IN suggestedby_loop %][% IF ( suggestedby_loo.selected ) %]<option value="[% suggestedby_loo.code %]" selected="selected">[% suggestedby_loo.desc %]</option>[% ELSE %]<option value="[% suggestedby_loo.code %]">[% suggestedby_loo.desc %]</option>[% END %][% END %]
                                                                      </select></li>
                     <li><label for="suggesteddate_from"> Suggested on:</label>[% suggesteddate %]
@@ -378,7 +378,7 @@ $(document).ready(function() { calcNewsuggTotal(); });
                         </tr>
                         </table>
                     </li>
-                    <li><label for="managedby"> Managed by:</label><select id="managedby" name="managedby"><option value="">Any</option>
+                    <li><label for="managedby"> Managed by:</label><select id="managedby" name="managedby">
 [% FOREACH managedby_loo IN managedby_loop %][% IF ( managedby_loo.selected ) %]<option value="[% managedby_loo.code %]" selected="selected">[% managedby_loo.desc %]</option>[% ELSE %]<option value="[% managedby_loo.code %]">[% managedby_loo.desc %]</option>[% END %][% END %]
                                                                      </select></li>
                     <li><label for="manageddate_from"> Management date:</label>[% manageddate %]
@@ -409,7 +409,7 @@ $(document).ready(function() { calcNewsuggTotal(); });
                         </tr>
                         </table>
                     </li>
-                    <li><label for="acceptedby"> Accepted by:</label><select id="acceptedby" name="acceptedby"><option value="">Any</option>
+                    <li><label for="acceptedby"> Accepted by:</label><select id="acceptedby" name="acceptedby">
 [% FOREACH acceptedby_loo IN acceptedby_loop %][% IF ( acceptedby_loo.selected ) %] <option value="[% acceptedby_loo.code %]" selected="selected">[% acceptedby_loo.desc %]</option>[% ELSE %]<option value="[% acceptedby_loo.code %]">[% acceptedby_loo.desc %]</option>[% END %][% END %]
 		                                      </select></li>
                     <li><label for="accepteddate_from"> Accepted on:</label>[% accepteddate %]
-- 
1.7.5.4


More information about the Patches mailing list