add empty like check

Author: ggggxiaolong

src/lib.rs

@@ -849,7 +849,7 @@ impl SqlBuilder {
         S: ToString,
         T: ToString,
     {
-        let expr = format!("{} = '{}'", &field.to_string(), &esc(&value.to_string()));
+        let expr = format!("{} = '{}'", &field.to_string(), &esc(value));
         self.sets.push(expr);
         self
     }
@@ -1247,8 +1247,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.and_where(&cond)
     }
@@ -1277,8 +1278,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty()  || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.and_where(&cond)
     }
@@ -1307,8 +1309,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.and_where(&cond)
     }
@@ -1337,8 +1340,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.and_where(&cond)
     }
@@ -1367,8 +1371,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.and_where(&cond)
     }
@@ -1397,8 +1402,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.and_where(&cond)
     }
@@ -1427,8 +1433,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.and_where(&cond)
     }
@@ -1457,8 +1464,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.and_where(&cond)
     }
@@ -1900,8 +1908,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.or_where(&cond)
     }
@@ -1931,8 +1940,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.or_where(&cond)
     }
@@ -1962,8 +1972,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.or_where(&cond)
     }
@@ -1993,8 +2004,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.or_where(&cond)
     }
@@ -2024,8 +2036,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.or_where(&cond)
     }
@@ -2055,8 +2068,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push('\'');
         self.or_where(&cond)
     }
@@ -2086,8 +2100,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.or_where(&cond)
     }
@@ -2117,8 +2132,9 @@ impl SqlBuilder {
         T: ToString,
     {
         let mut cond = field.to_string();
+        if cond.is_empty() || mask.to_string().is_empty() { return self; }
         cond.push_str(" NOT LIKE '%");
-        cond.push_str(&esc(&mask.to_string()));
+        cond.push_str(&esc(mask));
         cond.push_str("%'");
         self.or_where(&cond)
     }
@@ -3447,6 +3463,30 @@ mod tests {
 
         Ok(())
     }
+
+    #[test]
+    fn test_empty_like() -> Result<(), Box<dyn Error + Send + Sync>> {
+        let sql = SqlBuilder::select_from("books")
+            .and_where_like("", "")
+            .and_where_like_any("", "")
+            .and_where_like_left("", "")
+            .and_where_like_right("", "")
+            .and_where_not_like("", "")
+            .and_where_not_like_any("", "")
+            .and_where_not_like_left("", "")
+            .and_where_not_like_right("", "")
+            .or_where_like("", "")
+            .or_where_like_any("", "")
+            .or_where_like_left("", "")
+            .or_where_like_right("", "")
+            .or_where_not_like("", "")
+            .or_where_not_like_any("", "")
+            .or_where_not_like_left("", "")
+            .or_where_not_like_right("", "")
+            .sql()?;
+        assert_eq!(&sql, "SELECT * FROM books;");
+        Ok(())
+    }
 }
 
 //#[cfg(test)]