Changeset 1431 for branches/release-30/lib/MT/App/Search.pm

Timestamp:

02/29/08 08:17:29 (21 months ago)

Author:

fumiakiy

Message:

Added more extensibility to the new MT::App::Search. BugId:68481

Files:

• branches/release-30/lib/MT/App/Search.pm (modified) (10 diffs)

branches/release-30/lib/MT/App/Search.pm

@@ -11 +11 @@
 
 use MT::Util qw( encode_html );
-use MT::Entry qw( :constants );
 
 sub id { 'new_search' }
@@ -27 +26 @@
     return {
         'search' => \&process,
-        'author' => \&process,
     };
 }
@@ -66 +64 @@
             $app->config->$key() : ($q->param($key) || $app->config->$key());
     }
+
+    $app->{searchparam}{Type} = $q->param('type') || 'entry';
 }
 
@@ -121 +121 @@
     my $app = shift;
 
-    my ( $terms, $args ) = $app->search_terms();
+    my @arguments = $app->search_terms();
     return $app->error($app->errstr) if $app->errstr;
 
     my $count = 0;
     my $iter;
-    if ( $terms && @$terms ) {
-        ( $count, $iter ) = $app->execute( $terms, $args );
+    if ( @arguments ) {
+        ( $count, $iter ) = $app->execute( @arguments );
         return $app->error($app->errstr) unless $iter;
 
@@ -147 +147 @@
     my ( $terms, $args ) = @_;
 
-    my $class = $app->model( $app->mode eq 'search' ? 'entry' : $app->mode );
+    my $class = $app->model( $app->{searchparam}{Type} )
+        or return $app->errtrans('Unsupported type: [_1]', $app->{searchparam}{Type});
     my $count = $app->count( $class, $terms, $args );
     #TODO: cache!
@@ -167 +168 @@
     $limit = $max if !$limit || ( $limit - $offset > $max );
 
-    my $type = $q->param('type');
-    my $entry_type = $app->mode eq 'search'
-      ? 1
-      : $type
-        ? ( 'entry' eq lc($type) || 'page' eq lc($type)
-          ? 1
-          : 0 )
-        : 0;
-
-    my $def_terms = {
-        $entry_type ? ( status => MT::Entry::RELEASE() ) : (),
-        $type ? ( class  => $type ) : (),
-        exists( $app->{searchparam}{IncludeBlogs} )
-          ? ( blog_id => [ keys %{ $app->{searchparam}{IncludeBlogs} } ] )
-          : (),
-    };
+    my $params = $app->registry( $app->{searchparam}{Type} );
+    my %def_terms = exists( $params->{terms} )
+          ? %{ $params->{terms} }
+          : ();
+    delete $def_terms{'plugin'}; #FIXME: why is this in here?
+
+    if ( exists $app->{searchparam}{IncludeBlogs} ) {
+        $def_terms{blog_id} = [ keys %{ $app->{searchparam}{IncludeBlogs} } ];
+    }
+
     my @terms;
-    push @terms, $def_terms if %$def_terms;
-
-    my $columns = $app->mode eq 'search'
-      ? [ [ qw( title keywords text text_more ) ] ]
-      : $app->registry( $app->mode, 'columns' );
-    $columns = $columns->[0]; # FIXME: Why?
-    return $app->errtrans('No columns to search for was specified for [_1]', $app->mode)
+    push @terms, \%def_terms if %def_terms;
+
+    my $columns = $params->{columns};
+    my $sort = $params->{'sort'};
+    if ( $sort !~ /\w+\!$/  && exists($app->{searchparam}{Sort}) ) {
+        $sort = $app->{searchparam}{Sort};
+    }
+
+    return $app->errtrans('No column was specified to search for [_1].', $app->{searchparam}{Type})
         unless $columns && @$columns;
+
     my $number = scalar @$columns;
     my @and;
@@ -208 +206 @@
       'sort' => [
         { desc   => 'descend' eq $app->{searchparam}{ResultDisplay} ? 'DESC' : 'ASC',
-          column => $entry_type ? 'authored_on' : 'created_on' }
+          column => $sort }
       ]
     );
@@ -296 +294 @@
     $ctx->stash('results', $iter);
     $ctx->stash('count',   $count);
-    $ctx->stash('stash_key', $app->mode)
-        if 'search' ne $app->mode;
+    $ctx->stash('stash_key', $app->{searchparam}{Type} );
     $ctx->stash('include_blogs',
         join ',', keys %{ $app->{searchparam}{IncludeBlogs} });
@@ -313 +310 @@
     my $app = shift;
     my ( $tmpl ) = @_;
-    $tmpl
+    $tmpl;
 }
 
@@ -322 +319 @@
     my $tmpl = $app->load_search_tmpl( $count, $iter );
     $tmpl = $app->pre_render( $tmpl );
-
     $tmpl;
 }