rewite path check to lock only once

SebastiaanKlippert · Nov 5, 2018 · e692bbf · e692bbf
1 parent f959b70
commit e692bbf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,3 +2,4 @@ testfiles/*.pdf
 *.pdf
 .idea/
 *.exe
+wkhtmltopdf.exe.12.4
diff --git a/wkhtmltopdf.go b/wkhtmltopdf.go
@@ -226,8 +226,9 @@ func (pdfg *PDFGenerator) WriteFile(filename string) error {
 //a running program once it has been found
 func (pdfg *PDFGenerator) findPath() error {
 	const exe = "wkhtmltopdf"
-	if GetPath() != "" {
-		pdfg.binPath = GetPath()
+	pdfg.binPath = GetPath()
+	if pdfg.binPath != "" {
+		// wkhtmltopdf has already already found, return
 		return nil
 	}
 	exeDir, err := filepath.Abs(filepath.Dir(os.Args[0]))