From: Dobrica Pavlinusic <dpavlin@rot13.org>
Date: Sun, 6 Jun 2010 16:32:24 +0000 (+0200)
Subject: warn about MAX_FACETS but include rest of checked
X-Git-Url: http://git.rot13.org/?a=commitdiff_plain;ds=sidebyside;h=0ddef374bba89a3dfec7677cf732fa909dae4b46;hp=cfdaa5d6c8e961e2b82d58cb0f529a6b7eed1ff9;p=MojoFacets.git

warn about MAX_FACETS but include rest of checked

This allows us to keep selected values on next submit
---

diff --git a/templates/data/facet.html.ep b/templates/data/facet.html.ep
index a7583b8..bb0b020 100644
--- a/templates/data/facet.html.ep
+++ b/templates/data/facet.html.ep
@@ -26,12 +26,15 @@
 % my $max_facets = $ENV{MAX_FACETS} || 10000;
 % my $facet_nr  = 0;
 % foreach my $n ( @$facet_names ) {
-%  if ( $facet_nr++ > $max_facets ) {
+%  if ( $facet_nr++ == $max_facets ) {
 <div class=alert>
-To keep browser memory usage at sane level, only <%= $max_facets %> facets are loaded to browser.
-This can be changed in <a href="/config">config</a>.
+First <a href="/config"><%= $max_facets %></a> facets displayed in full.
+&middot;
+Below are just checked values from rest of <%= $facet_count - $max_facets %> items.
 </div>
-%    last;
+%  }
+%  if ( $facet_nr > $max_facets && ! $checked->{$n} ) {
+%    next;
 %  }
 <li>
 <label>