Now the selectors methods (css/_once, xpath/_once) works with str & String

Cargo.toml

@@ -14,4 +14,4 @@ http = "0.2"
 nipper = "0.1.8"
 serde_json = "1.0"
 serde = "1.0"
-cssifier = "0.1.2"
+cssifier = "0.1.3"

src/selector.rs

@@ -3,26 +3,25 @@ pub trait SelectorBase {
 
     fn html(&self) -> String;
 
-    // fn css<'a, T: SelectorBase>(&self, css_selector: &'a str) -> Vec<T> {
-    fn css<'a>(&self, css_selector: &'a str) -> Vec<Selector> {
+    fn css<S: AsRef<str>>(&self, css_selector: S) -> Vec<Selector> {
         let html = nipper::Document::from(self.html().as_str());
 
         let mut output = vec![];
 
-        for item in html.select(css_selector).iter() {
+        for item in html.select(css_selector.as_ref()).iter() {
             output.push(Selector::from_html(item.html().to_string()))
         }
         output
     }
 
-    fn css_once<'a>(&self, css_selector: &'a str) -> Option<Selector> {
-        self.css(css_selector).pop()
+    fn css_once<S: AsRef<str>>(&self, css_selector: S) -> Option<Selector> {
+        self.css(css_selector.as_ref()).pop()
     }
 
-    fn xpath(&self, xpath: &'static str) -> Vec<Selector> {
-        match cssifier::cssifier(xpath) {
+    fn xpath<S: AsRef<str>>(&self, xpath: S) -> Vec<Selector> {
+        match cssifier::cssifier(xpath.as_ref()) {
             Some(css_selector) => {
-                if css_selector == "" {
+                if css_selector.is_empty() {
                     Vec::default()
                 } else {
                     self.css(css_selector.as_str())
@@ -32,8 +31,8 @@ pub trait SelectorBase {
         }
     }
 
-    fn xpath_once(&self, xpath: &'static str) -> Option<Selector> {
-        self.xpath(xpath).pop()
+    fn xpath_once<S: AsRef<str>>(&self, xpath: S) -> Option<Selector> {
+        self.xpath(xpath.as_ref()).pop()
     }
 
     fn content(&self) -> String {
@@ -47,10 +46,9 @@ pub trait SelectorBase {
 
     fn attr<'a>(&self, attribute: &'a str) -> Option<String> {
         let html = nipper::Document::from(self.html().as_str());
-        match html.select("body > *").attr(attribute) {
-            Some(text) => Some(format!("{}", text)),
-            None => None,
-        }
+        html.select(" body > *")
+            .attr(attribute)
+            .map(|text| text.to_string())
     }
 }