Fix parsing of COLLATE after parentheses in expressions (#507)

src/parser.rs

@@ -525,8 +525,8 @@ impl<'a> Parser<'a> {
                     };
                 self.expect_token(&Token::RParen)?;
                 if !self.consume_token(&Token::Period) {
-                    return Ok(expr);
+                    Ok(expr)
-                }
+                } else {
                     let tok = self.next_token();
                     let key = match tok {
                         Token::Word(word) => word.to_ident(),
@@ -537,6 +537,7 @@ impl<'a> Parser<'a> {
                         key,
                     })
                 }
+            }
             Token::Placeholder(_) => {
                 self.prev_token();
                 Ok(Expr::Value(self.parse_value()?))

tests/sqlparser_common.rs

@@ -561,6 +561,15 @@ fn parse_collate() {
     );
 }
 
+#[test]
+fn parse_collate_after_parens() {
+    let sql = "SELECT (name) COLLATE \"de_DE\" FROM customer";
+    assert_matches!(
+        only(&all_dialects().verified_only_select(sql).projection),
+        SelectItem::UnnamedExpr(Expr::Collate { .. })
+    );
+}
+
 #[test]
 fn parse_select_string_predicate() {
     let sql = "SELECT id, fname, lname FROM customer \