Scraper fixes (#332)

* Fix panic on invalid xpath

* Add missing attrs to scraped performer fragment

pkg/scraper/xpath.go

@@ -55,7 +55,12 @@ func (s xpathScraperConfig) process(doc *html.Node, common commonXPathConfig) []
 				asStr = common.applyCommon(asStr)
 			}
 
-			found := htmlquery.Find(doc, asStr)
+			found, err := htmlquery.QueryAll(doc, asStr)
+			if err != nil {
+				logger.Warnf("Error parsing xpath expression '%s': %s", asStr, err.Error())
+				continue
+			}
+
 			if len(found) > 0 {
 				for i, elem := range found {
 					if i >= len(ret) {

pkg/scraper/xpath_test.go

@@ -730,3 +730,20 @@ xPathScrapers:
 		return
 	}
 }
+
+func TestLoadInvalidXPath(t *testing.T) {
+	config := make(xpathScraperConfig)
+
+	config["Name"] = `//a[id=']/span`
+
+	reader := strings.NewReader(htmlDoc1)
+	doc, err := htmlquery.Parse(reader)
+
+	if err != nil {
+		t.Errorf("Error loading document: %s", err.Error())
+		return
+	}
+
+	common := make(commonXPathConfig)
+	config.process(doc, common)
+}