From: Dobrica Pavlinusic Date: Sat, 19 Jun 2010 16:33:09 +0000 (+0200) Subject: cleanup all_filters name generation X-Git-Url: http://git.rot13.org/?p=MojoFacets.git;a=commitdiff_plain;h=d1053c20bb8140f501b401b14aa66d2db7c7ffad cleanup all_filters name generation --- diff --git a/lib/MojoFacets/Data.pm b/lib/MojoFacets/Data.pm index 20c8f1b..8a52a85 100644 --- a/lib/MojoFacets/Data.pm +++ b/lib/MojoFacets/Data.pm @@ -504,6 +504,11 @@ sub _data_sorted_by { } +sub __all_filters { + my $order = pop @_; + join(',', sort(@_), 'order', $order); +} + sub items { my $self = shift; @@ -547,7 +552,7 @@ sub items { } } - my $all_filters = join(' ',sort @filter_names,'order:',$order); + my $all_filters = __all_filters( @filter_names,$order ); # warn "# all_filters $all_filters ", dump( $loaded->{$path}->{filtered}->{$all_filters} ); @@ -782,7 +787,7 @@ sub facet { my $data = $self->_loaded('data'); my $filters = $self->_current_filters; - my $all_filters = join(' ',sort keys %$filters,'order:',$self->session('order')); + my $all_filters = __all_filters( keys %$filters,$self->session('order') ); my $filtered = $loaded->{$path}->{filtered}->{$all_filters} if defined $loaded->{$path}->{filtered}->{$all_filters};