Add a working "matches all keywords" query.

Diesel has a bug where array expression methods aren't implemented on
nullable expressions, so we can't just call Diesel's `.contains` method
until the fix lands. I've added a workaround for the time being. This
results in a runtime error since `keyword` was defined as `varchar`,
which coerces to `text`, but arrays do not. I've changed the column type
to address this. The keywords table is small enough and read
infrequently enough that I'm not concerned about the exclusive lock in
this migration.

Author: sgrif

migrations/2019-06-14-175833_change_keywords_to_text/down.sql

@@ -0,0 +1 @@
+ALTER TABLE keywords ALTER COLUMN keyword TYPE varchar;

migrations/2019-06-14-175833_change_keywords_to_text/up.sql

@@ -0,0 +1 @@
+ALTER TABLE keywords ALTER COLUMN keyword TYPE text;

src/controllers/krate/search.rs

@@ -96,14 +96,21 @@ pub fn search(req: &mut dyn Request) -> CargoResult<Response> {
     }
 
     if let Some(kws) = params.get("all_keywords") {
+        use diesel::sql_types::Array;
+        sql_function!(#[aggregate] fn array_agg<T>(x: T) -> Array<T>);
+
         let names: Vec<_> = kws.split_whitespace().map(|name| name.to_lowercase()).collect();
 
         query = query.filter(
-            crates::id.eq_any(
+            // FIXME: Just use `.contains` in Diesel 2.0
+            // https://github.com/diesel-rs/diesel/issues/2066
+            Contains::new(
                 crates_keywords::table
-                    .select(crates_keywords::crate_id)
                     .inner_join(keywords::table)
-                    .filter(crate::lower(keywords::keyword).eq(any(names))),
+                    .filter(crates_keywords::crate_id.eq(crates::id))
+                    .select(array_agg(keywords::keyword))
+                    .single_value(),
+                names.into_sql::<Array<Text>>(),
             ),
         );
     } else if let Some(kw) = params.get("keyword") {
@@ -241,3 +248,5 @@ pub fn search(req: &mut dyn Request) -> CargoResult<Response> {
         },
     }))
 }
+
+diesel_infix_operator!(Contains, "@>");